-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Add support for responsive charts #2969
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
Comments
Good call! This would probably work best as new config option (i.e. I don't think this should be a layout attribute). |
This could have interesting implications (I think positively so!) for Plotly's own embed and react-plotly.js |
This would be basically baking in https://plot.ly/javascript/responsive-fluid-layout/ via a config option? Seems helpful, indeed, but also kind of incomplete... Lots of people in the |
We could even shoot for moving all the embedplot logic in plotly.js. |
In the meantime, I will submit a PR doing what most (if not all) other plotting libraries claiming to be responsive do: simply react to window resize events. |
Closed by PR #2974 |
Let's make sure we get this in the documentation as well! :) /cc @cldougl what's the process for doing this? |
Yep- you can open an issue in the documentation repo: https://github.com/plotly/documentation/issues |
In order for charts to look great on any devices/orientation, I suggest that we add the option to resize charts whenever the window's size changes.
For example, it is common for a user to switch his phone from landscape to portrait mode which results in the following unaesthetic/broken result:
The text was updated successfully, but these errors were encountered: