-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Latest release leads to infinite loops when calling restyle
on plotly_hover
#1446
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
restyle
on plotly_hover
Looks like indeed #1304 is the culprit. cc @rreusser @alexcjohnson @jayfresh We're currently working on a fix that will be part of In the meantime adding var gd = document.getElementById('myDiv')
gd._hoverdata = undefined to the |
👍 |
Ah, there's still a problem on my app using |
I see what is probably the same problem (stack overflow, chrome hanging) when drawing/removing an annotation on hover and unhover events Here's a codepen (shamelessly copying etpinard's code) http://codepen.io/dictaartisan/pen/XRRYwe Hover in and out of the points often enough and browser will hang - seems to recover after a while in this case but on another more complex plot I have to kill the browser |
Same here, on the latest version. Trying to restyle a scatter 3D plot upon hovering and infinite amount of hover events are triggered until maximum stack call is reached. |
Since the latest release, using the hover and unhover events seems to throw a lot of JavaScript errors, ultimately locking up the browser. The easiest way to see this is on the hover event example on the Plotly docs: https://plot.ly/javascript/plotlyjs-events/#hover-event with accompanying codepen here: http://codepen.io/plotly/pen/xgxbrj.
I've tested that this doesn't happen locally for v1.23.1 (and there aren't any "hover" changes in v1.23.2 so I expect that would work, but https://cdn.plot.ly/plotly-1.23.2.min.js is not available).
The text was updated successfully, but these errors were encountered: