rubberduck-vba / Rubberduck

Every programmer needs a rubberduck. COM add-in for the VBA & VB6 IDE (VBE).
https://rubberduckvba.com
GNU General Public License v3.0
1.9k stars 299 forks source link

Have uniform behavior to dismiss windows with `Esc` key #5067

Open IvenBach opened 5 years ago

IvenBach commented 5 years ago

Justification It's habit for me to use the hotkey Esc to dismiss as title states. Some windows are dismissable while others aren't. Unify the UX with respect to the UI windows.

Description

Additional context Dismissable via Esc:

Non-Dismissable via Esc

Vogel612 commented 5 years ago

IIUC the windows you noted as dismissable are explicitly intended as Dialog windows. The others are mostly dockable toolwindows which should not be dismissable, (because they are not dialogs). Encapsulate Field and Search Results are probably better off as dialog windows, though.