Skip to content

Conversation

@jfversluis
Copy link
Member

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

@jfversluis jfversluis added this to the .NET 9 SR9 milestone Jun 27, 2025
Copilot AI review requested due to automatic review settings June 27, 2025 09:25
@jfversluis jfversluis requested a review from a team as a code owner June 27, 2025 09:25
@jfversluis jfversluis requested review from PureWeen and rmarinho June 27, 2025 09:25
@jfversluis jfversluis added platform/ios p/0 Current heighest priority issues that we are targeting for a release. labels Jun 27, 2025
@jfversluis jfversluis added area-controls-general General issues that span multiple controls, or common base classes such as View or Element i/regression This issue described a confirmed regression on a currently supported version labels Jun 27, 2025
@jfversluis jfversluis moved this from Todo to Ready To Review in MAUI SDK Ongoing Jun 27, 2025
Copy link
Contributor

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 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

@github-project-automation github-project-automation bot moved this from Ready To Review to Approved in MAUI SDK Ongoing Jun 27, 2025
@PureWeen PureWeen merged commit fc85d76 into main Jun 27, 2025
127 of 129 checks passed
@PureWeen PureWeen deleted the fix-30266 branch June 27, 2025 18:33
@github-project-automation github-project-automation bot moved this from Approved to Done in MAUI SDK Ongoing Jun 27, 2025
@PureWeen
Copy link
Member

/backport to release/9.0.1xx-sr8

@github-actions
Copy link
Contributor

Started backporting to release/9.0.1xx-sr8: https://github.com/dotnet/maui/actions/runs/15933415027

@github-actions github-actions bot locked and limited conversation to collaborators Jul 28, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-controls-general General issues that span multiple controls, or common base classes such as View or Element i/regression This issue described a confirmed regression on a currently supported version p/0 Current heighest priority issues that we are targeting for a release. platform/ios

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Objective-C exception thrown. Name: NSInvalidArgumentException Reason: +[UIColor labelColor]

3 participants