[DataGrid] Trigger row spanning computation on rows update

[MBilalShafi]

Fixes #15819

• Before: https://grfpfs.csb.app/
• After: https://wglxzl.csb.app/

Addresses a corner case, whenever rows get updated and the (index based) range doesn't change, it results in skipping the necessary re-computation of row spanning state.

• Add test coverage

[mui-bot]

Deploy preview: https://deploy-preview-15858--material-ui-x.netlify.app/

Generated by 🚫 dangerJS against 5dfe4d6

[arminmeh]

In the after preview, after clicking couple of times, the rows are back, but they are combined anymore

[MBilalShafi]

Thank you @arminmeh for pointing it out.

There were two issues.

1. The demo didn't have rowSpanning prop passed 😄
2. There was a more serious issue about the indexes of render context, which were designed in a way that the row spanning won't apply on a single row, which was intended in the beginning, since it doesn't make sense to span one row, but this corner case caused problem when we shuffle rows that have already a computed state.

Updated the demo and it seems to be working now.

[github-actions]

Cherry-pick PRs will be created targeting branches: v7.x

[kashyap-cephei]

Hi team,

I came across this thread and believe I'm experiencing a similar issue. When updating rows in the DataGrid, the row spanning computation doesn't seem to re-trigger if the index-based range remains unchanged.

I've recorded a short video showing the behavior. Could you please help confirm if this is the same issue as reported in #15819 and fixed in #15858?

Thank you for your assistance!

Recording.2025-01-04.010645-VEED.mp4

[cherniavskii]

Hey @kashyap-cephei
It looks like the same issue, but it's hard to say based on a video.
I recommend you upgrade to /~https://github.com/mui/mui-x/releases/tag/v7.23.3 which includes the fix.
If you want to make sure the issue is fixed before upgrading – create a minimal reproduction example on Codesandbox/Stackblitz and try changing the data grid version in package.json. You can start with this demo by using Stackblitz/Codesandbox buttons under the demo.