-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
[docs] Add scatter chart with linear regression demo #19900
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] Add scatter chart with linear regression demo #19900
Conversation
e75e9d7 to
017ab99
Compare
|
Deploy preview: https://deploy-preview-19900--material-ui-x.netlify.app/ Updated pages: Bundle size report
|
| See the [Composition—clipping](/x/react-charts/composition/#clipping) for more info. | ||
| ::: | ||
|
|
||
| ### Regression line |
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.
Placed this under composition because it requires composition, but not sure if it's the best place.
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.
Another option is to move it into the scatter-demo page. You might get more space to describe the different feature you used
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 was under the impression we wanted to remove the demo pages, isn't that right? That's why I placed it here
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.
We're not removing individual demo pages. I think this demo placed under Composition is fine, any new demo can go in the demo page.
| maximumFractionDigits: 0, | ||
| }); | ||
|
|
||
| export default function ScatterRegressionLine() { |
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.
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.
Maybe a softer red or decreased opacity.
Maybe add a vertical dashed line where the line exits the 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.
If you want a better red, you can pick the one from rainbowSurgePalette:
isLightTheme ? '#FA4F58' : '#F35865'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.
Done 👍
CodSpeed Performance ReportMerging #19900 will not alter performanceComparing Summary
Footnotes |
Related to #19869.
Add scatter chart with linear regression demo.