[jira] [Commented] (CB-12242) Use yarn js instead of npm when adding plugins

2018-03-03 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CB-12242:
-

janpio commented on issue #292: CB-12242 : Use yarn instead of npm
URL: https://github.com/apache/cordova-cli/pull/292#issuecomment-370184568
 
 
   Tests are failing because of a code style problem:
   
   > C:\projects\cordova-cli\src\cli.js
   >  496:34  error  Unexpected trailing comma  comma-dangle
   
   Could you please fix that so the actual tests can run an this can possibly 
be merged? Thanks.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Use yarn js instead of npm when adding plugins
> --
>
> Key: CB-12242
> URL: https://issues.apache.org/jira/browse/CB-12242
> Project: Apache Cordova
>  Issue Type: Improvement
>  Components: cordova-cli, cordova-lib
>Reporter: Jacques de Villiers
>Priority: Major
>
> Currently it can take quite long to add certain plugins to my project (using 
> cordova plugin add), especially if I need to re-add the plugins. My 
> suggestion is to update the cordova cli to start using yarn js instead of npm 
> directly.
> When I looked at this page, I realised yarn was just a wrapper for npm, and 
> much better at caching packages locally.
> https://shift.infinite.red/npm-vs-yarn-cheat-sheet-8755b092e5cc#.jiz27n1hc
> I would imagine this change would be relatively straightforward, and would be 
> a massive win for the cli.
> I was thinking of creating a fork to do a PR but realised I am not totally 
> sure how to proceed on this.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13772) CLI `cordova requirements` printing out [object Object] in place of some version numbers

2018-03-03 Thread ASF GitHub Bot (JIRA)

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

ASF GitHub Bot commented on CB-13772:
-

janpio commented on issue #291: CB-13772: print version numbers correctly in 
"cordova requirements"
URL: https://github.com/apache/cordova-cli/pull/291#issuecomment-370184497
 
 
   Hey @darcymeyer, this looks like a useful change - I also noticed this 
happening. Do you know why this suddenly is a problem?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> CLI `cordova requirements` printing out [object Object] in place of some 
> version numbers
> 
>
> Key: CB-13772
> URL: https://issues.apache.org/jira/browse/CB-13772
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-ios
>Affects Versions: cordova@8.0.0
> Environment: macOS High Sierra 10.13.2
> npm v5.6.0
> node v9.4.0
> cordova CLI v8.0.0
>Reporter: Darcy Meyer
>Priority: Minor
>
>  When `cordova requirements` is executed, the output prints [object Object] 
> in place of some of the version numbers.
> e.g.
> {code}
> $ cordova requirements
> Android Studio project detected
> Requirements check results for android:
> Java JDK: installed 1.8.0
> Android SDK: installed true
> Android target: not installed 
> avdmanager: Command failed with exit code 1
> Gradle: not installed 
> Could not find an installed version of Gradle either in Android Studio,
> or on your system to install the gradle wrapper. Please include gradle 
> in your path, or install Android Studio
> Requirements check results for ios:
> Apple macOS: installed darwin
> Xcode: installed [object Object]
> ios-deploy: installed [object Object]
> CocoaPods: installed [object Object]
> {code}
> Here, the versions of the ios requirements are not displayed.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Updated] (CB-13947) Back button behavior doesn't revert to default when removing listeners

2018-03-03 Thread Cedric Reichenbach (JIRA)

 [ 
https://issues.apache.org/jira/browse/CB-13947?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Cedric Reichenbach updated CB-13947:

Description: 
When adding a `backbutton` event listener to `document`, Cordova disables 
default back button behavior (which is usually closing the app).

Later, when removing the last event listener, it should re-enable default 
behavior, but does not.
h2. Technical details

Internally, cordova seems to use a channel to handle document event listener 
registrations, which should update the override state on every listener 
addition/removal: 
[https://github.com/apache/cordova-android/blob/eddad666ff60129bf716cee61c24d3d65741640f/bin/templates/project/assets/www/cordova.js#L1664]

However, debugging indicates that `onHasSubscribersChange` is only triggered 
once, when adding a listener for the first time, but never on removal.

  was:
When adding a `backbutton` event listener to `document`, Cordova disables 
default back button behavior (which is usually closing the app).

Later, when removing the last event listener, it should re-enable default 
behavior, but does not.

## Technical background

Internally, cordova seems to use a channel to handle document event listener 
registrations, which should update the override state on every listener 
addition/removal: 
[https://github.com/apache/cordova-android/blob/eddad666ff60129bf716cee61c24d3d65741640f/bin/templates/project/assets/www/cordova.js#L1664]

However, debugging indicates that `onHasSubscribersChange` is only triggered 
once, when adding a listener for the first time, but never on removal.


> Back button behavior doesn't revert to default when removing listeners
> --
>
> Key: CB-13947
> URL: https://issues.apache.org/jira/browse/CB-13947
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-android
>Affects Versions: cordova-android-7.0.0
>Reporter: Cedric Reichenbach
>Assignee: Joe Bowser
>Priority: Major
>
> When adding a `backbutton` event listener to `document`, Cordova disables 
> default back button behavior (which is usually closing the app).
> Later, when removing the last event listener, it should re-enable default 
> behavior, but does not.
> h2. Technical details
> Internally, cordova seems to use a channel to handle document event listener 
> registrations, which should update the override state on every listener 
> addition/removal: 
> [https://github.com/apache/cordova-android/blob/eddad666ff60129bf716cee61c24d3d65741640f/bin/templates/project/assets/www/cordova.js#L1664]
> However, debugging indicates that `onHasSubscribersChange` is only triggered 
> once, when adding a listener for the first time, but never on removal.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Created] (CB-13947) Back button behavior doesn't revert to default when removing listeners

2018-03-03 Thread Cedric Reichenbach (JIRA)
Cedric Reichenbach created CB-13947:
---

 Summary: Back button behavior doesn't revert to default when 
removing listeners
 Key: CB-13947
 URL: https://issues.apache.org/jira/browse/CB-13947
 Project: Apache Cordova
  Issue Type: Bug
  Components: cordova-android
Affects Versions: cordova-android-7.0.0
Reporter: Cedric Reichenbach
Assignee: Joe Bowser


When adding a `backbutton` event listener to `document`, Cordova disables 
default back button behavior (which is usually closing the app).

Later, when removing the last event listener, it should re-enable default 
behavior, but does not.

## Technical background

Internally, cordova seems to use a channel to handle document event listener 
registrations, which should update the override state on every listener 
addition/removal: 
[https://github.com/apache/cordova-android/blob/eddad666ff60129bf716cee61c24d3d65741640f/bin/templates/project/assets/www/cordova.js#L1664]

However, debugging indicates that `onHasSubscribersChange` is only triggered 
once, when adding a listener for the first time, but never on removal.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13400) Restructure and test "Installing" instructions

2018-03-03 Thread Gandhirajan (JIRA)

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

Gandhirajan commented on CB-13400:
--

Correctness of "installing-a-development-environment.md" is taken care for mac 
as well.

> Restructure and test "Installing" instructions
> --
>
> Key: CB-13400
> URL: https://issues.apache.org/jira/browse/CB-13400
> Project: Apache Cordova
>  Issue Type: Sub-task
>  Components: cordova-docs
>Reporter: Jan Piotrowski (Sujan)
>Priority: Major
>
> When https://github.com/apache/cordova-docs/pull/745 is merged, the resulting 
> "installing-a-development-environment.md" should be a) tested for correctness 
> and b) restructure by platform instead of by tech. Makes much more sense, as 
> one will only work through the list for one platform. c) might be to add some 
> explanation which tech is used for which reason and part of the site.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org