Skip to content

Conversation

alexfauquette
Copy link
Member

@alexfauquette alexfauquette commented Aug 25, 2025

Alternative solution is #19336

Before reviewing this PR, might be better to agree on the DX in #19343

We are moving with the following steps:

  • First the one you already did in the other PR, export from file
  • Then change import references in a new PR
  • Change file locations in a last PR

@mui-bot
Copy link

mui-bot commented Aug 25, 2025

Deploy preview: https://deploy-preview-19335--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%) 🔺+5B(0.00%)
@mui/x-data-grid-premium 0B(0.00%) 🔺+4B(0.00%)
@mui/x-charts 🔺+156B(+0.05%) ▼-371B(-0.39%)
@mui/x-charts-pro 🔺+198B(+0.05%) 🔺+54B(+0.05%)
@mui/x-date-pickers 0B(0.00%) 0B(0.00%)
@mui/x-date-pickers-pro 0B(0.00%) 🔺+7B(+0.01%)
@mui/x-tree-view 0B(0.00%) 0B(0.00%)
@mui/x-tree-view-pro 0B(0.00%) 🔺+3B(+0.01%)

Details of bundle changes

Generated by 🚫 dangerJS against 545e414

@alexfauquette alexfauquette added dx Related to developers' experience. 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 Aug 26, 2025
Copy link

codspeed-hq bot commented Aug 26, 2025

CodSpeed Performance Report

Merging #19335 will not alter performance

Comparing alexfauquette:export-plugins-2 (545e414) with master (90d94c7)1

Summary

✅ 10 untouched benchmarks

Footnotes

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

@alexfauquette alexfauquette marked this pull request as ready for review August 26, 2025 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dx Related to developers' experience. 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