-
Notifications
You must be signed in to change notification settings - Fork 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
Don't report associated text only for C0/C1 #7658
Conversation
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.
What motivated you to change this considering the following statement:
There's no wording saying that I can't do so and I don't see why I shouldn't. libxkbcommon explicitly tells me that I have text, thus, I send text.
|
19a314b
to
616458c
Compare
Don't squash this branch when merging. |
This has a side effect that we'll have text reported for Alt+Shift+T and similar, but only C0/C1 should be excluded and Alt+Shift+T is emitting neither, thus regular `T` will be reported. Fixes alacritty#7657.
6194f6f
to
c3a8eb1
Compare
This has a side effect that we'll have text reported for Alt+Shift+T and similar, but only C0/C1 should be excluded and Alt+Shift+T is emitting neither, thus regular
T
will be reported.Fixes #7657.
--
@rockorager