[studio] Cell shouldn't be editable by default

[oliviertassinari]

Steps to reproduce

1. Open https://tools-public.mui.com/prod/pages/baseUiNpmKpis
2. 🙃

https://github.com/mui/mui-toolpad/assets/3165635/fcccf85c-6035-40d3-a2ad-6401ca0a900e

Context

I suspect https://github.com/mui/mui-toolpad/pull/3001 is the origin of this regression. I noticed this one on the same app that I noticed the other regressions from (the Zendesk plugin), while I was fixing https://github.com/mui/mui-private/pull/470.

Benchmarks

• Retool does this right:

[b4s36t4]

Hi, @oliviertassinari. Would love to take up this issue, but I'm stuck at understanding the issue.

• Currently by default all cells are editable, which shouldn't be the case.
• We should let the user choose which column should be editable in the properties panel.

This is what I did understand, please let me know if it's not the case, and in case the it's same is it possible for me take up this one?

[Janpot]

@b4s36t4 The data grid should only be editable when it is linked to a data provider that supports editing rows. In all other cases there won't be any logic available to handle the row updates, so having the grid editable makes no sense.

As you may notice in the live example, you can change the value of the cel, but as soon as you exit editing mode, the value is reset again. This is meant to be a static grid, output only.

[b4s36t4]

@Janpot I have raised a PR, please check it.!