cordova-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CB-9193) [Camera] Add option to choose from library even if source is camera
Date Thu, 02 Jul 2015 23:36:04 GMT

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

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

GitHub user dpolivy opened a pull request:

    https://github.com/apache/cordova-plugin-camera/pull/104

    CB-9193: Add 'showLibraryButton' to allow choosing source

    A new camera option, `showLibraryButton`, is added which allows the user
    to easily switch from the camera to the photo library once they have
    entered the plugin.
    
    Currently, this is optimally supported on iOS (7+), with additional
    implementations for Android and WP8 based on what was available for
    those platforms. Please see the documentation updates (specifically, the 'Quirks' changes)
for details on how this is implemented on each platform.
    
    I realize that the behavior on Android and Windows Phone is not 100% consistent to iOS,
but it is the best that can be done given the constraints of what's available on those platforms.
If we were to rewrite this plugin to use completely custom camera UIs, we could do a whole
lot better -- but that's a much bigger undertaking.
    
    I also made some opportunistic changes while I was working on this, including cleaning
up parameter initialization for Android, moving hardcoded strings to localizable ones, fixing
some space/tab and indentation inconsistencies, and other code cleanup.
    
    All modifications here do not impact the default functionality of someone who is currently
using this plugin (though it would be great for additional test coverage on that). In order
for this change to work, `showLibraryButton: true` must be passed in the options, and the
source type must be `CAMERA`.
    
    @infil00p @shazron @nikhilkh @stevengill Is it possible to get reviews for your respective
platforms?
    
    Fixes https://issues.apache.org/jira/browse/CB-9193

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/dpolivy/cordova-plugin-camera cb9193

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cordova-plugin-camera/pull/104.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #104
    
----
commit c84b1f63a4fb8b51db78c2a735cb88a8c88b9b2f
Author: Dan Polivy <dan@cellartracker.com>
Date:   2015-06-17T22:55:20Z

    CB-9193: Add 'showLibraryButton' to allow choosing source
    
    A new camera option, showLibraryButton, is added which allows the user
    to easily switch from the camera to the photo library once they have
    entered the plugin.
    
    Currently, this is optimally supported on iOS (7+), with additional
    implementations for Android and WP8 based on what was available for
    those platforms.

----


> [Camera] Add option to choose from library even if source is camera
> -------------------------------------------------------------------
>
>                 Key: CB-9193
>                 URL: https://issues.apache.org/jira/browse/CB-9193
>             Project: Apache Cordova
>          Issue Type: Improvement
>          Components: Plugin Camera
>            Reporter: Dan Polivy
>
> There are many scenarios where an app might allow a user to take a picture via the camera,
OR choose a photo from their library. With the current camera plugin implementation, the developer
must first present some type of chooser UI to allow the user to initiate either the camera,
or the library.
> A potentially more efficient approach is to launch the camera by default, and within
the camera UI provide an option to switch over to the library -- similar to the way the default
Camera app works in iOS.
> I am working on a PR to add this functionality via a new CameraOptions setting.



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


Mime
View raw message