Latest release leads to infinite loops when calling `restyle` on `plotly_hover`

[jayfresh]

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).

[etpinard]

Thanks for the report.

I suspect #1304 and/or #1376 could have broken your example.

[etpinard]

Looks like indeed #1304 is the culprit. cc @rreusser @alexcjohnson

@jayfresh We're currently working on a fix that will be part of v1.24.1

In the meantime adding

  var gd = document.getElementById('myDiv')
  gd._hoverdata = undefined

to the 'plotly_unhover' event handler as in http://codepen.io/etpinard/pen/dvpKLw makes things work again.

[jayfresh]

👍

[jayfresh]

Ah, there's still a problem on my app using v1.24.1, but I'm not sure where that's coming from exactly. Trying to replicate on a codepen.

[dictaartisan]

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

[Linerd]

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.