Open oliviertassinari opened 4 months ago
Hi, @oliviertassinari. Would love to take up this issue, but I'm stuck at understanding the issue.
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?
@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.
@Janpot I have raised a PR, please check it.!
Steps to reproduce
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