Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Space#moveWindows on Sonoma 14.5. #350

Merged
merged 7 commits into from
Jun 9, 2024

Conversation

metakirby5
Copy link
Contributor

  • Updated related documentations
  • Added the change to the Changelog

Introduce a workaround to CGSMoveWindowsToManagedSpace breaking in MacOS 14.5, based on:

Fixes #348 and #349.

Copy link
Owner

@kasper kasper left a comment

Choose a reason for hiding this comment

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

@metakirby5 Thank you for the help! ❤️ I added some comments, could you address them? Thanks!

Phoenix/NSProcessInfo+PHExtension.h Outdated Show resolved Hide resolved
Phoenix/PHSpace.m Outdated Show resolved Hide resolved
Phoenix/PHSpace.m Outdated Show resolved Hide resolved
Phoenix/PHSpace.m Outdated Show resolved Hide resolved
Phoenix/PHSpace.m Outdated Show resolved Hide resolved
Phoenix/PHSpace.m Show resolved Hide resolved
Phoenix/PHSpace.m Outdated Show resolved Hide resolved
Phoenix/PHSpace.m Outdated Show resolved Hide resolved
Phoenix/PHSpace.m Outdated Show resolved Hide resolved
Phoenix/PHSpace.m Outdated Show resolved Hide resolved
@metakirby5 metakirby5 requested a review from kasper June 7, 2024 17:36
Phoenix/PHSpace.m Outdated Show resolved Hide resolved
@metakirby5 metakirby5 requested a review from kasper June 8, 2024 08:18
Copy link
Owner

@kasper kasper left a comment

Choose a reason for hiding this comment

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

Alright, last set of comments, sorry! Looks good to me after that. ❤️

Phoenix/PHSpace.m Outdated Show resolved Hide resolved
Phoenix/PHSpace.m Outdated Show resolved Hide resolved
Phoenix/PHSpace.m Outdated Show resolved Hide resolved
Phoenix/PHSpace.m Outdated Show resolved Hide resolved
Phoenix/PHSpace.m Outdated Show resolved Hide resolved
Phoenix/PHSpace.m Show resolved Hide resolved
Phoenix/PHSpace.m Outdated Show resolved Hide resolved
@metakirby5 metakirby5 requested a review from kasper June 8, 2024 10:05
kasper
kasper previously approved these changes Jun 8, 2024
Copy link
Owner

@kasper kasper left a comment

Choose a reason for hiding this comment

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

Thanks!

Phoenix/PHSpace.m Outdated Show resolved Hide resolved
Phoenix/PHSpace.m Outdated Show resolved Hide resolved
@metakirby5
Copy link
Contributor Author

@kasper Thanks for approving! I don't have permission to merge, so can you do so for me? 🙏

@kasper kasper merged commit 6955592 into kasper:master Jun 9, 2024
1 check passed
@kasper
Copy link
Owner

kasper commented Jun 9, 2024

@metakirby5 Thanks, merged! I will try to get this released soon. I want to update all the dependencies while at it.

@kasper kasper self-assigned this Jun 9, 2024
@kasper kasper added the bug label Jun 9, 2024
@kasper kasper added this to the 4.0.1 milestone Jun 9, 2024
@kasper kasper assigned metakirby5 and unassigned kasper Jun 9, 2024
@metakirby5 metakirby5 deleted the fix-sonoma-move-windows branch June 10, 2024 01:53
kasper added a commit that referenced this pull request Jun 10, 2024
@dan-phantom
Copy link

Just x-posing here, maybe it gets visible? 😅

#348 (comment)

@kasper
Copy link
Owner

kasper commented Jul 7, 2024

@dan-phantom Hey, thanks for the ping! Let’s continue in #351.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

space(i).moveWindows([x]) seems to no longer work under Sonoma 14.5
3 participants