Skip to content

Conversation

siriwatknp
Copy link
Member

@mui-bot
Copy link

mui-bot commented Sep 16, 2025

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

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%) 🔺+1B(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%) 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 335afb3

siriwatknp and others added 2 commits September 16, 2025 17:42
Co-authored-by: Lukas Tyla <[email protected]>
Co-authored-by: Bernardo Belchior <[email protected]>
Co-authored-by: Alexandre Fauquette <[email protected]>
Signed-off-by: Siriwat K <[email protected]>
Co-authored-by: Bernardo Belchior <[email protected]>
Signed-off-by: Siriwat K <[email protected]>
Copy link
Member

Choose a reason for hiding this comment

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

We seem to be very inconsistent with the tagging in the titles. We use ...

  • ... PascalCase for data grid: DataGrid, DataGridPro and DataGridPremium
  • ... kebab-case for charts: charts and charts-pro
  • ... normal case for tree view: tree view and tree view pro

Historically we even use TreeView and tree-view as well, as @alexfauquette pointed out.

Should we decide on a pattern?

Copy link
Member

Choose a reason for hiding this comment

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

Couldn't we add those based on tags? So a scope: charts, plan: pro tag would add the title [charts-pro] to the title?

@siriwatknp siriwatknp merged commit 93406bb into mui:master Sep 17, 2025
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release We are shipping :D.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants