-
-
Notifications
You must be signed in to change notification settings - Fork 12
Fix obsolescence warnings about interface orientation on iOS #72
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
Conversation
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.
Pull request overview
This PR updates the interface orientation detection logic for iOS to address obsolescence warnings. The code has been refactored to use EffectiveGeometry.InterfaceOrientation for newer iOS versions while maintaining backward compatibility.
Key Changes:
- Refactored version checking logic to extract
windowSceneonce before determining orientation - Added support for iOS 26+ using
EffectiveGeometry.InterfaceOrientation - Restructured code to eliminate duplication in orientation assignment
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…#73) * Add null check for windowScene to prevent NullReferenceException * Use FirstOrDefault to prevent InvalidOperationException --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: janusw <[email protected]>
* by avoiding duplicate use of StatusBarOrientation
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.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
* because the test app did not always react properly to orientation changes on iOS 26 (i.e. the camera preview did not rotate properly when changing from portrait to landscape orientation) * the obsolescence warning returns, but at least things work properly this way on iOS 26
No description provided.