[GitHub] cordova-plugin-dialogs pull request: CB-9347 - fix to allow to sta...

2016-02-12 Thread tony--
Github user tony-- commented on the pull request: https://github.com/apache/cordova-plugin-dialogs/pull/54#issuecomment-183488008 Thanks! I cherry-picked it into a fork; we'll let you know if it worked for us ASAP. --- If your project is set up for it, you can reply to this email a

[GitHub] cordova-plugin-dialogs pull request: CB-9347 - fix to allow to sta...

2016-02-12 Thread jcesarmobile
Github user jcesarmobile commented on the pull request: https://github.com/apache/cordova-plugin-dialogs/pull/54#issuecomment-183466583 I've just send the PR. https://github.com/phonegap/phonegap-plugin-barcodescanner/pull/170 --- If your project is set up for it, you can r

[GitHub] cordova-plugin-dialogs pull request: CB-9347 - fix to allow to sta...

2016-02-12 Thread tony--
Github user tony-- commented on the pull request: https://github.com/apache/cordova-plugin-dialogs/pull/54#issuecomment-183376049 @jcesarmobile - thanks for explaining `presentAlertcontroller,` I see it now. Thanks in advance for the phonegap-plugin-barcodescanner PR! --- If you

[GitHub] cordova-plugin-dialogs pull request: CB-9347 - fix to allow to sta...

2016-02-11 Thread jcesarmobile
Github user jcesarmobile commented on the pull request: https://github.com/apache/cordova-plugin-dialogs/pull/54#issuecomment-183218924 It's supposed to be recursive. If count > 1 it just stores it on the array (line 100), then, once it's presented (line 230), removes the presented a

[GitHub] cordova-plugin-dialogs pull request: CB-9347 - fix to allow to sta...

2016-02-11 Thread tony--
Github user tony-- commented on the pull request: https://github.com/apache/cordova-plugin-dialogs/pull/54#issuecomment-183153364 @jcesarmobile - With regard to phonegap-plugin-barcodescanner+cordova-plugin-dialogs, I was chatting with @imaffett about this. He found that reverting f

[GitHub] cordova-plugin-dialogs pull request: CB-9347 - fix to allow to sta...

2016-02-09 Thread imaffett
Github user imaffett commented on the pull request: https://github.com/apache/cordova-plugin-dialogs/pull/54#issuecomment-181897670 @jcesarmobile - thanks for the prompt reply. This is the plugin https://github.com/phonegap/phonegap-plugin-barcodescanner.git --- If your project i

[GitHub] cordova-plugin-dialogs pull request: CB-9347 - fix to allow to sta...

2016-02-09 Thread jcesarmobile
Github user jcesarmobile commented on the pull request: https://github.com/apache/cordova-plugin-dialogs/pull/54#issuecomment-181897098 getTopPresentedViewController returns the view controller on the top, not necessarily the mainViewController, the dialog has to be presented over the

[GitHub] cordova-plugin-dialogs pull request: CB-9347 - fix to allow to sta...

2016-02-09 Thread imaffett
Github user imaffett commented on the pull request: https://github.com/apache/cordova-plugin-dialogs/pull/54#issuecomment-181886490 I'm running into a similar issue where showing an alert after using the phonegap-barcode-scanner plugin is failing. the getTopPresentedViewController m

[GitHub] cordova-plugin-dialogs pull request: CB-9347 - fix to allow to sta...

2015-10-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-plugin-dialogs/pull/54 --- 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

[GitHub] cordova-plugin-dialogs pull request: CB-9347 - fix to allow to sta...

2015-08-03 Thread initialxy
Github user initialxy commented on the pull request: https://github.com/apache/cordova-plugin-dialogs/pull/54#issuecomment-127353615 Awesome, thanks @jcesarmobile, much appreciated. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cordova-plugin-dialogs pull request: CB-9347 - fix to allow to sta...

2015-07-16 Thread jcesarmobile
GitHub user jcesarmobile opened a pull request: https://github.com/apache/cordova-plugin-dialogs/pull/54 CB-9347 - fix to allow to stack multiple UIAlertControllers Now you can stack multiple UIAlertControllers I had to add a 0.5s delay in case the user tries to present a ne