-
Notifications
You must be signed in to change notification settings - Fork 431
Improved menu keyboard navigation #4480
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…ic for that into FormBase.
|
I don't understand what these mean:
Also did you mean 2.11 instead of 2.10? |
Yes. |
|
While comparing the first case to other apps I found that as you switch the menus using arrows, before the first menu and after the last one there's a hidden menu that represents that Alt+Space thing, and if you hit Enter while that hidden menu is selected, Alt+Space thing will show up. And yeah simply pressing Space while some other menu is selected, shows Alt+Space thing too. That functionality available through Enter is still not present in this PR, because you can't select that hidden menu, tho I dunno if that matters.
|
|
I don't like that this focuses the top menu now when I alt-tab, this was not the case in any previous version and a good thing imo. I also don't see this behavior in any other app. |
|
I missed that. Apparently it's not an issue if you release alt before releasing tab, which is my habit. |
Several bug fixes.
New behavior existing in
masterand this PR but not in 2.11, arguably a bug:Bugs that exist in 2.11 and
master, fixed in this PR:master)Regression from 2.11 in
master, fixed in this PR:Bug existing in all three:
Check if completed: