[GitHub] metron issue #956: METRON-1474 Add normalizecss

2018-05-15 Thread sardell 
Github user sardell commented on the issue:

https://github.com/apache/metron/pull/956
  
Normalize is included with Bootstrap, so it isn’t necessary to add it 
again as an NPM dependency. You can see it being imported in the metron-alerts 
application [on this 
line](https://github.com/apache/metron/blob/master/metron-interface/metron-alerts/src/vendor.scss#L31).
 We should be doing something similar in metron-config in a new PR.


---

[GitHub] metron issue #956: METRON-1474 Add normalizecss

2018-05-15 Thread nickwallen 
Github user nickwallen commented on the issue:

https://github.com/apache/metron/pull/956
  
@sardell Thanks.

@xyztdanid4 Please let us know if you feel this PR is still relevant.  If 
not, please close the PR.  Otherwise, the PR may be closed as inactive 
according to the Metron Development Guidelines.


---

[GitHub] metron issue #956: METRON-1474 Add normalizecss

2018-05-29 Thread nickwallen 
Github user nickwallen commented on the issue:

https://github.com/apache/metron/pull/956
  
-1 as this has become an abandoned PR.  I will open a request to close this 
PR.


---