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

set _NET_WM_WINDOW_TYPE on monitor root window #1453

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gandalf3
Copy link

@gandalf3 gandalf3 commented May 6, 2023

set window type for monitor roots to _NET_WM_WINDOW_TYPE_DESKTOP to prevent them from being considered an application window by, for example, screen capture software.

prior to this change firefox would offer monitor root windows in its window capture chooser interface, and if chosen, would raise them which would cause input events to be misdirected.

set window type for monitor roots to _NET_WM_WINDOW_TYPE_DESKTOP to
prevent them from being considered an application window by, for
example, screen capture software.

prior to this change firefox would offer monitor root windows in its
window capture chooser interface, and if chosen, would raise them which
would cause input events to be misdirected.
@gandalf3
Copy link
Author

gandalf3 commented May 6, 2023

perhaps _NET_WM_WINDOW_TYPE_DESKTOP isn't exactly a perfect fit for this, but this was the simplest/cleanest change i found that prevents firefox from offering these windows. a possible alternative might be not setting WM_STATE, but this would require more extensive changes.

for ease of reference, firefox's window listing code can be viewed here.

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.

1 participant