Skip to content

x11: Set window title prior to mapping window#362

Merged
august64 merged 1 commit intorust-windowing:masterfrom
august64:x11-earlier-title
Dec 13, 2017
Merged

x11: Set window title prior to mapping window#362
august64 merged 1 commit intorust-windowing:masterfrom
august64:x11-earlier-title

Conversation

@august64
Copy link
Copy Markdown
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.
@august64 august64 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