-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
prohibit WWaveformViewer getting focused #4128
Conversation
2.3 is also affected but the sidebar is auto-focused there, so minimizing the library still works. I'll rebase onto 2.3 |
Pull Request Test Coverage Report for Build 1048865715
💛 - Coveralls |
9c0ac1d
to
f48c276
Compare
so the currently focused widget stays focused, which is crucial for toggling maximize_library with <Spacebar> when any of the library widgets has focus.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can confirm this fixes / works around the bug linked. @Holzhaus do you want to take another look?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the quick fix, LGTM and works fine.
@Swiftb0y No, if you can reproduce the original bug and confirmed that this fixes it and the code looks good to you, feel free to merge without my explicit approval :D
Btw, I noticed that the search box is focused by default after starting mixxx. This means that spacebar to maximize the library doesn't work until clicking the tracks table first. No idea if this is intended or a bug, but no big issue either way.
oh, it wasn't noticed that I actually wanted to target 2.3, but this went into main. |
I can't reproduce that: no widget has focus, no matter if Mixxx was started in windowed or fullscreen mode (without controller). |
so the currently focused widget stays focused, which is crucial for toggling maximize_library
with when any of the library widgets has focus.
https://bugs.launchpad.net/mixxx/+bug/1936926
complements #2289 and #2307