Deploy preview: https://deploy-preview-13859--material-ui-x.netlify.app/

Updated pages:

• docs/data/charts/getting-started/getting-started.md
• docs/data/charts/heatmap/heatmap.md
• docs/data/charts/zoom-and-pan/zoom-and-pan.md
• docs/data/introduction/licensing/licensing.md

Generated by 🚫 dangerJS against 8b9ed6a

Cool to see near its release.

All the things that I can spot on https://deploy-preview-13859--material-ui-x.netlify.app/x/react-charts/heatmap/:

1. import { UnstableHeatmap } from '@mui/x-charts-pro/Heatmap'; is usually import { Unstable_Heatmap } from '@mui/x-charts-pro/Heatmap';
2. import '@mui/x-charts-pro/typeOverloads'; in the demo seems strange to me. I would expect this to be in the npm package
3. Should https://deploy-preview-13859--material-ui-x.netlify.app/x/react-charts/zoom-and-pan/ also have the "Preview" badge? It's released as 7.0.0-alpha.0
4. https://deploy-preview-13859--material-ui-x.netlify.app/x/introduction/licensing/#pro-plan should also list the charts?
5. The JS file isn't converted

6. The popover might be able to get too close to the edge of the screen:

7. https://deploy-preview-13859--material-ui-x.netlify.app/x/react-charts/zoom-and-pan/#zooming-options Sentence case
8. I'm not sold on the color contrast in dark mode https://deploy-preview-13859--material-ui-x.netlify.app/x/react-charts/zoom-and-pan/#zooming-options, both series looks hard to separate.

9. The zoom feature is very cool. However, I'm confused about the UX. Moved to [charts][zoom] Allow to zoom on data #12503 (comment).

10. Ads should be disabled on https://deploy-preview-13859--material-ui-x.netlify.app/x/react-charts/heatmap/, https://deploy-preview-13859--material-ui-x.netlify.app/x/react-charts/zoom-and-pan/

This pull request has conflicts, please resolve those before we can evaluate the pull request.

1. Done. The Unstable is useless since we are in alpha

2. Using the import '@mui/x-charts-pro/typeOverloads'; in the index.ts is not enough. To fix TS, I needed to add it to the Heatmap/index.ts. I don't know where else to put that

3. Added "Preview" badge to zoom and pan link

4. I added charts to the licencing page. @joserodolfofreitas should we rewrite some sections?

5. The JS file isn't converted -> can't do a lot about that. It's related to [docs-infra] Multi-tab demo component improvements material-ui#41754

6. The popover might be able to get too close to the edge of the screen. -> I don't get the point? The issue I see is the left axis not getting enough space, but no problem with the popover

7. https://deploy-preview-13859--material-ui-x.netlify.app/x/react-charts/zoom-and-pan/#zooming-options Sentence case -> I assume it's the "O" of "Zooming Options"

8. I'm not sold on the color contrast in dark mode https://deploy-preview-13859--material-ui-x.netlify.app/x/react-charts/zoom-and-pan/#zooming-options, both series looks hard to separate. -> We can change them, but in a major since it's the default colors of the charts

9. The zoom feature is very cool. However, I'm confused about the UX. Moved to [charts][zoom] Allow to zoom on data #12503 (comment).

10. Ads have been disabled

Thanks for the updates.

6. The popover might be able to get too close to the edge of the screen. -> I don't get the point? The issue I see is the left axis not getting enough space, but no problem with the popover

I have this in mind mui/base-ui#39. But maybe we can't fix this yet.

Per https://mui-org.slack.com/archives/C02EQ97NV8F/p1721308954099199, I suspect that we would want to merge this PR but keep "private": true in the package.json since those pages are already live, e.g. https://mui.com/x/react-charts/heatmap/.

This pull request has conflicts, please resolve those before we can evaluate the pull request.

CodSpeed Performance Report

Merging #13859 will not alter performance

_{Comparing alexfauquette:prepare-pro-release (8b9ed6a) with master (d54212d)}

Summary

✅ 3 untouched benchmarks