Skip to content

[docs] Remove preview from Toolbar & Funnel #19131

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
Aug 11, 2025
Merged

[docs] Remove preview from Toolbar & Funnel #19131

merged 3 commits into from
Aug 11, 2025

Conversation

mnajdova
Copy link
Member

@mnajdova mnajdova added the docs Improvements or additions to the documentation. label Aug 11, 2025
@mui-bot
Copy link

mui-bot commented Aug 11, 2025 •
edited
Loading

Deploy preview: https://deploy-preview-19131--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 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 57aedc1

@bernardobelchior bernardobelchior added 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. labels Aug 11, 2025
bernardobelchior
bernardobelchior reviewed Aug 11, 2025
View reviewed changes
@codspeed-hq CodSpeed HQ
Copy link

codspeed-hq bot commented Aug 11, 2025 •
edited
Loading

CodSpeed Performance Report

Merging #19131 will improve performances by 8.76%

Comparing mnajdova:docs/preview-label (57aedc1) with master (44aeba4)1

Summary

⚡ 1 improvements
✅ 9 untouched benchmarks

Benchmarks breakdown

Benchmark BASE HEAD Change
⚡ ScatterChartPro with big data amount 437.1 ms 401.9 ms +8.76%

Footnotes

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

@JCQuintas JCQuintas merged commit 61a4f97 into mui:master Aug 11, 2025
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JCQuintas JCQuintas JCQuintas approved these changes

@bernardobelchior bernardobelchior bernardobelchior approved these changes

Assignees
No one assigned
Labels
docs Improvements or additions to the documentation. 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
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mnajdova @mui-bot @JCQuintas @bernardobelchior