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

ASF GitHub Bot commented on CB-10180:
-------------------------------------

Github user nleclerc commented on the issue:

    https://github.com/apache/cordova-plugin-media-capture/pull/65
  
    If i may, i think that default behavior not being one of the explicit 
choices available is a bad idea.
    It doesn't allow you to proof your code against default behavior changes.
    
    I submitted another PR 
(https://github.com/apache/cordova-plugin-media-capture/pull/48) to implement 
video quality in iOS but the real problem is that whoever made the android 
quality change did quite a bad job at it.
    
    If we don't want to end up with stupid options (like 0 low, 1 high and 2 
medium) we're gonna have to break compatibility.
    Or Maybe we could use some String option values to have more verbose 
quality levels and keep the numeric ones to maintain compatibility.


> Add support for video capture quality on iOS
> --------------------------------------------
>
>                 Key: CB-10180
>                 URL: https://issues.apache.org/jira/browse/CB-10180
>             Project: Apache Cordova
>          Issue Type: Improvement
>          Components: Plugin Media Capture
>    Affects Versions: 1.1.0
>         Environment: iOS
>            Reporter: Nicolas Leclerc
>              Labels: iOS, triaged
>
> iOS does not support the new {{quality}} option.
> Adding it would be great.
> I submitted a PR some time ago but rebased it to v1.1.0.
> https://github.com/apache/cordova-plugin-media-capture/pull/48



--
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

Reply via email to