mui-x icon indicating copy to clipboard operation
mui-x copied to clipboard

[charts] Extract pie layout computation

Open alexfauquette opened this issue 1 month ago • 2 comments

Simplify the pie logic by sharing the center and radius computation in the plugins.

The idea is to latter extract an independent FocussedPieSlice without having to duplicate the logic

alexfauquette avatar Dec 10 '25 11:12 alexfauquette

Deploy preview: https://deploy-preview-20611--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 🔺+199B(+0.06%) 🔺+36B(+0.03%)
@mui/x-charts-pro 🔺+199B(+0.04%) 🔺+65B(+0.05%)
@mui/x-charts-premium 🔺+199B(+0.04%) 🔺+16B(+0.01%)
@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 :no_entry_sign: dangerJS against 930b2f8d655fbe85913abc8e5970cab92287ff81

mui-bot avatar Dec 10 '25 11:12 mui-bot

CodSpeed Performance Report

Merging #20611 will not alter performance

Comparing alexfauquette:pie-chart-layout (930b2f8) with master (06e8498)[^unexpected-base] [^unexpected-base]: No successful run was found on master (2fec5b6) during the generation of this report, so 06e8498 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

Summary

✅ 13 untouched

codspeed-hq[bot] avatar Dec 10 '25 11:12 codspeed-hq[bot]