Popup menu and pickers doesn't capture the input used to close it

[abey79]

One can close popup menus and, e.g., the colour picker by clicking outside of it or hitting escape. Unfortunately, these inputs are not captured and thus potentially have an unwanted secondary effect.

In the case of rerun, escape will have the effect of deselecting whatever is currently selected, and the click will be processed by whatever happens to be under the mouse cursor (often, the view, which again leads to deselection).

color_picker.mp4

[emilk]

This is by design (though arguably not good design). The thinking (within egui) is: pressing escape should escape everything. Otherwise what it escapes would depend on the order of which the UI is laid out, which is not obvious to a user ("why did A close instead of B?").

[abey79]

What is the "correct" way to opt-out of a popup menu then?

Otherwise what it escapes would depend on the order of which the UI is laid out, which is not obvious to a user ("why did A close instead of B?").

Perhaps naively, I'm expecting popup-menu/picker to be modal, so (1) they may not have sibling areas—only parent, over which they have precedence and (2) they may have zero or one "child" modal (e.g. a sub-menu), which would have precedence. In this model, it feels quite natural in how keystroke would be captured. Surely I'm missing dozen of edge cases and assorted implementation trickeries here, though.

pressing escape should escape everything.

My mental model doesn't conform to that. If have a stack of "modal things" open, I'm expecting to get out of it by hitting escape as many times as the number of modal things that are opened.

---

Related:

• Add click, drag and release in menus #2549 (in this case, the opt-out behaviour is natural: just release the click outside of the menu).
• Color picker closes on click when inside a context menu #3718

[emilk]

Switching all i.key_pressed(Key::Escape) to use consume_key instead might be the right solution, we just need to be careful to do it in the right place so that:

• Pressing escape closes all Menu:s (not just the innermost submenu)
• Pressing escape closes the innermost popup (if nested)
• …?