Superset edit filter popover disappearing on hover

[zuzana-vej]

The filter UI is more sensitive than before, leading to “catch me if you can” type of experience. When adding a filter to a chart (so in Explore view, you click on or trying to edit it, the popover containing filter settings disappears when moving the cursor from the filter to it (check out attached video). This doesn't happen all the time, but when it does, it's very frustrating. Sometimes, the popover displays, but is closed (disappears) before user clicks "save" or "cancel".

Browser: Chrome

Steps to Reproduce:

1. Create or open superset chart.
2. Click filter.
3. Hover to edit filter popover.

Result: Popover disappears.

Expected Result: Popover stays in place so I can edit the filter. Unfortunately not able to repro 100%. Happens sometimes... so planing to keep this bug open and keep checking if users encounter it again.

Popover should only close when user clicks on save or cancel, not when user moves mouse around. This piece of the issue is reproduceable more frequently.

Screenshots

Step one: Click on a metric, filter box displays

Step two: Move mouse into the filter box - it disappears

Environment

(please complete the following information):

• superset version: master

Checklist

Make sure these boxes are checked before submitting your issue - thank you!

• I have checked the superset logs for python stacktraces and included it here as text if there are any.
• I have reproduced the issue with at least the latest released version of superset.
• I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context

Add any other context about the problem here.

[issue-label-bot]

Issue-Label Bot is automatically applying the label #bug to this issue, with a confidence of 0.93. Please mark this comment with 👍 or 👎 to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

[zuzana-vej]

Related to #11388

[junlincc]

💔💔💔!

@eschutho could you please take a look at #11394, #11388 and #11379, i think they are all related. 🙏

[eschutho]

sure, I'll take a look. Do you want to assign this to me?

[junlincc]

sure, I'll take a look. Do you want to assign this to me?

let me know how long it's gonna take you. Kamil and Kasia can help 🙏

[eschutho]

I have a quick fix, although I think the use of state vs props in the component could use some improvements. If you think it's worth pushing out a fix with the existing code, I can put up a PR now and maybe Kamil and Kasia can help with the longer-term improvements.

[junlincc]

#11163 @kgabryje Kamil~

[zuzana-vej]

Here is another video recording when it disappears after user starts typing, if they move their mouse outside the popover area.