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

Fix handling of layer shell surface with exclusive interactivity #212

Merged
merged 1 commit into from
Nov 2, 2023

Conversation

ids1024
Copy link
Member

@ids1024 ids1024 commented Nov 2, 2023

This should now match the protocol specification, and how Sway behaves.

If a surface in the Top or Overlay has KeyboardInteractivity::Exclusive, only that surface can have keyboard focus. If multiple surfaces do, focus can be changed between those with keyboard exclusivity on the highest layer.

This is similar to session lock surfaces, except a session lock takes precedence.

This fixes the behavior of keyboard focus in cosmic-greeter, with the current layer-shell based implementation.

@ids1024 ids1024 requested a review from a team November 2, 2023 19:13
This should now match the protocol specification, and how Sway behaves.

If a surface in the `Top` or `Overlay` has
`KeyboardInteractivity::Exclusive`, only that surface can have keyboard
focus. If multiple surfaces do, focus can be changed between those with
keyboard exclusivity on the highest layer.

This is similar to session lock surfaces, except a session lock takes
precedence.

This fixes the behavior of keyboard focus in `cosmic-greeter`, with the
current layer-shell based implementation.
@ids1024 ids1024 force-pushed the exclusive-keyboard_jammy branch from 053c7a8 to 3bec008 Compare November 2, 2023 19:49
@jackpot51 jackpot51 merged commit f3a8ff8 into master_jammy Nov 2, 2023
1 check passed
@jackpot51 jackpot51 deleted the exclusive-keyboard_jammy branch November 2, 2023 22:44
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.

3 participants