Skip to content

[charts] Rename ChartZoomSlider to ChartsZoomSlider - #21553

Merged
JCQuintas merged 4 commits into
mui:masterfrom
JCQuintas:rename-chartslider-chartsslider
Mar 4, 2026
Merged

[charts] Rename ChartZoomSlider to ChartsZoomSlider#21553
JCQuintas merged 4 commits into
mui:masterfrom
JCQuintas:rename-chartslider-chartsslider

Conversation

@JCQuintas

@JCQuintas JCQuintas commented Mar 2, 2026

Copy link
Copy Markdown
Member

Fixes #20353

ChartZoomSlider → ChartsZoomSlider
ChartAxisZoomSlider → ChartsAxisZoomSlider
ChartAxisZoomSliderTrack → ChartsAxisZoomSliderTrack
ChartAxisZoomSliderThumb → ChartsAxisZoomSliderThumb
ChartAxisZoomSliderPreview → ChartsAxisZoomSliderPreview
ChartAxisZoomSliderPreviewContent → ChartsAxisZoomSliderPreviewContent
ChartAxisZoomSliderActiveTrack → ChartsAxisZoomSliderActiveTrack

@JCQuintas JCQuintas self-assigned this Mar 2, 2026
@JCQuintas JCQuintas added breaking change Introduces changes that are not backward compatible. type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature. needs cherry-pick The PR should be cherry-picked to master after merge. scope: charts Changes related to the charts. v8.x labels Mar 2, 2026
@mui-bot

mui-bot commented Mar 2, 2026 •
edited
Loading

Copy link
Copy Markdown

Deploy preview: https://deploy-preview-21553--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%) 0B(0.00%)
@mui/x-charts 0B(0.00%) 0B(0.00%)
@mui/x-charts-pro 🔺+110B(+0.02%) 🔺+45B(+0.03%)
@mui/x-charts-premium 🔺+110B(+0.02%) 🔺+25B(+0.02%)
@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 84ffe55

@codspeed

codspeed Bot commented Mar 2, 2026 •
edited
Loading

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 14 untouched benchmarks

Comparing JCQuintas:rename-chartslider-chartsslider (84ffe55) with master (4c9b019)

Open in CodSpeed

bernardobelchior
bernardobelchior reviewed Mar 3, 2026
View reviewed changes
Comment thread scripts/x-charts-premium.exports.json Resolved
@JCQuintas
JCQuintas enabled auto-merge (squash) March 3, 2026 18:11
bernardobelchior
bernardobelchior approved these changes Mar 4, 2026
View reviewed changes

@bernardobelchior bernardobelchior left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good 👍

The PR has a breaking change label, but this PR isn't a breaking change. We're just deprecating stuff and adding new components/types.

@JCQuintas
JCQuintas merged commit c1e5b60 into mui:master Mar 4, 2026
22 checks passed
@JCQuintas
JCQuintas deleted the rename-chartslider-chartsslider branch March 4, 2026 07:26
@github-actions

github-actions Bot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

Cherry-pick PRs will be created targeting branches: v8.x

github-actions Bot pushed a commit that referenced this pull request Mar 4, 2026
@JCQuintas @web-flow
[charts] Rename ChartZoomSlider to ChartsZoomSlider (#21553)
0a17d01
@github-actions github-actions Bot mentioned this pull request Mar 4, 2026
Merged
@JCQuintas JCQuintas removed the breaking change Introduces changes that are not backward compatible. label Mar 4, 2026
@JCQuintas JCQuintas mentioned this pull request Mar 4, 2026
Merged
@oliviertassinari oliviertassinari added the deprecation New deprecation message. label May 15, 2026
@oliviertassinari oliviertassinari mentioned this pull request May 15, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bernardobelchior bernardobelchior bernardobelchior approved these changes
@alexfauquette alexfauquette Awaiting requested review from alexfauquette alexfauquette is a code owner

Assignees

@JCQuintas JCQuintas

Labels

deprecation New deprecation message. needs cherry-pick The PR should be cherry-picked to master after merge. 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. v8.x

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[charts] v9 component renaming

4 participants

@JCQuintas @mui-bot @bernardobelchior @oliviertassinari