-
-
Notifications
You must be signed in to change notification settings - Fork 50
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
Pass all focus events to clients #104
base: master
Are you sure you want to change the base?
Conversation
9ac2d26
to
96d4767
Compare
OpenQA test summaryComplete test suite and dependencies: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.2&build=2023052602-4.2&flavor=pull-requests New failures, excluding unstableCompared to: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.2&build=2023021823-4.2&flavor=update
Failed tests55 failures
Fixed failuresCompared to: https://openqa.qubes-os.org/tests/60652#dependencies 9 fixed
Unstable tests
|
Ignoring some focus events can result in the client and server disagreeing on which window is focused or which keys have been pressed. The results are unpredictable and generally undesirable. Furthermore, it turns out that clients do care about the mode. Therefore, pass all focus events to clients, without clobbering the mode value. Fixes QubesOS/qubes-issues#7599.
96d4767
to
f01a742
Compare
PipelineRetryFailed |
PipelineRetryFailed |
Ignoring some focus events can result in the client and server
disagreeing on which window is focused or which keys have been pressed.
The results are unpredictable and generally undesirable.
Therefore, pass all focus events to clients. The mode is still
overridden with NotifyNormal to avoid leaking window manager
implementation details.
Fixes QubesOS/qubes-issues#7599.