-
Notifications
You must be signed in to change notification settings - Fork 94
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
Y-axis title behaves strangely in combination with responsive rules #284
Comments
The Highcharts responsive options are difficult to use with React JSX Highcharts to be honest, as it's not how updating the chart works behind the scenes. Plus it's not great for debugging either, as it means the React components don't really reflect the displayed state of the chart anymore. I would suggest using Let me know your thoughts. |
Hi Will, Thank you for replying so quickly! 👍 Yes, I can imagine the dynamic nature of the responsive feature doesn't match well with the way React handles things. I saw your great example using The only feedback I have at this point is that it may be useful to make a list of Highchart features that JSX does (and does not) support. You could even go as far as to change the BaseChart and emit a warning to the console if a developer tries to use a known unsupported features
Again, thank you for the great work on this library. |
Just to confirm the setup with Demo: https://omi.cr/sorting/counting-sort |
Thanks for the feedback! P.s. that site looks great, I should look into making a community showcase or something... |
Bug
Bug
Description
I'm trying to make my chart responsive by hiding the legend and y-axis title on smaller viewports. According to this Highcharts Responsive Demo, this can be achieved by passing the following object to the
reponsive
property on the root:With React JSX Highcharts I'm getting mixed results:
How to reproduce
Take any chart and add the responsive object posted above. Then resize the preview window beyond the breakpoint. The legend will (dis)appear as expected but the Y-axis title won't. I've made a live demo below with the responsive setup.
Live demo demonstrating bug
https://stackblitz.com/edit/base-chart-example-responsive
Versions
The text was updated successfully, but these errors were encountered: