Skip to content
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

Fix camera sensor rotation/Position #35

Merged
merged 1 commit into from
Feb 3, 2016

Conversation

eduardocalazansjr
Copy link
Contributor

Problem: Some devices like LG Nexus 5x and Samsung Galaxy Tab 2 Lite the
preview image appears mirrored and upsided down.

Solution: Added setCameraDisplayOrientation to fix the camera sensor
rotation on some devices.

I've surrounded the setCameraDisplayOrientation with check for API>9.

Problem: Some devices like LG Nexus 5x and Samsung Galaxy Tab 2 Lite the
preview image appears mirrored and upsided down.

Solution: Added setCameraDisplayOrientation to fix the camera sensor
rotation on some devices.

I've surrounded the setCameraDisplayOrientation with check for API>9.
@dlazaro66
Copy link
Owner

I really appreciate this contribution! Sorry for the delay, I'm merging this right now and uploading a new version to jcenter asap, also adding you as a contributor :)

Thank you so much!

@dlazaro66 dlazaro66 closed this Feb 3, 2016
@dlazaro66 dlazaro66 reopened this Feb 3, 2016
dlazaro66 added a commit that referenced this pull request Feb 3, 2016
Fix camera sensor rotation/Position
@dlazaro66 dlazaro66 merged commit 19c4ba8 into dlazaro66:master Feb 3, 2016
@spong spong mentioned this pull request Feb 21, 2016
@yueban
Copy link

yueban commented May 11, 2016

@dlazaro66 didn't see a new version on bintray

@dlazaro66
Copy link
Owner

@yueban we're doing several changes on the library in our free time, and we'll upload a new version. Meanwhile, I recommend you to download the dependency via jitpack.io

@yueban
Copy link

yueban commented Jul 19, 2016

@dlazaro66 ok, thanks for reply

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.

3 participants