Skip to content
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

Give windows focus before sending events #171

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

DemiMarie
Copy link
Contributor

@DemiMarie DemiMarie commented Aug 22, 2022

Events from the GUI daemon are directed at specific windows. Ensuring
that they are sent to the correct window is more importent than obeying
the ICCCM input hint. Giving pointer focus to a window is accomplished
by raising it.

Fixes QubesOS/qubes-issues#3267.

Events from the GUI daemon are directed at specific windows.  Ensuring
that they are sent to the correct window is more importent than obeying
the ICCCM input hint.  Giving pointer focus to a window is accomplished
by raising it.

Fixes QubesOS/qubes-issues#3267.
@DemiMarie DemiMarie marked this pull request as ready for review August 23, 2022 01:04
@marmarek
Copy link
Member

The "ignore input hint" part sounds suspicious. Support for it was added for a reason. See this thread from 9 years ago and verify if it works in that case too. Furthermore, some toolkits (GTK?) handle input events through a dedicated off-screen window instead of a window you see, I guess calling XSetInputFocus and ignoring their preferences will break them too. Worth testing with Firefox too (which I think uses this technique too).

@DemiMarie
Copy link
Contributor Author

What about merging QubesOS/qubes-gui-daemon#104? That has been reported to help.

Some windows, such as override redirect windows, should never get input
focus.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

window focus issue
2 participants