Skip to content

Conversation

oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Aug 1, 2025

This used to work, proof: https://github.com/d3/d3-shape/tree/v2.0.0?tab=readme-ov-file#stack-orders, but the docs got moved afterward.

Preview: https://deploy-preview-19020--material-ui-x.netlify.app/x/react-charts/stacking/#stacking-strategy


Off-topic. I landed here trying to use the stacking feature with an LLM. (v0, MUI Chat, Bolt, Lovable, etc).

@oliviertassinari oliviertassinari added docs Improvements or additions to the documentation. type: regression A bug, but worse, it used to behave as expected. scope: charts Changes related to the charts. labels Aug 1, 2025
@mui-bot
Copy link

mui-bot commented Aug 1, 2025

Deploy preview: https://deploy-preview-19020--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/DataGrid 0B(0.00%) 0B(0.00%)
@mui/x-data-grid-pro 0B(0.00%) 0B(0.00%)
@mui/x-data-grid-pro/DataGridPro 0B(0.00%) 0B(0.00%)
@mui/x-data-grid-premium 0B(0.00%) 0B(0.00%)
@mui/x-data-grid-premium/DataGridPremium 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 a67b897

Copy link

codspeed-hq bot commented Aug 1, 2025

CodSpeed Performance Report

Merging #19020 will not alter performance

Comparing oliviertassinari:fix-dead-link-to-d3 (a67b897) with master (ceaa46b)1

Summary

✅ 10 untouched benchmarks

Footnotes

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

@JCQuintas
Copy link
Member

JCQuintas commented Aug 2, 2025

I myself struggle to make sense of the docs, I need to read the demo's source carefully to understand, it feels like there are too many hops, so should have simpler code examples, for humans and LLMs. cc @mapache-salvaje for context.

The feature is complex, and I think we provide an api that is very close to D3. I've noticed we are missing two options though.

I can't think of a way to make the code easier for LLMS to understand, only how to simplify it a bit for real people.

What I think however, is that since these demos are complex, there is no "preview" and that is where the LLM would be able to understand how to use the properties in question. Note that the preview is not in the DOM if is it not expanded.

@JCQuintas JCQuintas merged commit 46fcaa0 into mui:master Aug 4, 2025
29 checks passed
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: regression A bug, but worse, it used to behave as expected.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants