-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Add hint for blocking win key when using raw keyboard #13066
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
base: main
Are you sure you want to change the base?
Conversation
91a9e26
to
6f443ff
Compare
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've only looked at the code, I haven't yet tested it.
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.
From my side and from our intended usage of this hint, I've tested it works as expected. ✅ I'd just like to note that on Windows 11, there is no old-style Alt-Tab dialog and pressing Alt-Tab just switches to the previously used application (this is fine and works as expected).
The internal implementation seems a bit convoluted and hacked in, there's space to improve it and improving it isn't too difficult, just try to follow existing practices and match existing code patterns.
The current code in this PR could affect GameInput, but refactoring the code should remove unexpected interactions. I haven't tested GameInput.
c20904a
to
789aeb0
Compare
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.
The code looks much better now!
@Susko3 windows' rawinput keyboard natively support blocking the Windows key via RIDEV_NOHOTKEYS while leaving alt-tab intact. The only drawback is that the Alt-Tab menu becomes the classic style when tabbing out.