Fix flashing of tooltip on mouseover, mouseout

28mm / blast-radius

Interactive visualizations of Terraform dependency graphs using d3.js

https://28mm.github.io/blast-radius-docs/

MIT License

2.04k stars 257 forks source link

Fix flashing of tooltip on mouseover, mouseout #41

Closed clebio closed 5 years ago

clebio commented 5 years ago

When hovering over the edges of the polygons within a node, the tooltip is quickly hidden and shown, and the rest of the graph is cycled from opacity 1 to 0.2 and back. This creates a large-scale flicker of the graph, which is quite hard to watch.

This PR addresses the issue, by moving the tip.hide and tip.show logic to new mouseenter and mouseleave handlers.

clebio commented 5 years ago

flicker

28mm commented 5 years ago

@clebio thanks for the patch. That flicker has been super annoying! should have a chance to try it this weekend.

shelbyspees commented 5 years ago

@28mm bump