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

x11: Set window title prior to mapping window #362

Merged
merged 1 commit into from
Dec 13, 2017

Conversation

francesca64
Copy link
Member

Fixes #282

Some tiling window managers (i3, dwm, etc.) determine how a window should behave based on
its name. If the name is set after mapping, then window managers will check the name before
we set it, followed by them detecting it as a change when the name is actually set. That
results in the window briefly behaving in an unexpected way, followed by a rapid switch to
the expected behavior.

In accordance to section 4.1.2 of ICCCM, the name, decorations, size hints, and window
deletion redirection have all been moved up to be set before mapping.

Fixes rust-windowing#282

Some tiling window managers (i3, dwm, etc.) determine how a window should behave based on
its name. If the name is set after mapping, then window managers will check the name before
we set it, followed by them detecting it as a change when the name is actually set. That
results in the window briefly behaving in an unexpected way, followed by a rapid switch to
the expected behavior.

In accordance to section 4.1.2 of ICCCM, the name, decorations, size hints, and window
deletion redirection have all been moved up to be set before mapping.
@francesca64 francesca64 merged commit 9698d0a into rust-windowing:master Dec 13, 2017
tmfink pushed a commit to tmfink/winit that referenced this pull request Jan 5, 2022
`usvg` should never have nested `<svg>` nodes, so remove code that (doesn't) handle that case.

Closes rust-windowing#349.
madsmtm pushed a commit to madsmtm/winit that referenced this pull request Jun 11, 2022
…dowing#362)

* refactor(tray): split gtk and ayatana appindicator features

* fix ci

* install dev package

* fix: use v3

* chore: short feature flag name
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants