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

[GitHub] cordova-plugin-media-capture pull request: Various Fixes to crashe...

GitHub user aroberson opened a pull request:

    https://github.com/apache/cordova-plugin-media-capture/pull/38

    Various Fixes to crashes in Windows Phone 8

    This pull request contains the following fixes.
    
    1.  The app crashes in wp8 when trying to take a picture or video and use the back arrow key.  This is fixed by checking for defined file object before processing it.
    2.  The app crashes in wp8 when using gestures on the capture picture or capture video screens.  The solution is to not use the click event and move to the MSGestures events for gestures.  This change replaces the onclick event with listening for MSGestureTap event.

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

    $ git pull https://github.com/ABB-Austin/cordova-plugin-media-capture master

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

    https://github.com/apache/cordova-plugin-media-capture/pull/38.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 #38
    
----
commit 600afcab6c82819d5b065311054eea9c95dd4adf
Author: aroberson <ad...@ventyx.abb.com>
Date:   2015-05-19T16:14:15Z

    Fixed cancel of image capture to return no media

commit acf815e6faa9c460c0b89fc05731b33811139f42
Author: aroberson <ad...@ventyx.abb.com>
Date:   2015-05-19T16:16:13Z

    remove dependency

commit 18b3a8cfba95e9f2e533898d3fc116ddd5c7f3a8
Author: aroberson <ad...@ventyx.abb.com>
Date:   2015-05-19T16:19:38Z

    Remove file dependency

commit 7af759049eaf34958f852d9ed20d9dda1c7a61ca
Author: aroberson <ad...@ventyx.abb.com>
Date:   2015-05-19T16:35:57Z

    Added back dependency

commit 13a60dd6cd48b1e477023268b5351434282357c1
Author: aroberson <ad...@ventyx.abb.com>
Date:   2015-05-20T19:57:16Z

    Added catch error function for unsupported actions.

commit cb19f02d048f9bff131e51671314fdc9b3dac06c
Author: aroberson <ad...@ventyx.abb.com>
Date:   2015-06-01T18:12:41Z

    Catch exceptions caused by zoom events

commit 3d0af79944df315ff6d3318e773f44319b9ae3c1
Author: aroberson <ad...@ventyx.abb.com>
Date:   2015-06-04T16:28:56Z

    Removed the on click for camera preview and replaced with gesture tracking.

commit 77b01c564989862f127b6f93411cba59add9ae29
Author: aroberson <ad...@ventyx.abb.com>
Date:   2015-06-04T18:41:26Z

    Fixed Tap Gesture processing for ui

----


---
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-media-capture issue #38: Various Fixes to crashes in Windows ...

Posted by cordova-qa <gi...@git.apache.org>.
Github user cordova-qa commented on the issue:

    https://github.com/apache/cordova-plugin-media-capture/pull/38
  
    Cordova CI Build has completed successfully.
    
    **Commit**     - [Link](https://github.com/apache/cordova-plugin-media-capture/pull/38/commits/29961a01717c5345a4d2f35c9328578207c52ac9)
    **Dashboard** - [Link](http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-capture-pr/34/)
    
    | Builder Name  | Console Output | Test Report | Device Logs  |
    |     :---:     |     :---:      |   :---:     |     :---:    |
    | [Windows 10  Store]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-capture-pr/34//PLATFORM=windows-10-store/) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-capture-pr/34//PLATFORM=windows-10-store/console) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-capture-pr/34//PLATFORM=windows-10-store/testReport/) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-capture-pr/34//PLATFORM=windows-10-store/artifact/) |
    | [iOS 9.3]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-capture-pr/34//PLATFORM=ios-9.3/) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-capture-pr/34//PLATFORM=ios-9.3/console) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-capture-pr/34//PLATFORM=ios-9.3/testReport/) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-capture-pr/34//PLATFORM=ios-9.3/artifact/) |
    | [iOS 10.0]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-capture-pr/34//PLATFORM=ios-10.0/) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-capture-pr/34//PLATFORM=ios-10.0/console) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-capture-pr/34//PLATFORM=ios-10.0/testReport/) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-capture-pr/34//PLATFORM=ios-10.0/artifact/) |
    | [Android 4.4]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-capture-pr/34//PLATFORM=android-4.4/) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-capture-pr/34//PLATFORM=android-4.4/console) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-capture-pr/34//PLATFORM=android-4.4/testReport/) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-capture-pr/34//PLATFORM=android-4.4/artifact/) |
    | [Android 5.1]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-capture-pr/34//PLATFORM=android-5.1/) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-capture-pr/34//PLATFORM=android-5.1/console) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-capture-pr/34//PLATFORM=android-5.1/testReport/) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-capture-pr/34//PLATFORM=android-5.1/artifact/) |
     



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