Skip to content

Conversation

prakhargupta1
Copy link
Member

@prakhargupta1 prakhargupta1 commented Aug 6, 2025

I've not added individual docs pages because a user would quickly like to see the chart which is only on Github. It also helps get the upvote faster.

https://deploy-preview-19077--material-ui-x.netlify.app/x/react-charts/#all-mui-x-charts

@prakhargupta1 prakhargupta1 added docs Improvements or additions to the documentation. scope: charts Changes related to the charts. labels Aug 6, 2025
@mui-bot
Copy link

mui-bot commented Aug 6, 2025

Deploy preview: https://deploy-preview-19077--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%) ▼-1B(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%) ▼-1B(0.00%)
@mui/x-date-pickers 0B(0.00%) 0B(0.00%)
@mui/x-date-pickers-pro 0B(0.00%) ▼-1B(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 4d8cb91

Copy link

codspeed-hq bot commented Aug 6, 2025

CodSpeed Performance Report

Merging #19077 will improve performances by 9.36%

Comparing prakhargupta1:more-charts (4d8cb91) with master (cdea241)1

Summary

⚡ 1 improvements
✅ 9 untouched benchmarks

Benchmarks breakdown

Benchmark BASE HEAD Change
ScatterChartPro with big data amount 438.6 ms 401.1 ms +9.36%

Footnotes

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

@prakhargupta1 prakhargupta1 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 Aug 6, 2025
@prakhargupta1 prakhargupta1 marked this pull request as ready for review August 6, 2025 15:39
srcDark: '/static/x/component-illustrations/treemap-dark.png',
href: '/x/react-charts/treemap/',
planned: true,
pro: true,
Copy link
Member

Choose a reason for hiding this comment

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

Pricing page on material-ui repository shoudl be updated too

Copy link
Member Author

Choose a reason for hiding this comment

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

@bernardobelchior
Copy link
Member

The list is getting quite big, should we have a "Planned" section instead so we can remove all these "Planned" pills which mostly add noise?

image

@prakhargupta1
Copy link
Member Author

Made two sections and also removed component illustrations.

@JCQuintas
Copy link
Member

Made two sections and also removed component illustrations.

I would probably make them a grid of cards rather than a list, we can then style the existing ones blue-ish as the feature cards above without the icons though, and the planned ones with a gray-ish background instead?

long lists are boring 😆

@prakhargupta1
Copy link
Member Author

long lists are boring 😆

Agree, I just wanted to give a break to the user as the entire page is already quite overwhelming. Also, this is a faster implementation while styling icons/illustrations can take time, we can do a revamp later if needed.

@prakhargupta1 prakhargupta1 merged commit 0a432d3 into mui:master Aug 8, 2025
23 checks passed
@prakhargupta1 prakhargupta1 deleted the more-charts branch August 8, 2025 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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
Development

Successfully merging this pull request may close these issues.

5 participants