Skip to content

Alignment viewer: Issues with sync between subplots #159

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
2 tasks
jackparmer opened this issue Feb 7, 2019 · 8 comments
Closed
2 tasks

Alignment viewer: Issues with sync between subplots #159

jackparmer opened this issue Feb 7, 2019 · 8 comments
Labels
App QA UX/UI issues with gallery apps bug Something isn't working

Comments

@jackparmer
Copy link
Contributor

  • If you zoom into the center heatmap, the viewport of the bottom global view doesn't always update.
  • When you are very zoomed into the main heatmap, if you click-drag on the bottom global view, only the x-axis labels of the center heatmap update.

image

I thought this might be a plotly.js issue, but I don't get this off synch behavior in Chart Studio: https://plot.ly/~jackp/18501/#/

So I wonder if this app is using an older version of plotly.js.

The JS console is full of these bundle.js errors from click-dragging. I see calcdata in the compressed source, so it does seem plotly.js is having trouble in this particular environment.

image

@jackparmer jackparmer added bug Something isn't working App QA UX/UI issues with gallery apps labels Feb 7, 2019
@jackluo
Copy link
Contributor

jackluo commented Feb 8, 2019

Difference between here and on Chart Studio is that drag is manually handled on MSA
Seems like its the same bug I previously mentionned: plotly/plotly.js#3305

I'll make a local copy of the React code inside the Dash component instead of just wrapping the imported react-alignment-viewer for now if we can't get a quick solution before release, thoughts

@etpinard
Copy link

etpinard commented Feb 8, 2019

@jackluo I'm curious. What makes you so sure that plotly/plotly.js#3305 is at the root of the problem?

@jackluo
Copy link
Contributor

jackluo commented Feb 8, 2019

@etpinard zoom causes trouble when its the drag zoom tool, but + - (zoom buttons) works fine
Seems like onRelayout events don't always get triggered either, can be confirmed by console logging

@etpinard
Copy link

etpinard commented Feb 8, 2019

Ok, and what is the app suppose to do on drag?

@jackluo
Copy link
Contributor

jackluo commented Feb 8, 2019

Both the heatmap and webgl text overlay should drag/zoom together

@etpinard
Copy link

etpinard commented Feb 8, 2019

OK now, why do need an on-relayout handler?

https://plot.ly/~jackp/18501/#/ has a shared x-axis, and seems to zoom/pan properly.

Moreover, are you using subplots are multiple graph divs in your app? If you're using multiple graph divs, then you should not encounter plotly/plotly.js#3305

@jackparmer
Copy link
Contributor Author

jackparmer commented Feb 8, 2019 via email

@etpinard
Copy link

etpinard commented Apr 1, 2019

Is there an easy way to try a new plotly.js bundle in a dash app?

If so, could something try (or tell me how to do it) this bundle -> https://31572-45646037-gh.circle-artifacts.com/0/dist/plotly.min.js ? I think it might solve the syncing problems presented here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
App QA UX/UI issues with gallery apps bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants