[GitHub] cordova-plugin-statusbar pull request: fix warning on IOS8

2015-06-11 Thread jcesarmobile
Github user jcesarmobile commented on the pull request: https://github.com/apache/cordova-plugin-statusbar/pull/26#issuecomment-111018380 using [[UIDevice currentDevice] orientation] is not reliable maybe you can try with [[UIApplication sharedApplication] statusBarOrientation

[GitHub] cordova-lib pull request: Implementation for Unified platform API

2015-06-11 Thread vladimir-kotikov
Github user vladimir-kotikov commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/235#discussion_r32198647 --- Diff: cordova-lib/src/platforms/PlatformApi.js --- @@ -0,0 +1,365 @@ +/** +Licensed to the Apache Software Foundation (ASF) under

[GitHub] cordova-plugin-inappbrowser pull request: [] Fix zoom option not r...

2015-06-11 Thread syndbg
GitHub user syndbg opened a pull request: https://github.com/apache/cordova-plugin-inappbrowser/pull/104 [] Fix zoom option not respected on Android. Also minor whitespaces fixes. You can merge this pull request into a Git repository by running: $ git pull https://github.com/sy

[GitHub] cordova-plugin-inappbrowser pull request: CB-9158 - InAppBrowser z...

2015-06-11 Thread syndbg
Github user syndbg commented on a diff in the pull request: https://github.com/apache/cordova-plugin-inappbrowser/pull/104#discussion_r32199198 --- Diff: src/android/InAppBrowser.java --- @@ -656,7 +647,7 @@ public void onClick(View v) { WebSettings settings =

iOS signing custom rules

2015-06-11 Thread Victor Sosa
Hello community. I'm trying to understand iOS apps signing. For this I'm looking at this document: https://github.com/apache/cordova-docs/blob/master/docs/en/edge/guide/platforms/ios/tools.md Most of it is self-explanatory but I found "--codeSignResourceRules" kind of confusing as it doesn't men

[GitHub] cordova-lib pull request: Implementation for Unified platform API

2015-06-11 Thread omefire
Github user omefire commented on the pull request: https://github.com/apache/cordova-lib/pull/235#issuecomment-58775 Since you've already ported the android implementation from cordova-lib to cordova-android, shouldn't you delete the android parser file from the metadata directory

[GitHub] cordova-plugin-inappbrowser pull request: CB-3360: Set custom inap...

2015-06-11 Thread samkelleher
Github user samkelleher commented on the pull request: https://github.com/apache/cordova-plugin-inappbrowser/pull/94#issuecomment-66441 One side effect of this is that the main cordova webview will also have its userAgent changed (navigator.appVersion) once at least one instance o

[GitHub] cordova-lib pull request: Implementation for Unified platform API

2015-06-11 Thread vladimir-kotikov
Github user vladimir-kotikov commented on the pull request: https://github.com/apache/cordova-lib/pull/235#issuecomment-70935 `android_parser` is left for backward compatibility, since user can install older version of cordova-android, which doesn't provides own `PlatformAPI`. In

[DISCUSS] Switch to tilde versions for --save

2015-06-11 Thread Tim Barham
When we implemented --save for platforms and plugins, we decided to save versions as caret ranges to be consistent with current npm behavior. However, when we made our pinned platforms more flexible we decided to go with tilde ranges rather than caret ranges to keep things a bit more locked down

[GitHub] cordova-lib pull request: Implementation for Unified platform API

2015-06-11 Thread omefire
Github user omefire commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/235#discussion_r32236650 --- Diff: cordova-lib/src/platforms/PlatformApi.js --- @@ -0,0 +1,414 @@ +/** +Licensed to the Apache Software Foundation (ASF) under one

[GitHub] cordova-lib pull request: Implementation for Unified platform API

2015-06-11 Thread omefire
Github user omefire commented on the pull request: https://github.com/apache/cordova-lib/pull/235#issuecomment-84250 cool. (ref: android_parser & backward compatibility). --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] cordova-lib pull request: Implementation for Unified platform API

2015-06-11 Thread omefire
Github user omefire commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/235#discussion_r32236936 --- Diff: cordova-lib/src/platforms/PlatformApi.js --- @@ -0,0 +1,414 @@ +/** +Licensed to the Apache Software Foundation (ASF) under one

RE: [DISCUSS] Switch to tilde versions for --save

2015-06-11 Thread Mefire O .
+1 for moving towards consistency, following the same strategy as platforms that we pin down. Thanks, Mefire -Original Message- From: Tim Barham [mailto:tim.bar...@microsoft.com] Sent: Thursday, June 11, 2015 8:23 AM To: 'dev@cordova.apache.org' Subject: [DISCUSS] Switch to tilde versio

[GitHub] cordova-lib pull request: Implementation for Unified platform API

2015-06-11 Thread omefire
Github user omefire commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/235#discussion_r32238811 --- Diff: cordova-lib/src/cordova/prepare.js --- @@ -56,147 +53,44 @@ function prepare(options) { }) .then(function(){ opti

Re: Keeping Cordova app open in background all th time

2015-06-11 Thread Jay Patel
Dear team, Am not sure if I can do this but any assistance would be appreciated. I have a windows cordova app, within this app am opening another external app, as soon as I open the external app cordova app is minimised in the background. What I want to do is keep my Cordova app maximised so I ca

[GitHub] cordova-js pull request: CB 9156 - Move to sibling based platform ...

2015-06-11 Thread nikhilkh
Github user nikhilkh commented on the pull request: https://github.com/apache/cordova-js/pull/117#issuecomment-94328 LGTM. Thanks for making this change! --- 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

RE: Latest --browserify status?

2015-06-11 Thread Vladimir Kotikov (Akvelon)
Hey, guys. I've tried to run mobilespec today with '--browserify' option, and it seems that it still doesn't work. The application builds and runs, but after running none of tests are working (I'm aware of issues with test-framework, so I've tried to run non-plugin tests), and there is a lot o

RE: Building cordova.js on first build

2015-06-11 Thread Nikhil Khandelwal
Thanks for making this change. I know createmobilspec (and in turn the Buildbot CI) depends on this as it attempts to create the latest cordova.js. After this change it will generate the latest cordova.js (instead of the released one) for testing purposes which IMO is the correct default behavio

[GitHub] cordova-lib pull request: Implementation for Unified platform API

2015-06-11 Thread omefire
Github user omefire commented on the pull request: https://github.com/apache/cordova-lib/pull/235#issuecomment-111213610 Ah, I see. (ref: android_parser & backward compatibility). However, in case a platform is implemented via the new way, shouldn't we be actually using that ?

[GitHub] cordova-lib pull request: Implementation for Unified platform API

2015-06-11 Thread omefire
Github user omefire commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/235#discussion_r32246260 --- Diff: cordova-lib/src/platforms/PlatformApi.js --- @@ -0,0 +1,414 @@ +/** +Licensed to the Apache Software Foundation (ASF) under one

Re: [DISCUSS] Switch to tilde versions for --save

2015-06-11 Thread Steven Gill
Our thought process to use ~ over ^ for pinned platforms was because we don't have a solid testing matrix to make sure we are supporting backwards compatibility. If we are all sticking to semver (which I think we are), we should be able to allow minor updates too. It is just safer to allow only pa

RE: [DISCUSS] Switch to tilde versions for --save

2015-06-11 Thread Tim Barham
Yep, you summarized my thinking on this exactly :). -Original Message- From: Steven Gill [mailto:stevengil...@gmail.com] Sent: Thursday, June 11, 2015 10:51 AM To: dev@cordova.apache.org Subject: Re: [DISCUSS] Switch to tilde versions for --save Our thought process to use ~ over ^ for pi

Re: cordova-osx - mothball

2015-06-11 Thread Steven Gill
Thanks Nikhil, I will take a look at that PR as I work on this and provide feedback. On Wed, Jun 10, 2015 at 5:34 PM, Nikhil Khandelwal wrote: > There is a massive PR currently in the works that is looking to change how > platforms should expose their functionality: > https://github.com/apach

[GitHub] cordova-ios pull request: CB-8197 Switch to nodejs for ios platfor...

2015-06-11 Thread daserge
GitHub user daserge opened a pull request: https://github.com/apache/cordova-ios/pull/146 CB-8197 Switch to nodejs for ios platform scripts Convert `copy-www-build-step.sh` to nodejs Adds `glob` module and `bundledDependencies` to `package.json` [Jira issue](https://issu

Re: [iOS 8] WKWebView moving forward

2015-06-11 Thread Andrew Grieve
Could maybe try creating symlinks / hardlinks to save on space / creation time? On Wed, Jun 10, 2015 at 12:00 PM, Shazron wrote: > Safari View Controller video: > https://developer.apple.com/videos/wwdc/2015/?id=504 > > What's New in Web Development in WebKit and Safari video: > https://develope

RE: [iOS 8] WKWebView moving forward

2015-06-11 Thread Chuck Lantz
Hmmm. Symlinks will break if the code ever touches Windows but they could be safely created as a part of a before_compile hook since that would only ever run on OSX practically speaking. (after_prepare would still fire if you add iOS on Windows - which is something we talked about being a pote

Re: cordova-osx - mothball

2015-06-11 Thread Shazron
The potential maintainer for cordova-osx in this thread has decided to go in another direction -- the final nail in its coffin. Full speed ahead cordova-electron! Godspeed On Thu, Jun 11, 2015 at 10:59 AM, Steven Gill wrote: > Thanks Nikhil, > > I will take a look at that PR as I work on this an

Re: [iOS 8] WKWebView moving forward

2015-06-11 Thread Shazron
I've already tried both sym and hard links. They result in errors, which is to be expected (if not there's some 'splaining to do) :P GOOD news though today! They are fixing the app bundle loadFileURL bug: https://twitter.com/grorgwork/status/609052546179530752 Apparently you can also UI test in W

Re: cordova-osx - mothball

2015-06-11 Thread Kerri Shotts
I’ll throw in my +1. Electron is awesome, and I’ve used it to wrap a PG project for a client where they are demoing the app to their potential clients because they don’t have devices to share from, and they don’t have the expertise for emulators (they are sales reps). Because I can wrap everythi

Re: [iOS 8] WKWebView moving forward

2015-06-11 Thread Kerri Shotts
Yeah, I was just about to say that I was pretty sure that sym/hard links was going to error. But that tweet looks promising! I hope that means the fix makes it in time for Beta 2 so we can play with it. :-) On June 11, 2015 at 1:55:58 PM, Shazron (shaz...@gmail.com) wrote: I've already trie

[GitHub] cordova-coho pull request: Initial coho handling for the new cordo...

2015-06-11 Thread stevengill
Github user stevengill commented on the pull request: https://github.com/apache/cordova-coho/pull/84#issuecomment-111257786 Haven't tested this yet, but first question that comes to mind is how would this affect repo-clone? --- If your project is set up for it, you can reply to this

Re: Latest --browserify status?

2015-06-11 Thread Steven Gill
Thanks Vladimir for giving this a go. mobilespec with --browserify was working for me last time I tried but I will have to try it again. I'm assuming you are using the latest versions of things. Appreciate the tips for the windows environment. I need to test this on Windows and make some of the c

[GitHub] cordova-js pull request: CB 9156 - Move to sibling based platform ...

2015-06-11 Thread stevengill
Github user stevengill commented on the pull request: https://github.com/apache/cordova-js/pull/117#issuecomment-111261704 I am going to take another pass it this before I commit it. See if I can improve some of the path.join/fs.existSync logic. I also want to test this on Windows + w

[GitHub] cordova-coho pull request: Initial coho handling for the new cordo...

2015-06-11 Thread TimBarham
Github user TimBarham commented on the pull request: https://github.com/apache/cordova-coho/pull/84#issuecomment-111262521 Ah, good question. I'll try it out. I tried a few scenarios, and was trying to think of one where having two items in the repo list pointing to the same repo coul

Re: Proposal: Cordova Plugin Search

2015-06-11 Thread Steven Gill
Nice list of tasks. I'll try to find some time to look at the PR in the next few days. It might be nice to turn this search functionality (with the blacklist) into a module others can incorporate into their tools. We could potentially use it in the cordova cli (cordova plugin search) and maybe som

[DISCUSS] 5.1.1 Docs Release?

2015-06-11 Thread Steven Gill
Let me know when translations have been checked in and what languages need to get the new version. For example, only Russian and english docs have 5.0.0 currently. Edge for all of the other docs is recent though.

RE: [CI] cordova-vm.apache.org is up

2015-06-11 Thread Dmitry Blotsky
Hey Steve, let's set you up with access and sudo powers to cordova-vm.apache.org. Once you've successfully logged in to the machine, follow these steps to set up OPIE: https://reference.apache.org/committer/opie. Then Infra will grant you sudo powers, and we can close this ticket: https://issue

RE: [CI] cordova-vm.apache.org is up

2015-06-11 Thread Steven Gill
Thanks. I'll go through the steps On Jun 11, 2015 1:23 PM, "Dmitry Blotsky" wrote: > Hey Steve, let's set you up with access and sudo powers to > cordova-vm.apache.org. Once you've successfully logged in to the machine, > follow these steps to set up OPIE: > https://reference.apache.org/committer

Re: iOS signing custom rules

2015-06-11 Thread Shazron
It's for excluding certain files from code signing from a bundle. In Mavericks this has been removed -- all bundle files are signed, so I think that should go away. We might have a JIRA for this already. https://developer.apple.com/library/mac/technotes/tn2206/_index.html#//apple_ref/doc/uid/DTS40

Re: [DISCUSS] 5.1.1 Docs Release?

2015-06-11 Thread Victor Sosa
Hello Steven. I'm working on the translation right now, I expect to finish everything byt tomorrow evening. Will reply this thread when I'm done. On Thu, Jun 11, 2015 at 3:16 PM Steven Gill wrote: > Let me know when translations have been checked in and what languages need > to get the new vers

[GitHub] cordova-lib pull request: CB-9122 Added tests for platform/plugin ...

2015-06-11 Thread omefire
Github user omefire commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/246#discussion_r32271456 --- Diff: cordova-lib/spec-cordova/save.spec.js --- @@ -0,0 +1,435 @@ +/** +Licensed to the Apache Software Foundation (ASF) under one +

[GitHub] cordova-lib pull request: CB-9122 Added tests for platform/plugin ...

2015-06-11 Thread omefire
Github user omefire commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/246#discussion_r32271532 --- Diff: cordova-lib/spec-cordova/save.spec.js --- @@ -0,0 +1,435 @@ +/** +Licensed to the Apache Software Foundation (ASF) under one +

[GitHub] cordova-lib pull request: CB-9122 Added tests for platform/plugin ...

2015-06-11 Thread omefire
Github user omefire commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/246#discussion_r32272314 --- Diff: cordova-lib/spec-cordova/save.spec.js --- @@ -0,0 +1,435 @@ +/** +Licensed to the Apache Software Foundation (ASF) under one +

[GitHub] cordova-lib pull request: CB-9122 Added tests for platform/plugin ...

2015-06-11 Thread omefire
Github user omefire commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/246#discussion_r32273230 --- Diff: cordova-lib/spec-cordova/save.spec.js --- @@ -0,0 +1,435 @@ +/** +Licensed to the Apache Software Foundation (ASF) under one +

[GitHub] cordova-lib pull request: CB-9122 Added tests for platform/plugin ...

2015-06-11 Thread omefire
Github user omefire commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/246#discussion_r32273251 --- Diff: cordova-lib/spec-cordova/save.spec.js --- @@ -0,0 +1,435 @@ +/** +Licensed to the Apache Software Foundation (ASF) under one +

[GitHub] cordova-registry-web pull request: CB-9146 Initial commit for npm-...

2015-06-11 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-registry-web/pull/11 --- 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 fe

[GitHub] cordova-windows pull request: CB-9164 Better error message when de...

2015-06-11 Thread muratsu
GitHub user muratsu opened a pull request: https://github.com/apache/cordova-windows/pull/90 CB-9164 Better error message when deploying to Windows10 phone emulator Windows 10 Phone emulator is currently not supported but deployment from VS is possible. We should tell users to u

[GitHub] cordova-coho pull request: CB-8214 COHO. Rename windowsphone to wp...

2015-06-11 Thread nikhilkh
Github user nikhilkh commented on the pull request: https://github.com/apache/cordova-coho/pull/60#issuecomment-111293178 @sgrebnov Shall we close 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

[GitHub] cordova-lib pull request: CB-9126 Fix ios pbxproj' resources paths...

2015-06-11 Thread nikhilkh
Github user nikhilkh commented on the pull request: https://github.com/apache/cordova-lib/pull/237#issuecomment-111293336 LGTM. Good find! --- 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

[GitHub] cordova-lib pull request: CB-9122 Added tests for platform/plugin ...

2015-06-11 Thread omefire
Github user omefire commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/246#discussion_r32275358 --- Diff: cordova-lib/spec-cordova/helpers.js --- @@ -43,17 +54,101 @@ if (host.match(/win/)) { } */ -// Just use Android everywhere;

