-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[docs][Charts] Give a general revision to the docs #11249
Conversation
A similar effort has already been merged on master and is cherry-picked on next #11239. So some conflict might be expected Especially on the nav bar structure, and the introduction page |
@alexfauquette, should I wait for you to finish/merge that other PR? Any recommendations? 😬 |
The other PR is ready to be merged. It's a cherry-pick of another PR that has already been accepted |
Ok, I'll wait for that and then resolve any conflicts with this one! |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
@flaviendelangle + @alexfauquette can y'all take a second look at this when possible? 😃 |
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.
👌
A quick fix on top of #11249.
title: 'Bar Chart', | ||
srcLight: '/static/x/component-illustrations/bar-light.png', | ||
srcDark: '/static/x/component-illustrations/bar-dark.png', | ||
href: '/x/react-charts/bar/', |
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.
Oops https://next.mui.com/x/react-charts/bar/
Fixed in 1a131e0.
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.
Thank you!
A quick fix on top of #11249.
This PR gives a general revision to the MUI X Charts space, proposing changes such as:
👉 https://deploy-preview-11249--material-ui-x.netlify.app/x/react-charts/