[
https://issues.apache.org/jira/browse/CB-7276?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14988051#comment-14988051
]
Richard B Knoll commented on CB-7276:
-------------------------------------
I guess [we do already re-compress the
file|https://github.com/apache/cordova-plugin-camera/blob/master/src/android/CameraLauncher.java#L498]
if we modify it (by setting width/height, quality etc.) so maybe that is the
way to go
> correctOrientation is applied only for JPEG images
> --------------------------------------------------
>
> Key: CB-7276
> URL: https://issues.apache.org/jira/browse/CB-7276
> Project: Apache Cordova
> Issue Type: Bug
> Components: Plugin Camera
> Affects Versions: 3.5.0
> Environment: Android
> Reporter: Petrus Repo
>
> Some Android devices (eg. Samsung S3 running Android 4.1 and Android 4.3)
> will set wrong orientation for pictures. Using correctOrientation==true will
> fix the resulting image to have the correct orientation.
> However, orientation is only corrected for JPEG images. If you set
> Camera.EncodingType.PNG then the resulting image will have wrong orientation
> on those devices carrying this bug. (eg. Nexus5 on 4.4 does not have the bug
> so it's not affected, but Samsung S3s are)
> Line CameraLauncher#processResultFromCamera:344 (*) has "if
> (this.encodingType == JPEG)" and inside the orientation is inspected. There
> should be no reason limit the orientation fix only for JPEG images.
> rotate = N is needed in order to apply the orientation fix (**).
> (*)
> https://github.com/apache/cordova-plugin-camera/blob/e4ab155fd097381151357b64e69a1e6f40b076a5/src/android/CameraLauncher.java#L344
> (**)
> https://github.com/apache/cordova-plugin-camera/blob/e4ab155fd097381151357b64e69a1e6f40b076a5/src/android/CameraLauncher.java#L371
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org
