Skip to content

feat(spaces): add force arg for moveWindowToSpace - #3298

Merged
cmsj merged 1 commit into
Hammerspoon:masterfrom
okuuva:force-move-window-to-space
Nov 10, 2022
Merged

cmsj merged 1 commit into
Hammerspoon:masterfrom
okuuva:force-move-window-to-space

Conversation

@okuuva

@okuuva okuuva commented Oct 13, 2022

Copy link
Copy Markdown
Contributor

Floating windows can be drawn over full screen windows so there should be a way to move them into full screen spaces as well.

Tested that this works with a floating window (kitty to be precise). Also tested moving a fullscreen window to a fullscreen (and normal) space and confirmed that it does not work. It does try to do it and return true but the source space just goes black and the window doesn't appear at the destination space (which is expected but wanted to check what happens). My extremely limited (read: none) knowledge in Obj C can get only so far, if we want to prevent even trying to move fullscreen windows even with force = true I need some help with that. I think it would be better to check if the window is in fullscreen rather than checking the source space type but don't know how to do that from within spaces extension or if it's even possible.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This pull request does not contain a valid label. Please add one of the following labels: pr-fix, pr-change, pr-feature, pr-maintenance

@okuuva
okuuva force-pushed the force-move-window-to-space branch from 69fcddc to 110aa0e Compare October 13, 2022 16:08
Comment thread extensions/spaces/libspaces.m Outdated
Comment thread extensions/spaces/libspaces.m Outdated
@github-actions

github-actions Bot commented Nov 2, 2022 •

Copy link
Copy Markdown

View Test Results

348 tests   295 ✔️  1h 2m 49s ⏱️
    2 suites    52 💤
    1 files        1 ❌

For more details on these failures, see this check.

Results for commit f71e586.

♻️ This comment has been updated with latest results.

@cmsj cmsj added the pr-feature Pull Request implementing a feature label Nov 3, 2022
Floating windows can be drawn over full screen windows so there should
be a way to move them into full screen spaces as well.
@okuuva
okuuva force-pushed the force-move-window-to-space branch from 110aa0e to f71e586 Compare November 4, 2022 07:57
@cmsj
cmsj merged commit 281f4cb into Hammerspoon:master Nov 10, 2022
@cmsj

cmsj commented Nov 10, 2022

Copy link
Copy Markdown
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-feature Pull Request implementing a feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants