Skip to content

Conversation

romgrk
Copy link
Contributor

@romgrk romgrk commented Sep 2, 2025

Closes #19167

Clarify the need to have intrinsic dimensions.

@romgrk romgrk added docs Improvements or additions to the documentation. 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. labels Sep 2, 2025
@mui-bot
Copy link

mui-bot commented Sep 2, 2025

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

Updated pages:

Bundle size report

Bundle Parsed size Gzip size
@mui/x-data-grid 0B(0.00%) 0B(0.00%)
@mui/x-data-grid-pro 0B(0.00%) 0B(0.00%)
@mui/x-data-grid-premium 0B(0.00%) ▼-1B(0.00%)
@mui/x-charts 0B(0.00%) 0B(0.00%)
@mui/x-charts-pro 0B(0.00%) ▼-1B(0.00%)
@mui/x-date-pickers 0B(0.00%) 0B(0.00%)
@mui/x-date-pickers-pro 0B(0.00%) 🔺+1B(0.00%)
@mui/x-tree-view 0B(0.00%) 0B(0.00%)
@mui/x-tree-view-pro 0B(0.00%) 🔺+1B(0.00%)

Details of bundle changes

Generated by 🚫 dangerJS against ec393ef

:::error
By default, the Data Grid has **no intrinsic dimensions**.
Instead, it takes up the space given by its parent.
The Data Grid will raise an error in the console if its container has no intrinsic dimensions.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not true due to this. If we want the check, we need to rework it and it would be a different rendering sequence in dev than in prod. I don't have a strong opinion, I'd prefer to have feedback from the @mui/xgrid team for how to proceed.

@romgrk romgrk requested review from a team and mapache-salvaje September 2, 2025 19:10
romgrk and others added 3 commits September 3, 2025 14:35
Co-authored-by: mapache-salvaje <[email protected]>
Signed-off-by: Rom Grk <[email protected]>
Co-authored-by: mapache-salvaje <[email protected]>
Signed-off-by: Rom Grk <[email protected]>
Co-authored-by: mapache-salvaje <[email protected]>
Signed-off-by: Rom Grk <[email protected]>
@romgrk romgrk merged commit 86f8dbd into mui:master Sep 24, 2025
21 checks passed
@romgrk romgrk deleted the docs-intrinsic-dimensions branch September 24, 2025 03:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to the documentation. 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

Development

Successfully merging this pull request may close these issues.

[data grid] Crashes randomly since 8.9.2

4 participants