-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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] Divide charts tooltip
and highlighting
pages
#14824
[docs] Divide charts tooltip
and highlighting
pages
#14824
Conversation
CodSpeed Performance ReportMerging #14824 will not alter performanceComparing Summary
|
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 would like to do line-by-line reviews of both docs in the near future but this looks good for now. Good call on separating them 👍
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.
Great work. 👍
Cleaner result. 👌
Co-authored-by: Lukas Tyla <[email protected]> Signed-off-by: Jose C Quintas Jr <[email protected]>
@@ -88,7 +88,7 @@ BarChartPro.propTypes = { | |||
* The configuration of axes highlight. | |||
* Default is set to 'band' in the bar direction. | |||
* Depends on `layout` prop. | |||
* @see See {@link https://mui.com/x/react-charts/tooltip/#highlights highlight docs} for more details. | |||
* @see See {@link https://mui.com/x/react-charts/highlighting highlighting docs} for more details. |
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.
* @see See {@link https://mui.com/x/react-charts/highlighting highlighting docs} for more details. | |
* @see See {@link https://mui.com/x/react-charts/highlighting/ highlighting docs} for more details. |
Fixed in 23646d8
A side note, we will move to non-trailing URLs, at one point mui/mui-public#248.
highlight
sections fromtooltip
pagehighlighting
page for that aspect