-
Notifications
You must be signed in to change notification settings - Fork 9
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
Support macOS v12 #13
Conversation
view.sizingOptions = [] | ||
if #available(macOS 13.0, *) { | ||
view.sizingOptions = [] | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What are the implications of these changes? Any changes to behavior?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As far as I can tell, no. I tested on the macOS 12 VM
collectionBehavior = [.auxiliary, .stationary, .moveToActiveSpace, .fullScreenAuxiliary] | ||
if #available(macOS 13.0, *) { | ||
collectionBehavior = [.auxiliary, .stationary, .moveToActiveSpace, .fullScreenAuxiliary] | ||
} else { | ||
collectionBehavior = [.stationary, .moveToActiveSpace, .fullScreenAuxiliary] | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here, how does this affect behavior (i.e. animations) on macOS 12? Do you have a screen recording of the result?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup, here ya go
ProducerToolkitMacOS12-720p.mov
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool, that seems to work well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @andrewtheis! These changes look good to me.
I tested this on one of my apps using a macOS 12 VM (easy to do w/VirtualBuddy). Appears everything works, but obviously just one use case.