Skip to content

Fix “unrecognized selector sent to instance” crash in transition on iOS < 17#792

Merged
kean merged 1 commit intokean:mainfrom
ejensen:transition-dynamic-range-fix
Jun 18, 2024
Merged

Fix “unrecognized selector sent to instance” crash in transition on iOS < 17#792
kean merged 1 commit intokean:mainfrom
ejensen:transition-dynamic-range-fix

Conversation

@ejensen
Copy link
Copy Markdown
Contributor

@ejensen ejensen commented Jun 17, 2024

Prevents an “unrecognized selector sent to instance” crash caused by UIKit incorrectly marking UIImageIVew.preferredImageDynamicRange available since iOS 2.0. Its actual availability is iOS 17.0+

The reference to preferredImageDynamicRange was added in #787 and released in Nuke v12.7.1

Incorrect documentation

…Kit incorrectly marking preferredImageDynamicRange available on UIImageView since iOS 2.0
@ejensen ejensen closed this Jun 17, 2024
@kean
Copy link
Copy Markdown
Owner

kean commented Jun 18, 2024

Hey, @ejensen. Thanks for opening this PR. Yeah, it looks like a mistake in the label. The type that it returns, UIImage.DynamicRange is labeled as iOS 17+. Let's get this merged, or is there some issue?

@ejensen ejensen reopened this Jun 18, 2024
@ejensen
Copy link
Copy Markdown
Contributor Author

ejensen commented Jun 18, 2024

@kean Sorry for closing the PR. GitHub automatically closed it when I merged a linked PR in a private repository. It is now reopened.

@kean
Copy link
Copy Markdown
Owner

kean commented Jun 18, 2024

** TEST BUILD FAILED ** on Xcode 14.3.1. I'm going to take care of it in the main branch. I think it time to drop its support because Xcode 15 has been a requirement for shipping to App Store since April.

@kean kean merged commit 8b93a87 into kean:main Jun 18, 2024
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