You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cordova.apache.org by "Nikhil Khandelwal (JIRA)" <ji...@apache.org> on 2016/05/16 17:59:12 UTC

[jira] [Commented] (CB-11269) Android quality is set to 80, but default should be 50.

    [ https://issues.apache.org/jira/browse/CB-11269?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15284949#comment-15284949 ] 

Nikhil Khandelwal commented on CB-11269:
----------------------------------------

Should the default be 50? I believe the intent was that it will use lesser memory and hence, default of 50 is a good thing - but I did some profiling and setting the quality to anything but 100 on Android actually causes more memory consumption because the image has to be loaded as a bitmap and re-compressed. Setting appropriate target height & width along with FILE_URI is the correct way to reduce the memory requirements. I propose that the default be changed to 100.


> Android quality is set to 80, but default should be 50.
> -------------------------------------------------------
>
>                 Key: CB-11269
>                 URL: https://issues.apache.org/jira/browse/CB-11269
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: Plugin Camera
>            Reporter: jcesarmobile
>            Assignee: jcesarmobile
>            Priority: Trivial
>
> CameraLauncher.java sets quality to 80, but according to the docs it should be 50.
> It's not a real problem as Camera.js will pass quality 50 if no quality option is set. But the java code should have the 50 value too so people isn't confused.
> https://github.com/apache/cordova-plugin-camera/blob/master/www/Camera.js#L145



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org