Skip to content

fix(event): Differentiate WindowOpened and WindowChanged#35

Merged
Antiz96 merged 2 commits intoAntiz96:mainfrom
1Naim:fix/window-change
Apr 21, 2026
Merged

fix(event): Differentiate WindowOpened and WindowChanged#35
Antiz96 merged 2 commits intoAntiz96:mainfrom
1Naim:fix/window-change

Conversation

@1Naim
Copy link
Copy Markdown
Contributor

@1Naim 1Naim commented Apr 21, 2026

Description

So, after waking up I thought of this more elegant solution for the Nautilus hack, and in turn this fixes a bug that oniri has as well, namely: If a single window is not maximized, then a state change (window title changing) can maximize that window.

Screenshots / Logs

Paste any relevant logs here (if you have some)

Fixed bug

Fixes "issue_URL" (if any)

Addressed feature request

Closes "issue_URL" (if any)

1Naim added 2 commits April 21, 2026 14:19
…96#34)"

This reverts commit 8526532.

Signed-off-by: Eric Naim <dnaim@cachyos.org>
We can differentiate if a window is being opened or a window's state has
changed by checking if the window is already in the same workspace. If
it is, we know the window state just changed, and we can skip processing
the whole event.

Signed-off-by: Eric Naim <dnaim@cachyos.org>
@Antiz96 Antiz96 added this to the v1.1.0 milestone Apr 21, 2026
Copy link
Copy Markdown
Owner

@Antiz96 Antiz96 left a comment

Choose a reason for hiding this comment

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

Thanks!

@Antiz96 Antiz96 merged commit 67132a3 into Antiz96:main Apr 21, 2026
1 check passed
@1Naim 1Naim deleted the fix/window-change branch April 21, 2026 16:19
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