-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
[docs] Provide workaround for pie chart composition #18600
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Deploy preview: https://deploy-preview-18600--material-ui-x.netlify.app/ Updated pages: Bundle size reportTotal Size Change: 0B(0.00%) - Total Gzip Change: ▼-79B(0.00%) Show details for 100 more bundles (22 more not shown)@mui/x-charts parsed: 0B(0.00%) gzip: 0B(0.00%) |
CodSpeed Performance ReportMerging #18600 will degrade performances by 21.01%Comparing Summary
Benchmarks breakdown
|
docs/data/charts/pie/pie.md
Outdated
|
||
This is a workaround the time to build a proper data provider for the pie chart. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a workaround the time to build a proper data provider for the pie chart. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think mentioning this is necessary or helps
Co-authored-by: Bernardo Belchior <Bernardo.belchior1@gmail.com> Signed-off-by: Alexandre Fauquette <45398769+alexfauquette@users.noreply.github.com>
About the workaround. My idea was to mention that it's normal that this behavior look weird. It's a workaround |
Maybe we can create an issue and link it? "This is a workaround which we plan to fix in the future (#issue number)" |
Document pie center issue with composition #18538