Bug 1872715 Comment 0 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Some things I saw recently:
1. we're showing the popup for primitives (meaning hovering `"booh"` will show a popup with `"booh"` in it, not really useful)
1. the hovered token is not the one being highlighted (when moving the mouse quickly from a token to another)
1. the popup doesn't point to the right token (again, when moving the mouse quickly from a token to another)

1. can be avoided by checking the target class
2. seems to be caused because `Popup` only sets things on mount/unmount, but when moving quickly, the popup is updated, so we should do something in `componentDidUpdate`
3. is due to the fact the `Popover` only updates his coordinates when the popover type changes, we should rather check if the target changed

I'll try to provide a patch to make the popup more reliable
We're showing the popup for primitives (meaning hovering `"booh"` will show a popup with `"booh"` in it, not really useful)

Back to Bug 1872715 Comment 0