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

wayland: ensure bemenu is shown on top of fullscreen windows #376

Merged
merged 1 commit into from
Dec 16, 2023

Conversation

Consolatis
Copy link
Contributor

Before this patch, wayland compositors that render fullscreen windows between the top and overlay layers would show bemenu behind an active fullscreen window. For other compositors the z-rendering order was undefined.

This change ensures that bemenu will always be on top of other usual windows (including fullscreen ones).

Fixes

Before this patch, wayland compositors that render fullscreen
windows between the top and overlay layers would show bemenu
behind an active fullscreen window. For other compositors
the z-rendering order was undefined.

This change ensures that bemenu will always be on top of other
usual windows (including fullscreen ones).

Fixes Cloudef#81
@Cloudef
Copy link
Owner

Cloudef commented Dec 16, 2023

There used to be security issue related to this in swaylock but I guess it's now ok?
swaywm/swaylock#107

@Consolatis
Copy link
Contributor Author

There used to be security issue related to this in swaylock but I guess it's now ok? swaywm/swaylock#107

Indeed, the current locking protocol ext-session-lock-v1 doesn't use the overlay layer and is completely independent from the layershell protocol.

@Cloudef Cloudef merged commit d671226 into Cloudef:master Dec 16, 2023
4 checks passed
@Consolatis Consolatis deleted the fix/layershell_overlay_layer branch December 16, 2023 06:08
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.

2 participants