Skip to content

window-rules: preserve explicit initial position - #3151

Merged
ammen99 merged 1 commit into
WayfireWM:masterfrom
mrsteve0924:window-rules-initial-position
Oct 3, 2026
Merged

ammen99 merged 1 commit into
WayfireWM:masterfrom
mrsteve0924:window-rules-initial-position

Conversation

@mrsteve0924

Copy link
Copy Markdown
Contributor

When applying a window rule to a view that generates multiple pre-map transactions, such as Microsoft Edge, the place plugin can run again after the rule has already set the initial position. Since on created rules are only applied once, a subsequent placement pass can override move or set geometry, causing the view to be positioned by place instead of at the coordinates specified by the rule.

Mark views moved by window-rules before mapping with the existing startup-x and startup-y properties. The place plugin already treats these properties as an indication that the initial position has been explicitly specified and skips placement.

This preserves explicit move and set geometry actions across subsequent pre-map transactions without changing the behavior of views that have not been explicitly positioned.

When applying a window rule to a view that generates multiple pre-map transactions, such as Microsoft Edge, the place plugin can override the rule during startup. This affects set geometry and move actions, causing the window to be placed by the place plugin instead of at the position specified by the window rule.

Signed-off-by: stephen finkel <mrsteve0924@verizon.net>
Assisted-by: ChatGPT:GPT-5.6 Sol

@ammen99 ammen99 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense, thanks!

@ammen99
ammen99 merged commit 6fdde77 into WayfireWM:master Oct 3, 2026
3 of 4 checks passed
@mrsteve0924
mrsteve0924 deleted the window-rules-initial-position branch October 9, 2026 04:28
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