Custom menus should absorb keystrokes and dismiss if not mnemonic (letter)

[stevencl]

Testing #97801

Choose the step into targets command
See a list of targets appear
Ignore list and press F10
List of targets remains in view
Expected: list of targets to close

[isidorn]

Hmm agree that it should auto close, however on Mac we use a native context menu, and thus users can not even execute Step Over with F10, since VS Code is blocked while a context menu is shown.

It is a bit strange that when custom context menus are shown we allow commands to be executed via global keybindings.

fyi @sbatten

[sbatten]

I can try putting a listener on the same div that blocks the mouse from interacting with anything to absorb all keystrokes, though I'm not sure of the consequences. It looks like native menus dismiss the context view on f-keys, but absorb letters (in case they are mnemoncis)

[isidorn]

@sbatten I think we should behave like native menus as much as possible.
Though I would suggest to only do this change on a start of milestone so we have enuogh time to get feedback.

[isidorn]

I do not plan to tackle this on the debug side, thus forwarding to @sbatten so he can decide if he wants to tackle it on the menu side