Skip to content

Conversation

@mrousavy
Copy link
Owner

What

Previously, when the phone is flat, outputOrientation would be updated to 'portrait' - which is not correct since it can also be landscape before the phone is being laid flat.

Instead, we now return nil if the orientation is flat, since it is not known how the user is holding the phone. In other words, we skip an orientation update for flat now.

Changes

Tested on

Related issues

@vercel
Copy link

vercel bot commented Feb 17, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
react-native-vision-camera ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 17, 2025 2:17pm

@mrousavy mrousavy merged commit 85a3024 into main Feb 17, 2025
8 checks passed
@mrousavy mrousavy deleted the fix/face-up-orientation branch February 17, 2025 14:31
@mrousavy
Copy link
Owner Author

Released in v4.6.4! 🥳

tomtargosz pushed a commit to tomtargosz/react-native-vision-camera that referenced this pull request Feb 21, 2025
…avy#3423)

* fix: Fix orientation updating to `portrait` when phone is flat

* fix: Make `[weak self]`
zehra-tok pushed a commit to tos-analytics/react-native-vision-camera that referenced this pull request Oct 21, 2025
…avy#3423)

* fix: Fix orientation updating to `portrait` when phone is flat

* fix: Make `[weak self]`
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