Skip to content

[charts-premium] Share helper function#23030

Merged
alexfauquette merged 1 commit into
mui:masterfrom
alexfauquette:share-visible-helper
Jul 1, 2026
Merged

[charts-premium] Share helper function#23030
alexfauquette merged 1 commit into
mui:masterfrom
alexfauquette:share-visible-helper

Conversation

@alexfauquette

Copy link
Copy Markdown
Member

Moving the helper function to internal to reuse it in the map zoom PR

@alexfauquette
alexfauquette requested a review from JCQuintas as a code owner July 1, 2026 12:42
@alexfauquette alexfauquette added plan: Premium Impact at least one Premium user. 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 Jul 1, 2026
@code-infra-dashboard

code-infra-dashboard Bot commented Jul 1, 2026

Copy link
Copy Markdown

Deploy preview

https://deploy-preview-23030--material-ui-x.netlify.app/
QR code for https://deploy-preview-23030--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 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%)
@mui/x-scheduler 0B(0.00%) 0B(0.00%)
@mui/x-scheduler-premium 0B(0.00%) 0B(0.00%)
@mui/x-license 0B(0.00%) 0B(0.00%)

Details of bundle changes

Performance

Total duration: 1,797.36 ms +29.43 ms(+1.7%) | Renders: 63 (+0)

Test Duration Renders
ScatterChartPremium with big data amount (webgl renderer) 88.76 ms 🔺+16.74 ms(+23.2%) 5 (+0)
ScatterChartPro with big data amount and zoomed in (single renderer) 12.06 ms 🔺+3.83 ms(+46.5%) 2 (+0)
FunnelChart with big data amount 5.66 ms 🔺+1.07 ms(+23.4%) 2 (+0)
ScatterChartPremium with big data amount and zoomed in (webgl renderer) 91.79 ms +13.95 ms(+17.9%) 5 (+0)

22 tests within noise — details

Metric alarms

Test Metric Change
ScatterChartPro with big data amount and zoomed in (single renderer) bench:paint 🔺 +4.69 ms
ScatterChartPremium with big data amount and zoomed in (webgl renderer) bench:paint 🔺 +16.38 ms

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

@alexfauquette
alexfauquette merged commit 4c4307f into mui:master Jul 1, 2026
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

plan: Premium Impact at least one Premium user. 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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants