Skip to content

[charts] Simplify classes type - #22626

Merged
JCQuintas merged 4 commits into
mui:masterfrom
alexfauquette:simplify-types
Jun 23, 2026
Merged

JCQuintas merged 4 commits into
mui:masterfrom
alexfauquette:simplify-types

Conversation

@alexfauquette

Copy link
Copy Markdown
Member

Test after merging #22591 if it simplifies types

@alexfauquette alexfauquette added typescript type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature. scope: charts Changes related to the charts. labels May 29, 2026
@code-infra-dashboard

code-infra-dashboard Bot commented May 29, 2026 •

Copy link
Copy Markdown

Deploy preview

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

Bundle size

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%) 0B(0.00%)
@mui/x-charts 🔺+1B(0.00%) 🔺+4B(0.00%)
@mui/x-charts-pro 🔺+1B(0.00%) 🔺+5B(0.00%)
@mui/x-charts-premium 🔺+1B(0.00%) 🔺+5B(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%)
@mui/x-license 0B(0.00%) 0B(0.00%)

Details of bundle changes

Performance

Total duration: 2,200.40 ms -127.47 ms(-5.5%) | Renders: 63 (+0)

Test Duration Renders
RadarChart with many axes and multiple series 9.62 ms 🔺+1.91 ms(+24.8%) 2 (+0)
ScatterChartPremium with big data amount (webgl renderer) 87.53 ms ▼-22.82 ms(-20.7%) 5 (+0)
LineChart with big data amount (with marks) 77.91 ms ▼-20.89 ms(-21.1%) 2 (+0)
LineChartPro with big data amount and zoomed in (with marks) 29.89 ms ▼-11.50 ms(-27.8%) 2 (+0)

22 tests within noise — details


Check out the code infra dashboard for more information about this PR.

@alexfauquette
alexfauquette marked this pull request as ready for review June 23, 2026 08:45
@JCQuintas
JCQuintas merged commit 8d5b1ef into mui:master Jun 23, 2026
21 checks passed
mbrookes pushed a commit to mbrookes/mui-x that referenced this pull request Jun 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: charts Changes related to the charts. type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature. typescript

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants