-
Notifications
You must be signed in to change notification settings - Fork 1
Conversation
the only things now that maybe should be fixed:
|
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.
Looks good. Let's get this in so I can continue with the zoom state. @haxxmaxx what do you think?
src/tree/box.js
Outdated
.on('mousedown', () => { | ||
tooltip | ||
.html('') | ||
.attr('style', 'visibility: hidden;opacity: 0;'); | ||
}); |
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.
This does not interfere with zoom/panning or making selections?
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.
This is mostly so that the tooltip will close when panning, though they can reopen, so it isn't very effective. The panning and selections do not seem to be affected.
some basic tooltips containing all the card data
I know we have tooltips in sense, these should probably look the same as well