[GitHub] cordova-lib pull request: CB-9122 Added tests for platform/plugin ...

2015-06-11 Thread omefire
Github user omefire commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/246#discussion_r32275447 --- Diff: cordova-lib/spec-cordova/helpers.js --- @@ -43,17 +54,101 @@ if (host.match(/win/)) { } */ -// Just use Android everywhere;

June 2015 Board Report

2015-06-11 Thread Shazron
https://github.com/cordova/apache-board-reports/blob/master/2015/2015-06.md Let me know your comments, if any -- this time I just used the automatic tool at reporter.apache.org which collects stats that I had been doing manually. Releases still have to be added manually using that tool.

[GitHub] cordova-lib pull request: Use cp instead of mv, as it would fail i...

2015-06-11 Thread nikhilkh
Github user nikhilkh commented on the pull request: https://github.com/apache/cordova-lib/pull/245#issuecomment-111305397 @TimBarham @omefire to take a quick look --- 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 pr

Re: June 2015 Board Report

2015-06-11 Thread Brian LeRoux
looks great shaz: thank you! On Thu, Jun 11, 2015 at 4:25 PM, Shazron wrote: > https://github.com/cordova/apache-board-reports/blob/master/2015/2015-06.md > > Let me know your comments, if any -- this time I just used the automatic > tool at reporter.apache.org which collects stats that I had be

[GitHub] cordova-registry-web pull request: Adding download count for plugi...

2015-06-11 Thread rakatyal
GitHub user rakatyal opened a pull request: https://github.com/apache/cordova-registry-web/pull/12 Adding download count for plugin list You can merge this pull request into a Git repository by running: $ git pull https://github.com/rakatyal/cordova-registry-web raghav/downloa

[GitHub] cordova-registry-web pull request: CB-9165 Adding download count f...

2015-06-11 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-registry-web/pull/12 --- 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 fe

[GitHub] cordova-lib pull request: Use cp instead of mv, as it would fail i...

2015-06-11 Thread omefire
Github user omefire commented on the pull request: https://github.com/apache/cordova-lib/pull/245#issuecomment-111337985 I'm trying to reproduce this error. @sencenan does this happen as well when you use different partitions of the same device ? or is it only on totally different dev

[GitHub] cordova-android pull request: CB-9149: Make gradle alias subprojec...

2015-06-11 Thread agrieve
Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-android/pull/182#discussion_r32287653 --- Diff: bin/templates/cordova/lib/build.js --- @@ -245,18 +258,29 @@ var builders = { } } +

[GitHub] cordova-android pull request: CB-9149: Make gradle alias subprojec...

2015-06-11 Thread agrieve
Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-android/pull/182#discussion_r32287669 --- Diff: bin/templates/cordova/lib/build.js --- @@ -245,18 +258,29 @@ var builders = { } } +

[GitHub] cordova-android pull request: CB-9149: Make gradle alias subprojec...

2015-06-11 Thread agrieve
Github user agrieve commented on a diff in the pull request: https://github.com/apache/cordova-android/pull/182#discussion_r3228 --- Diff: bin/templates/cordova/lib/build.js --- @@ -102,6 +102,19 @@ function hasCustomRules() { return fs.existsSync(path.join(ROOT, 'custo

[GitHub] cordova-android pull request: CB-9149: Make gradle alias subprojec...

2015-06-11 Thread agrieve
Github user agrieve commented on the pull request: https://github.com/apache/cordova-android/pull/182#issuecomment-111340031 a few nits, but LGTM! Thanks for doing 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

Re: [DISCUSS] 5.1.1 Docs Release?

2015-06-11 Thread Andrey Kurdumov
Hi, I would like that docs for `clean` hooks would be also included, so both English and Russian version would be exactly the same. This is require that docs would be updated, very easy fix - See ( https://github.com/apache/cordova-docs/pull/293/files) Also before releasing Russian docs, I have t

[GitHub] cordova-lib pull request: CB-9162 - Default Variables

2015-06-11 Thread ogoguel
GitHub user ogoguel opened a pull request: https://github.com/apache/cordova-lib/pull/247 CB-9162 - Default Variables You can merge this pull request into a Git repository by running: $ git pull https://github.com/ogoguel/cordova-lib variables Alternatively you can review and