<div dir="ltr">Hi <span style="font-size:12.8px">Vojtech,</span><div><br></div><div>Sorry for keep you waiting, this email got lost during my PTO.</div><div><br></div><div>I couldn't agree more, PatternFly CSS needs to be modular and easier to consume. </div><div><br></div><div>I wanted start a discussion with the community about it when I have a clearer idea of a path to move forward. But since you've mention it, if there any though/solution you'd like to share?<br></div><div><br></div><div>Thanks a lot,</div><div><br></div><div>Andrés</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Aug 26, 2016 at 4:18 PM, Vojtech Szocs <span dir="ltr"><<a href="mailto:vszocs@redhat.com" target="_blank">vszocs@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Patrick,<br>
<br>
sorry for my late response, I wasn't subscribed to PF mailing list :[<br>
<span class=""><br>
> I have opened a PR to address the issue with d3 and shrinkwrap here:<br>
> <a href="https://github.com/patternfly/patternfly/pull/412" rel="noreferrer" target="_blank">https://github.com/patternfly/<wbr>patternfly/pull/412</a><br>
<br>
</span>Nice! This fix should be part of PatternFly 3.9.0 if I see correctly.<br>
<span class=""><br>
> Regarding bootstrap-treeview, we had already removed bower in our upcoming changes to treeview later this month, but I have gone ahead and added another PR since you are having issues now:<br>
> <a href="https://github.com/patternfly/patternfly-bootstrap-treeview/pull/4" rel="noreferrer" target="_blank">https://github.com/patternfly/<wbr>patternfly-bootstrap-treeview/<wbr>pull/4</a><br>
<br>
</span>Thanks! This fix should be part of patternfly-bootstrap-treeview 1.0.1.<br>
<br>
But I think you'll also need to update:<br>
<br>
  # still contains "patternfly-bootstrap-<wbr>treeview" .. "version": "1.0.0"<br>
  <a href="https://github.com/patternfly/patternfly/blob/master/npm-shrinkwrap.json" rel="noreferrer" target="_blank">https://github.com/patternfly/<wbr>patternfly/blob/master/npm-<wbr>shrinkwrap.json</a><br>
