You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cordova.apache.org by "Sergey Grebnov (JIRA)" <ji...@apache.org> on 2014/07/08 12:56:05 UTC

[jira] [Comment Edited] (CB-6874) Consolidate tag additions into 1 ItemGroup

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

Sergey Grebnov edited comment on CB-6874 at 7/8/14 10:55 AM:
-------------------------------------------------------------

Propose to modify addSourceFile method to work with arrays as well (type detection) so we can pass all references at once and don't create separate groups for each www/ reference.

https://github.com/apache/cordova-lib/blob/master/cordova-lib/src/util/windows/jsproj.js#L126

Alternatively we can add separate method addSourceFiles


was (Author: sgrebnov):
Propose to modify addSourceFile method to work with arrays as well (type detection) so we can pass all references in once and don't create separate groups for each reference.

https://github.com/apache/cordova-lib/blob/master/cordova-lib/src/util/windows/jsproj.js#L126

Alternatively we can add separate method addSourceFiles

> Consolidate <Content> tag additions into 1 ItemGroup
> ----------------------------------------------------
>
>                 Key: CB-6874
>                 URL: https://issues.apache.org/jira/browse/CB-6874
>             Project: Apache Cordova
>          Issue Type: Improvement
>          Components: CLI, Plugman, Windows 8, WP8
>            Reporter: Jesse MacFadyen
>            Assignee: Sergey Grebnov
>
> Currently when we add/remove project items from the www project folder in our xml parser, we create a new ItemGroup for each item.
> A better approach would be to combine all the www/ items into one ItemGroup.
> See CB-6699 for more discussion on this, and the rejected wildcard solution.



--
This message was sent by Atlassian JIRA
(v6.2#6252)