Skip to content

Conversation

@janusw
Copy link
Owner

@janusw janusw commented Dec 29, 2025

No description provided.

Copy link

Copilot AI left a 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 windowScene once 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.

Copy link

Copilot AI commented Dec 29, 2025

@janusw I've opened a new pull request, #73, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot AI and others added 2 commits December 30, 2025 09:58
…#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
Copy link

Copilot AI left a 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.

@janusw janusw merged commit 6c5303e into master Dec 30, 2025
12 checks passed
@janusw janusw deleted the fix-obsolete-interface-orientation branch December 30, 2025 09:42
janusw added a commit that referenced this pull request Dec 30, 2025
* 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
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.

2 participants