Skip to content

fix: Track and act on the previous workspace after the active window is moved to another one#43

Merged
Antiz96 merged 4 commits intomainfrom
mv_workspace
May 10, 2026
Merged

fix: Track and act on the previous workspace after the active window is moved to another one#43
Antiz96 merged 4 commits intomainfrom
mv_workspace

Conversation

@Antiz96
Copy link
Copy Markdown
Owner

@Antiz96 Antiz96 commented May 9, 2026

Description

When a window is being moved from one workspace to another, track the "previous" workspace and also act on it if needed.
For instance, if the active window is moved to another workspace and there's only one window remaining on the previous one, also maximize this window (unless if running in "first-only" mode).

Fixed bug

Fixes #42

@Antiz96 Antiz96 added this to the v1.2.1 milestone May 9, 2026
@Antiz96
Copy link
Copy Markdown
Owner Author

Antiz96 commented May 9, 2026

Hi @1Naim 👋

Allow me a ping about this PR as it slightly modifies a past change you've made in 67132a3.
As far as I can tell, it should work as intended and should still cover the Nautilus case, but feel free to verify and tell me if anything needs to be revised.

Thanks in advance! 🤗

Antiz96 added 3 commits May 9, 2026 21:01
…is moved to another one

When a window is being moved from one workspace to another, track the "previous" workspace and also act on it if needed.
For instance, if the active window is moved to another workspace and there's only one window remaining on the previous one, also maximize this window (unless if running in "first-only" mode).

Fixes #42
Don't act on the previous workspace if running in "first-only" mode
@Antiz96 Antiz96 force-pushed the mv_workspace branch 2 times, most recently from 462dd76 to 87d7587 Compare May 9, 2026 19:12
Copy link
Copy Markdown
Contributor

@1Naim 1Naim left a comment

Choose a reason for hiding this comment

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

I haven't tested it yet, but it looks good to me.

@Antiz96
Copy link
Copy Markdown
Owner Author

Antiz96 commented May 10, 2026

I haven't tested it yet, but it looks good to me.

Thanks 🙏

@Antiz96 Antiz96 merged commit 0eb0755 into main May 10, 2026
1 check passed
@Antiz96 Antiz96 deleted the mv_workspace branch May 10, 2026 07:06
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.

Maximize window after moving another from one workspace to another

2 participants