Skip to content
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

Hoverdata undefined when animating. #1274

Closed
rreusser opened this issue Dec 28, 2016 · 6 comments
Closed

Hoverdata undefined when animating. #1274

rreusser opened this issue Dec 28, 2016 · 6 comments
Assignees
Labels
bug something broken

Comments

@rreusser
Copy link
Contributor

rreusser commented Dec 28, 2016

Investigating the cause of failure for plotly_click to work sometimes when animations are taking place. Tracing through things:

  1. Fx.click fires
  2. gd._hoverdata is undefined sometimes

Investigating the further causes…

cc @bpostlethwaite

@rreusser rreusser added the bug something broken label Dec 28, 2016
@rreusser rreusser self-assigned this Dec 28, 2016
@rreusser rreusser changed the title Hover behavior when animating. Hoverdata undefined when animating. Dec 28, 2016
@rreusser
Copy link
Contributor Author

rreusser commented Dec 28, 2016

I'm having a lot of trouble coming up with a sequence of events that reproduces this 100% of the time, but the problem definitely seems to be the way hoverdata is set. It seems like maybe it's set on mousemove and the used on click. Need to investigate further. I suspect the cause is that I'm simply failing to set it 100% of the time when animation changes the hoverdata.

cc @etpinard I might need some help to get to the bottom of this (tomorrow)

@etpinard
Copy link
Contributor

etpinard commented Jan 4, 2017

cc @etpinard I might need some help to get to the bottom of this (tomorrow)

I hope 8 days later is ok too 😄

@rreusser
Copy link
Contributor Author

rreusser commented Jan 4, 2017

Haha, yup!

@rreusser
Copy link
Contributor Author

Update. Have isolated a better example. Working on debugging.

@etpinard
Copy link
Contributor

@rreusser does #1304 indeed close this issue?

@rreusser
Copy link
Contributor Author

I'll try to do a better job of catching related issues!

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

No branches or pull requests

2 participants