-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
fix(device_info_plus): update iOS/macOS deployment targets to 15.0 and fix isiOSAppOnVision detection #3812
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
Changes from all commits
18c31a6
93d94b0
e5a3292
d880c31
86207ac
d20ad6d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| platform :ios, '15.0' |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| platform :osx, '15.0' |
| Original file line number | Diff line number | Diff line change | ||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -31,8 +31,16 @@ - (void)handleMethodCall:(FlutterMethodCall *)call | |||||||||||||||||
| isiOSAppOnMac = [NSNumber numberWithBool:[info isiOSAppOnMac]]; | ||||||||||||||||||
| } | ||||||||||||||||||
| NSNumber *isiOSAppOnVision = [NSNumber numberWithBool:NO]; | ||||||||||||||||||
| if (@available(iOS 26.1, *)) { | ||||||||||||||||||
| isiOSAppOnVision = [NSNumber numberWithBool:[info isiOSAppOnVision]]; | ||||||||||||||||||
| if (@available(iOS 15.0, *)) { | ||||||||||||||||||
| if (@available(iOS 26.1, *)) { | ||||||||||||||||||
| SEL isiOSAppOnVisionSelector = NSSelectorFromString(@"isiOSAppOnVision"); | ||||||||||||||||||
| if ([info respondsToSelector:isiOSAppOnVisionSelector]) { | ||||||||||||||||||
| BOOL (*isiOSAppOnVisionIMP)(id, SEL) = | ||||||||||||||||||
| (BOOL (*)(id, SEL))[info methodForSelector:isiOSAppOnVisionSelector]; | ||||||||||||||||||
| isiOSAppOnVision = | ||||||||||||||||||
| [NSNumber numberWithBool:isiOSAppOnVisionIMP(info, isiOSAppOnVisionSelector)]; | ||||||||||||||||||
| } | ||||||||||||||||||
|
Comment on lines
+34
to
+42
|
||||||||||||||||||
| if (@available(iOS 15.0, *)) { | |
| if (@available(iOS 26.1, *)) { | |
| if ([info respondsToSelector:@selector(isiOSAppOnVision)]) { | |
| isiOSAppOnVision = [NSNumber numberWithBool:[info isiOSAppOnVision]]; | |
| } | |
| if (@available(iOS 26.1, *)) { | |
| if ([info respondsToSelector:@selector(isiOSAppOnVision)]) { | |
| isiOSAppOnVision = [NSNumber numberWithBool:[info isiOSAppOnVision]]; |
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.
Raising the minimum iOS/macOS deployment targets is a breaking change for plugin consumers, but the PR title doesn’t include the Conventional Commits breaking-change marker (
!) and the PR description’s breaking-change checklist is contradictory (both Yes and No checked). Please align the PR metadata (title/description) with the actual breaking nature of this change.