Skip to content

fix(ios): Crash when taking a photo on iOS 15#3827

Closed
Developerius wants to merge 1 commit intomrousavy:mainfrom
Developerius:main
Closed

fix(ios): Crash when taking a photo on iOS 15#3827
Developerius wants to merge 1 commit intomrousavy:mainfrom
Developerius:main

Conversation

@Developerius
Copy link
Copy Markdown

This PR fixes a crash that occurs on iOS 15 when attempting to capture a photo.

For iOS versions earlier than 16, AVCapturePhotoSettings.isHighResolutionPhotoEnabled is explicitly set. However, this flag requires isHighResolutionCaptureEnabled = true to be configured on the corresponding AVCapturePhotoOutput. Without this, the app crashes at runtime.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 30, 2026

@Developerius is attempting to deploy a commit to the Margelo Team on Vercel.

A member of the Team first needs to authorize it.

@mrousavy
Copy link
Copy Markdown
Owner

mrousavy commented May 4, 2026

Thanks for your PR, but should this property really always be true? Shouldn't we either leave it false by default, or only set it to true if a certain targetSize is configured? e.g. if its >=4k, or if quality prioritization is quality then we can enable this flag?

Honestly after writing this I feel like it might be best to just leave it untouched by default since iOS <15 is such a small market share nowadays and we can just drop this for a smaller test matrix...

@mrousavy
Copy link
Copy Markdown
Owner

mrousavy commented May 5, 2026

I merged #3836 now

@mrousavy mrousavy closed this May 5, 2026
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