-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Reinstate some iOS13+ guards to support iOS 12.2+ #30268
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 reinstates version guards for iOS to continue supporting iOS 12.2+ by reverting some removals from a previous change.
- Reintroduces OS version checks in the keyboard auto manager to properly compute status bar height.
- Adds fallback color values in ColorExtensions for devices running pre-iOS13.
- Reinforces iOS lifecycle event handling with additional OS version checks for scene-based APIs.
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| src/Core/src/Platform/iOS/KeyboardAutoManagerScroll.cs | Adds conditional logic to select the proper API based on iOS version |
| src/Core/src/Platform/iOS/ColorExtensions.cs | Restores version guards for color properties compatibility |
| src/Core/src/Hosting/LifecycleEvents/AppHostBuilderExtensions.iOS.cs | Reintroduces pre-iOS13 checks in lifecycle events handlers |
|
/backport to release/9.0.1xx-sr8 |
|
Started backporting to release/9.0.1xx-sr8: https://github.com/dotnet/maui/actions/runs/15933415027 |
Description of Change
Effectively reverts (part of) #26751. In that PR some checks were removed to see if the app was running iOS 13+ and make decisions on which API to use. For .NET 9 we still support iOS 12.2+ and by just using APIs that are only available on iOS 13+ we now suddenly broke some things.
This PR adds that back so we can still keep supporting iOS 12.2+ for a little while longer.
Issues Fixed
Fixes #30266