You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cordova.apache.org by muratsu <gi...@git.apache.org> on 2015/04/16 02:22:56 UTC

[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

GitHub user muratsu opened a pull request:

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

    CB-8054 Support taking pictures from file for WP8

    Add missing take picture from file capability for WP8

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

    $ git pull https://github.com/MSOpenTech/cordova-plugin-camera CB-8054

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

    https://github.com/apache/cordova-plugin-camera/pull/86.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 #86
    
----
commit 2175a03e659a9c81243624841a04580db3c0ac3c
Author: Murat Sutunc <su...@gmail.com>
Date:   2015-04-16T00:20:30Z

    CB-8054 Support taking pictures from file for WP8

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by muratsu <gi...@git.apache.org>.
Github user muratsu commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-94503319
  
    This one is ready to be merged. Would be glad if someone can merge this in.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by muratsu <gi...@git.apache.org>.
Github user muratsu commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-105603159
  
    You can get the latest (edge) from [github](https://github.com/apache/cordova-plugin-camera.git) however I suggest you clone the github repo and checkout 19feee9cb049 for version 1.1.0. Releases are tested more thoroughly than edge. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by muratsu <gi...@git.apache.org>.
Github user muratsu commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-105752108
  
    That just got fixed today. Please update :) 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by ekambarrao <gi...@git.apache.org>.
Github user ekambarrao commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-131809954
  
    I used a cordova navigation sample. 
    
    worked perfectly alright up to some extent. 
    
    When I click on the camera, it perfectly returns to library, 
    
    click save, 
    
    Important Point: "it returns to page then tries to render the image"
    
    I started making changes line by line and was testing it......to see if this makes difference or that makes difference...it broke at one point...
    
    i rolled back changes...but it was not rolling back....
    
    Does this have any thing to do cordovaApp_temporarykey.pfx?
    
    do we need to have this file in the solution to have this work?
    
    when i create another application with the same lines of code it was not working......what is happening guys....cordova is really very bad in managing its state or I am using winjs for navigation, it is bad in managing the state....it is not hitting the resume event in index.js at all.... to capture the reason for failure.....


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by ekambarrao <gi...@git.apache.org>.
Github user ekambarrao commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-129221770
  
    Muratsu, Can you suggest me a solution for this as my app has been held back only due to this!!!.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by ekambarrao <gi...@git.apache.org>.
Github user ekambarrao commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-129891456
  
    Muratsu, please find the following videos where you can see the problems with this plugin.
    
    works in windows emulator.
    
    https://www.youtube.com/watch?v=e17Dhujnm1A&feature=youtu.be
    
    android device failure, not able to record the emulator, but it is also failing there. 
    
    https://www.youtube.com/watch?v=Y7cQHc3i5Ew&feature=youtu.be
    
    Windows device failure
    
    https://www.youtube.com/watch?v=LLofT2ghxIs&feature=youtu.be
    
    What figure out is, it is not retaining the app state, earlier it was not that way, do i need to explicitly maintain the app state?
    
    If you can't access let me know.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by ekambarrao <gi...@git.apache.org>.
Github user ekambarrao commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-105581675
  
    hi all, is this problem resolved for windows phone universal. I am not able to use this for accessing photolibrary. Also camera is not working as it is with windows phone 8. Your quick reply is very helpful to me.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by muratsu <gi...@git.apache.org>.
Github user muratsu commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-105598275
  
    Make sure you've version 1.1.0 (May 06, 2015). You can check installed plugin version by executing `cordova plugin` on your project root. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by vladimir-kotikov <gi...@git.apache.org>.
Github user vladimir-kotikov commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-93689388
  
    LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by muratsu <gi...@git.apache.org>.
Github user muratsu commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-93788932
  
    The docs didn't mention this as a quirk, so no doc updates.
    I've also updated the phone video codec list. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by purplecabbage <gi...@git.apache.org>.
Github user purplecabbage commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-131932455
  
    I don't see anything at the url you posted.
    
    
    
    My team is hiring!
    @purplecabbage
    risingj.com
    
    On Mon, Aug 17, 2015 at 10:23 AM, ekambarrao <no...@github.com>
    wrote:
    
    > @purplecabbage <https://github.com/purplecabbage> this is like light in
    > dark....your offer itself is like a great savior to me....:-)
    >
    > I have uploaded the solutions @ the following onedrive
    >
    >
    > https://onedrive.live.com/?id=463D607901FE1563%21114&cid=463D607901FE1563&group=0
    >
    > blankcordovaapp1 is failing where I experimented.
    >
    > javascript(2) has been working fine.
    >
    > they were in VS 2013 and it is picking up cordova 3.7.1
    >
    > I have moved the code to VS 2015 and I selected 5.1.1 s platform but it is
    > showing 4.0 folder in .cordova folder.
    >
    > 2015 working as long as the device in debug when disconnected again the
    > same.
    >
    > —
    > Reply to this email directly or view it on GitHub
    > <https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-131895353>
    > .
    >



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by ekambarrao <gi...@git.apache.org>.
Github user ekambarrao commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-131806741
  
    @purplecabbage , mine is cordova javascript application. I am really in a very bad shape due to lack of support from all OEM of these techs. Very bad!!!! Its been 500 hours of trials after trails to figure out how to make this work consistent way without fail but could not resolve this. Stackoverflow did not help, slack did not help.....what is the wrong .....where....god only knows....


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by muratsu <gi...@git.apache.org>.
Github user muratsu commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-105584955
  
    Yes, this is resolved for windows phone universal. 
    What problem are you running into with winphone8? You can always open a [jira](https://issues.apache.org/jira/browse/CB-9074?jql=project%20%3D%20CB) for us to investigate. The more information you provide, faster we can resolve the issue. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by muratsu <gi...@git.apache.org>.
Github user muratsu commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-128766483
  
    Hmm, can you share your camera settings with me? I've tried the following with ee5cfe89 (master):
    ```
            destinationType: Camera.DestinationType.FILE_URI,
            encodingType: Camera.EncodingType.JPEG,
            sourceType : Camera.PictureSourceType.PHOTOLIBRARY
    ```
    ```
            destinationType: Camera.DestinationType.FILE_URI,
            encodingType: Camera.EncodingType.JPEG,
            sourceType : Camera.PictureSourceType.CAMERA,
            saveToPhotoAlbum: true
    ```
    
    both of them working fine for me (win-phone 8.1).


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by ekambarrao <gi...@git.apache.org>.
Github user ekambarrao commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-130137902
  
    As I was doubtful, the issue was related to App State Maintenance. Some how it has been poorly maintained in the app. This experience and solution might be useful to others.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by nikhilkh <gi...@git.apache.org>.
Github user nikhilkh commented on a diff in the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#discussion_r28528079
  
    --- Diff: src/windows/CameraProxy.js ---
    @@ -134,16 +134,84 @@ function resizeImageBase64(successCallback, errorCallback, file, targetWidth, ta
         }, function(err) { errorCallback(err); });
     }
     
    -function takePictureFromFile(successCallback, errorCallback, mediaType, destinationType, targetWidth, targetHeight, encodingType) {
    -    // TODO: Add WP8.1 support
    -    // WP8.1 doesn't allow to use of pickSingleFileAsync method
    -    // see http://msdn.microsoft.com/en-us/library/windows/apps/br207852.aspx for details
    -    // replacement of pickSingleFileAsync - pickSingleFileAndContinue method
    -    // will take application to suspended state and this require additional logic to wake application up
    -    if (navigator.appVersion.indexOf('Windows Phone 8.1') >= 0 ) {
    -        errorCallback('Not supported');
    -        return;
    +function takePictureFromFile(successCallback, errorCallback, args) {
    +    // Detect Windows Phone
    +    if (navigator.appVersion.indexOf('Windows Phone 8.1') >= 0) {
    +        takePictureFromFileWP(successCallback, errorCallback, args);
    +    } else {
    +        takePictureFromFileWindows(successCallback, errorCallback, args);
         }
    +}
    +
    +function takePictureFromFileWP(successCallback, errorCallback, args) {
    +    var mediaType = args[6],
    +        destinationType = args[1],
    +        targetWidth = args[3],
    +        targetHeight = args[4],
    +        encodingType = args[5];
    +
    +    /*
    +        Need to add and remove an event listener to catch activation state
    +        Using FileOpenPicker will suspend the app and it's required to catch the PickSingleFileAndContinue
    +        https://msdn.microsoft.com/en-us/library/windows/apps/xaml/dn631755.aspx
    +    */
    +    var filePickerActivationHandler = function(eventArgs) {
    +        if (eventArgs.kind === Windows.ApplicationModel.Activation.ActivationKind.pickFileContinuation) {
    +            var file = eventArgs.files[0];
    +            if (!file) {
    +                errorCallback("User didn't choose a file.");
    +                Windows.UI.WebUI.WebUIApplication.removeEventListener("activated", filePickerActivationHandler);
    +                return;
    +            }
    +            if (destinationType == Camera.DestinationType.FILE_URI || destinationType == Camera.DestinationType.NATIVE_URI) {
    +                if (targetHeight > 0 && targetWidth > 0) {
    +                    resizeImage(successCallback, errorCallback, file, targetWidth, targetHeight, encodingType);
    +                }
    +                else {
    +                    var storageFolder = Windows.Storage.ApplicationData.current.localFolder;
    +                    file.copyAsync(storageFolder, file.name, Windows.Storage.NameCollisionOption.replaceExisting).done(function (storageFile) {
    +                        successCallback(URL.createObjectURL(storageFile));
    +                    }, function () {
    +                        errorCallback("Can't access localStorage folder.");
    +                    });
    +                }
    +            }
    +            else {
    +                if (targetHeight > 0 && targetWidth > 0) {
    +                    resizeImageBase64(successCallback, errorCallback, file, targetWidth, targetHeight);
    +                } else {
    +                    Windows.Storage.FileIO.readBufferAsync(file).done(function (buffer) {
    +                        var strBase64 = Windows.Security.Cryptography.CryptographicBuffer.encodeToBase64String(buffer);
    +                        successCallback(strBase64);
    +                    }, errorCallback);
    +                }
    +            }
    +            Windows.UI.WebUI.WebUIApplication.removeEventListener("activated", filePickerActivationHandler);
    +        }
    +    };
    +
    +    var fileOpenPicker = new Windows.Storage.Pickers.FileOpenPicker();
    +    fileOpenPicker.suggestedStartLocation = Windows.Storage.Pickers.PickerLocationId.documentsLibrary;
    +    if (mediaType == Camera.MediaType.PICTURE) {
    +        fileOpenPicker.fileTypeFilter.replaceAll([".png", ".jpg", ".jpeg"]);
    +    }
    +    else if (mediaType == Camera.MediaType.VIDEO) {
    +        fileOpenPicker.fileTypeFilter.replaceAll([".avi", ".flv", ".asx", ".asf", ".mov", ".mp4", ".mpg", ".rm", ".srt", ".swf", ".wmv", ".vob"]);
    --- End diff --
    
    Wonder if this list of video formats should be shared with the one from windows below? It's amazing that so many video formats are supported here.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by ekambarrao <gi...@git.apache.org>.
Github user ekambarrao commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-128634479
  
    Hi, I am back....I used this earlier and was working perfectly alright. The same code is not working now. Camera is abending when saving to library on phone..Gallery lists the library and abends....kicks out of application. i reimplemented the plugin, no use. I have downloaded the 1.2.1, but no use. Any idea what could be the reason!!? Environment is Windows 8.1 phone universal.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by ekambarrao <gi...@git.apache.org>.
Github user ekambarrao commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-105600781
  
    Thanks for quick reply. can you pls share me the url link of camera plugin! I am using org.apache.cordova.camera plugin of visual studio and seems that I am outdated here with this.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by ekambarrao <gi...@git.apache.org>.
Github user ekambarrao commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-105754685
  
    It is working perfect now thx.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by ekambarrao <gi...@git.apache.org>.
Github user ekambarrao commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-132004684
  
    i have a question...do i need to capture the state of the app while capturing the photo or gone to library? or the plugin will take care of that?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by ekambarrao <gi...@git.apache.org>.
Github user ekambarrao commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-128988322
  
    I just reinstalled Android and Cordova, what the difference in behavior i have seen is, earlier, it was exiting in nano seconds, now it is showing the library and waiting there for a second in a no response state and then exiting. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by nikhilkh <gi...@git.apache.org>.
Github user nikhilkh commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-93784451
  
    Is there a docs update needed - now that this is supported?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by purplecabbage <gi...@git.apache.org>.
Github user purplecabbage commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-130357726
  
    You can try this: 
    https://msdn.microsoft.com/en-us/library/windows/apps/jj735579(v=vs.105).aspx
     
    
    To enable Fast Resume for your app, add the ActivationPolicy attribute to the DefaultTask element in WMAppManifest.xml and set the value to “Resume”. For this task, you need to edit the app manifest directly instead of using the manifest editor. To do this, right-click WMAppManifest.xml, click Open with, and then choose XML (Text) Editor.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by ekambarrao <gi...@git.apache.org>.
Github user ekambarrao commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-105748756
  
    Hi Muratsu, thanks for quick reply again. I was good with camera but bad with gallery. Now good with Gallery but bad with camera. It is breaking with error "Camera Initialization error TypeError: getAvailableMediaStreamProperties: 'this' is not an Inspectable Object". Can you suggest the solution please! thx


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by ekambarrao <gi...@git.apache.org>.
Github user ekambarrao commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-131895353
  
    @purplecabbage this is like light in dark....your offer itself is like a great savior to me....:-)
    
    I have uploaded the solutions @ the following onedrive
    
    https://onedrive.live.com/?id=463D607901FE1563%21114&cid=463D607901FE1563&group=0
    
    blankcordovaapp1 is failing where I experimented.
    
    javascript(2) has been working fine.
    
    they were in VS 2013 and it is picking up cordova 3.7.1
    
    I have moved the code to VS 2015 and I selected 5.1.1 s platform but it is showing 4.0 folder in .cordova folder.
    
    2015 working as long as the device in debug when disconnected again the same.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by ekambarrao <gi...@git.apache.org>.
Github user ekambarrao commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-128982914
  
    I am not sure what is the reason, my code is working fine, plugin is working fine, but after the closure  of execution of returning the image to page of App, Android 19 with Cordova 3.6.4, after the execution, cordova is executing further that is killing the app. Where the same may be true in other platforms as well. What is your suggestion in the platform versions, what should I be using?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by purplecabbage <gi...@git.apache.org>.
Github user purplecabbage commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-131881725
  
    Please post your non-working example, and I will look at it.
    Speculation is dangerous.
    
    This has nothing to do with cordovaApp_temporarykey.pfx
    Define 'rolled back changes'... via version control? did you run 'cordova prepare'?



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by nikhilkh <gi...@git.apache.org>.
Github user nikhilkh commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-132002862
  
    I agree with @purplecabbage that this is not the correct forum for this. Please use stack overflow and mark it with visual studio cordova. People from the product team look at that forum and will respond to you.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by ekambarrao <gi...@git.apache.org>.
Github user ekambarrao commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-105754391
  
    yep...now it is working perfect. however I am looking for one more suggestion about, how to cancel the camera!! 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by purplecabbage <gi...@git.apache.org>.
Github user purplecabbage commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-132002001
  
    Okay, it appears you are doing some magic with the Visual Studio TACO ( tools for apache cordova )
    You will need to bring this up with the visual studio team, as ultimately what you are running is NOT strictly a cordova project.  
    I was able to run your javascript(2) version as posted, but the BlankCordovaApp1 app was missing multiple files and could not be built.
    Ultimately this is not the place to have this discussion, as it has nothing to do with this pull request.
    If you need to discuss this further, please open a jira issue, or contact me on the cordova slack channel.
    to join :
    http://slack.cordova.io/ 
    to view :
    https://cordova.slack.com/
    
    I am @purplecabbage everywhere.
    



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by nikhilkh <gi...@git.apache.org>.
Github user nikhilkh commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-93841334
  
    LGTM - Please rebase for a merge


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by ekambarrao <gi...@git.apache.org>.
Github user ekambarrao commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-105597375
  
    I am having the problem of "not Supported" error. Would you suggest me to reinstall the plugin?



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by ekambarrao <gi...@git.apache.org>.
Github user ekambarrao commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-132003333
  
    Thanks guys...for taking time on this....


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by ekambarrao <gi...@git.apache.org>.
Github user ekambarrao commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-131989448
  
    <iframe src="https://onedrive.live.com/embed?cid=463D607901FE1563&resid=463D607901FE1563%21114&authkey=ADmtINFmJhqVSBI" width="165" height="128" frameborder="0" scrolling="no"></iframe> 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


[GitHub] cordova-plugin-camera pull request: CB-8054 Support taking picture...

Posted by ekambarrao <gi...@git.apache.org>.
Github user ekambarrao commented on the pull request:

    https://github.com/apache/cordova-plugin-camera/pull/86#issuecomment-105808283
  
    muratsu, I have footer in the page where I activate the camera, camera has "cancel" but it is going under the footer. How can I control the size of the camera screen??


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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