Skip to content

Conversation

@Janpot
Copy link
Member

@Janpot Janpot commented Nov 25, 2025

  • Remove route discovery through reading links, just pass it through with javascript and page.evaluate.
  • Don't await page.waitForFunction(() => window.muiFixture.isReady()); for every navigation, just do it once for all.
  • Don't move mouse in every test, we're not clicking links anymore to navigate between cases
  • Don't do emulate media in every test, just once for all should be fine
  • Do image loading in a single evaluation to avoid back and forth in the CDP multiple times
  • Don't use #dev, we don't need the links to be in the page for discovering tests. no need to render hundreds of links

@Janpot Janpot added the scope: code-infra Involves the code-infra product (https://www.notion.so/mui-org/5562c14178aa42af97bc1fa5114000cd). label Nov 25, 2025
@mui-bot
Copy link

mui-bot commented Nov 25, 2025

Deploy preview: https://deploy-preview-20441--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 5e13f57

@Janpot Janpot added the type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature. label Nov 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: code-infra Involves the code-infra product (https://www.notion.so/mui-org/5562c14178aa42af97bc1fa5114000cd). 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