Skip to content

When mouse-zooming with 3d orthographic projection, no exposed property is updated #4274

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

Closed
emmanuelle opened this issue Oct 11, 2019 · 5 comments · Fixed by #4292
Closed
Assignees
Labels
bug something broken
Milestone

Comments

@emmanuelle
Copy link
Contributor

See the codepen https://codepen.io/emmanuelle-plotly/pen/QWWyjxg?editors=0011 and try to zoom using the mouse wheel. One would expect one property of the figure to be updated, such as scene.camera.eye or scene.aspectratio, but none of them changes.

In perspective projection, the scene.camera.eye is updated when wheel-zooming.

At the moment it is not possible to dump to JSON a figure which has been zoomed and then to reproduce it because the information about the zoom is not included.

@archmoj archmoj added the bug something broken label Oct 11, 2019
@archmoj archmoj self-assigned this Oct 11, 2019
@archmoj
Copy link
Contributor

archmoj commented Oct 18, 2019

@emmanuelle the codepen now works with the wheel zoom: demo.

Wondering is we should apply those changes to _fullLayout.scene.aspectratio?

@alexcjohnson
Copy link
Collaborator

Yes, it’s a good idea to keep fullLayout in sync with layout, for the sake of Plotly.react

@emmanuelle
Copy link
Contributor Author

Thank you @archmoj ! At the moment is there any way to retrieve this property in Dash? Using a clientside callbck in Javascript maybe? (I'm not familiar with this, just asking)

@etpinard
Copy link
Contributor

At the moment is there any way to retrieve this property in Dash?

We should add this as part of a plotly_relayout event data object.

Here's what happens currently with the default (perspective) projection: https://codepen.io/etpinard/pen/MWWJoQM (open the console and start scrolling)

@archmoj
Copy link
Contributor

archmoj commented Oct 23, 2019

orthographic demo

@archmoj archmoj added this to the v1.51.0 milestone Oct 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something broken
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants