-
Notifications
You must be signed in to change notification settings - Fork 14
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
Tooltip #282
Tooltip #282
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cool! looks like this does it. one comment and possible change, so i'll wait to hear back from you before merging even though i'm approving now
|
||
// remove all children from any tooltip group: | ||
d3.selectAll("#tooltip-group").selectAll("*").remove(); | ||
// does the above work to clear out multiple tooltips from multiple svgs? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think it does - i commented out line 7 in developer tools, created a bunch of tooltips on the 2 svgs, and then called this line before and after calling d3.selectAll("#tooltip-group").selectAll("*")
. all of the tooltips in both groups disappeared after the remove()
call. are you confident now, too, and if so how about removing this line?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, confident - how about I remove it in #284 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Fixes #273
Now will PR on
example
viz to add this functionality