<br>
to make sure it's effectively used by PatternFly.<br>
<span class=""><br>
> The hope is to remove bower as a dependency and move to NPM, as well as update node (if you'll notice, we still build on node 0.10).<br>
<br>
</span>Big +1 on that one.<br>
<br>
(in oVirt Dashboard, we're using the latest LTS of Node which is v4)<br>
<span class=""><br>
> I am not sure regarding your question about Patternfly css - you may want to discuss with Andres when he gets back.<br>
<br>
</span>That would be great.<br>
<br>
Thanks again for stepping up and fixing those issues, much appreciated!<br>
<br>
Regards,<br>
Vojtech<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
----- Original Message -----<br>
> From: "Vojtech Szocs" <<a href="mailto:vszocs@redhat.com">vszocs@redhat.com</a>><br>
> To: "patternfly" <<a href="mailto:patternfly@redhat.com">patternfly@redhat.com</a>><br>
> Cc: "Serena Doyle" <<a href="mailto:sdoyle@redhat.com">sdoyle@redhat.com</a>>, "Oved Ourfali" <<a href="mailto:oourfali@redhat.com">oourfali@redhat.com</a>>, "Greg Sheremeta" <<a href="mailto:gshereme@redhat.com">gshereme@redhat.com</a>>,<br>
> "Ryan Barry" <<a href="mailto:rbarry@redhat.com">rbarry@redhat.com</a>>, "Martin Betak" <<a href="mailto:mbetak@redhat.com">mbetak@redhat.com</a>><br>
> Sent: Thursday, August 11, 2016 9:13:11 PM<br>
> Subject: Technical questions and ideas<br>
><br>
> Hello PatternFly team :-)<br>
><br>
> Greg (CC'ed) and me are working on oVirt Dashboard UI and we have<br>
> some technical questions regarding PatternFly reference impl. [1]<br>
><br>
> [1] <a href="https://github.com/patternfly/patternfly" rel="noreferrer" target="_blank">https://github.com/patternfly/<wbr>patternfly</a><br>
><br>
> First of all, in our project, we'd like to lock down dependencies<br>
> via `npm shrinkwrap`. Let's say our package.json contains following<br>
> dependency:<br>
><br>
>   "patternfly": "~3.8.1"<br>
><br>
> but when we run `npm shrinkwrap` it fails with an error:<br>
><br>
>   npm ERR! extraneous: d3@3.5.17<br>
>   /home/vszocs/work/releng-<wbr>tools/specs/ovirt-engine-<wbr>nodejs-modules/node_modules/<wbr>patternfly/node_modules/d3<br>
><br>
> this is because PatternFly package.json [2] is inconsistent with<br>
> npm-shrinkwrap.json [3] - "d3" is not specified in package.json.<br>
><br>
> [2] <a href="https://github.com/patternfly/patternfly/blob/v3.8.1/package.json" rel="noreferrer" target="_blank">https://github.com/patternfly/<wbr>patternfly/blob/v3.8.1/<wbr>package.json</a><br>
> [3] <a href="https://github.com/patternfly/patternfly/blob/v3.8.1/npm-shrinkwrap.json" rel="noreferrer" target="_blank">https://github.com/patternfly/<wbr>patternfly/blob/v3.8.1/npm-<wbr>shrinkwrap.json</a><br>
><br>
> Can you please fix this? Should I open ticket on GitHub or JIRA?<br>
><br>
> Second, when running `npm install --no-optional` in our project<br>
> (using above mentioned PatternFly dependency), we're getting:<br>
><br>
>   sh: bower: command not found<br>
>   npm WARN optional dep failed, continuing<br>
>   patternfly-bootstrap-treeview@<wbr>1.0.0<br>
><br>
> that's because patternfly-bootstrap-treeview does `bower install`<br>
> in its `install` script [4]. This basically assumes that Bower is<br>
> installed *prior* to executing `npm install`.<br>
><br>
> [4]<br>
> <a href="https://github.com/patternfly/patternfly-bootstrap-treeview/blob/v1.0.0/package.json#L27" rel="noreferrer" target="_blank">https://github.com/patternfly/<wbr>patternfly-bootstrap-treeview/<wbr>blob/v1.0.0/package.json#L27</a><br>
><br>
> Is there a way to fix that in patternfly-bootstrap-treeview?<br>
><br>
> In today's world, where most projects just use `npm` to manage<br>
> all kinds of dependencies, Bower should not be a requirement<br>
> (or at least not for end users of PatternFly).<br>
><br>
> Finally, I've seen Brian's e-mail on consolidating PatternFly<br>
> dependency management and I agree with the suggested approach:<br>
> use package.json to declare all of your dependencies and use<br>
> bower.json only for publishing/consuming PatternFly via Bower.<br>
><br>
> One last thing - for now, PatternFly CSS includes CSS of all<br>
> its dependencies. While seemingly convenient, this generally<br>
> goes against the principle of multiple CSS processed & applied<br>
> one after another, with a natural consequence that you need to<br>
> instruct your JavaScript build tool to exclude CSS of Bootstrap,<br>
> C3 etc. [5]<br>
><br>
> [5] <a href="https://github.com/patternfly/patternfly#using-wiredep" rel="noreferrer" target="_blank">https://github.com/patternfly/<wbr>patternfly#using-wiredep</a><br>
><br>
> Are there any plans to revisit this approach?<br>
><br>
> My specs are:<br>
><br>
>   $ node -v<br>
>   v4.4.6<br>
><br>
>   $ npm -v<br>
>   2.15.5<br>
><br>
> Thanks in advance!<br>
><br>
> Regards,<br>
> Vojtech<br>
><br>
<br>
______________________________<wbr>_________________<br>
Patternfly mailing list<br>
<a href="mailto:Patternfly@redhat.com">Patternfly@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/patternfly" rel="noreferrer" target="_blank">https://www.redhat.com/<wbr>mailman/listinfo/patternfly</a><br>
</div></div></blockquote></div><br></div>