Skip to content

[charts] Remove unused generics from bar charts#20642

Merged
bernardobelchior merged 1 commit intomui:masterfrom
bernardobelchior:revert-unused-changes-bar-chart
Dec 16, 2025
Merged

[charts] Remove unused generics from bar charts#20642
bernardobelchior merged 1 commit intomui:masterfrom
bernardobelchior:revert-unused-changes-bar-chart

Conversation

@bernardobelchior
Copy link
Copy Markdown
Member

These were added in preparation for #20275, but probably won't be used.

@bernardobelchior bernardobelchior requested a review from a team as a code owner December 12, 2025 13:42
@bernardobelchior bernardobelchior added 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 Dec 12, 2025
@mui-bot
Copy link
Copy Markdown

mui-bot commented Dec 12, 2025

Deploy preview: https://deploy-preview-20642--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%) 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%)

Details of bundle changes

Generated by 🚫 dangerJS against 42d0d9a

@bernardobelchior bernardobelchior force-pushed the revert-unused-changes-bar-chart branch from a3f2d5e to c8d52d6 Compare December 12, 2025 13:48
@bernardobelchior bernardobelchior force-pushed the revert-unused-changes-bar-chart branch from c8d52d6 to 42d0d9a Compare December 12, 2025 13:57
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Dec 12, 2025

CodSpeed Performance Report

Merging #20642 will not alter performance

Comparing bernardobelchior:revert-unused-changes-bar-chart (42d0d9a) with master (9bda61c)1

Summary

✅ 13 untouched

Footnotes

  1. No successful run was found on master (52d6d8b) during the generation of this report, so 9bda61c was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

@bernardobelchior bernardobelchior merged commit 139a02a into mui:master Dec 16, 2025
22 checks passed
@bernardobelchior bernardobelchior deleted the revert-unused-changes-bar-chart branch December 16, 2025 09:24
siriwatknp pushed a commit to siriwatknp/material-ui-x that referenced this pull request Dec 19, 2025
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants