Skip to content

[DataGridPro] Keep children in the tree after parent row is re-fetched with the data source #19934

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

[DataGridPro] Keep children in the tree after parent row is re-fetched with the data source #19934

wants to merge 9 commits into from
+686 −55

Conversation

arminmeh
Copy link
Contributor

@arminmeh arminmeh commented Oct 13, 2025 •
edited
Loading

Continuation on #19873

This PR solves the UX issue described in #19799

  1. While refetching the whole path, the rows are jumping, because each level is fetched in another request

It also aligns the behavior with the client-side, where we want to persist the tree as much as possible.

By default, grid now tries to keep the child rows intact. Users can opt out of this behavior and pass collapseChildren: true, to remove all child rows from the grid once the parent is re-fetched.

TODO:

  • Update docs to make collapseChildren more visible

@arminmeh arminmeh added scope: data grid Changes related to the data grid. plan: Pro Impact at least one Pro user. feature: Tree data Related to the data grid Tree data feature feature: Row grouping Related to the data grid Row grouping feature type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature. feature: Server integration Better integration with backends, e.g. data source labels Oct 13, 2025
@mui-bot
Copy link

mui-bot commented Oct 13, 2025 •
edited
Loading

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

Updated pages:

Bundle size report

Bundle Parsed size Gzip size
@mui/x-data-grid 🔺+235B(+0.06%) 🔺+39B(+0.03%)
@mui/x-data-grid-pro 🔺+966B(+0.20%) 🔺+237B(+0.17%)
@mui/x-data-grid-premium 🔺+1.21KB(+0.19%) 🔺+292B(+0.15%)
@mui/x-charts 0B(0.00%) 0B(0.00%)
@mui/x-charts-pro 0B(0.00%) 0B(0.00%)
@mui/x-charts-premium 0B(0.00%) 0B(0.00%)
@mui/x-date-pickers 0B(0.00%) 0B(0.00%)
@mui/x-date-pickers-pro 0B(0.00%) 0B(0.00%)
@mui/x-tree-view 0B(0.00%) 0B(0.00%)
@mui/x-tree-view-pro 0B(0.00%) 0B(0.00%)

Details of bundle changes

Generated by 🚫 dangerJS against a704b15

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MBilalShafi MBilalShafi Awaiting requested review from MBilalShafi

@cherniavskii cherniavskii Awaiting requested review from cherniavskii

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

feature: Row grouping Related to the data grid Row grouping feature feature: Server integration Better integration with backends, e.g. data source feature: Tree data Related to the data grid Tree data feature plan: Pro Impact at least one Pro user. scope: data grid Changes related to the data grid. type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@arminmeh @mui-bot