-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[charts] Add reference links to shared/misc chart components #10660
Conversation
Deploy preview: https://deploy-preview-10660--material-ui-x.netlify.app/ |
* - [Bars demonstration](https://mui.com/x/react-charts/bar-demo/) | ||
* - [Lines demonstration](https://mui.com/x/react-charts/line-demo/) | ||
* - [Area demonstration](https://mui.com/x/react-charts/area-demo/) | ||
* - [Pie demonstration](https://mui.com/x/react-charts/pie-demo/) |
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.
* - [Bars demonstration](https://mui.com/x/react-charts/bar-demo/) | |
* - [Lines demonstration](https://mui.com/x/react-charts/line-demo/) | |
* - [Area demonstration](https://mui.com/x/react-charts/area-demo/) | |
* - [Pie demonstration](https://mui.com/x/react-charts/pie-demo/) | |
* - [Bars demonstration](https://mui.com/x/react-charts/bars/) | |
* - [Lines demonstration](https://mui.com/x/react-charts/lines/) | |
* - [Pie demonstration](https://mui.com/x/react-charts/pie/) |
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 tend to think it's better to link people chare their is some explanation. But might be wrong
/** | ||
* API: | ||
* | ||
* - [DrawingProvider API](https://mui.com/x/api/charts/drawing-provider/) |
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.
Why not, but there is a lot to do for those pages to be readable 🫣
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 know ... but we have it in place once its ready, aye? 🏴☠️
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.
Ho yes, I agree with putting them. it was more to be sure you're also aware about it :)
3fc1a63
to
977eeab
Compare
This adds Demo and API links to the component annotation for ...
CartesianContextProvider
ChartsAxis
ChartsXAxis
ChartsYAxis
DrawingProvider
ChartsTooltip
ChartsClipPath
ChartsAxisHighlight
... components
It's part of a series to solve #9226