<div dir="ltr">Would it be helpful to stand up <a href="https://github.com/CSSLint/csslint">https://github.com/CSSLint/csslint</a>
 (or some other css linter) in CI / reviewbot / mach lint? We could have
 control over which rules cause errors, warnings or are ignored 
completely. We would also have control over which directories it runs on (if we don't want to enable it across the tree in one go).<br><div><br></div><div>Cheers,</div><div>Andrew<br></div><br><div class="gmail_quote"><div dir="ltr">On Sat, Nov 17, 2018 at 8:52 PM Tim Nguyen <<a href="mailto:ntim.bugs@gmail.com">ntim.bugs@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div>Hello everyone,</div><div><br></div><div>While CSS quality hasn't been too big of a problem in mozilla-central, I have sometimes noticed the following problems:</div><div>* Explaining Firefox Desktop's CSS structure/practices to newcomers can be difficult</div><div>* Some CSS reviews take too many roundtrips, which is great in terms of CSS quality, but it is also cumbersome for both the patch author and the patch reviewer.</div><div>* Big CSS changes/refactorings can be difficult despite sometimes being essential for moving forward<br></div><div><br></div><div>With those 3 problems in mind, I started writing the following MDN document with:<br></div><div><a href="https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/CSS_Guidelines" target="_blank">https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/CSS_Guidelines</a> <br></div><div><br></div><div>It is based on the Developer Tools guide: <a href="https://docs.firefox-dev.tools/contributing/css.html" target="_blank">https://docs.firefox-dev.tools/contributing/css.html</a></div><div><br></div><div>I would love to hear your thoughts about those guidelines. The document is incomplete and I invite everyone to complete it with everything they know about CSS in mozilla-central :) Please feel free to correct anything you think is incorrect or inaccurate.<br></div><div><br></div><div>Hopefully this document will be helpful to make iterating over CSS easier and more efficient, while preserving the quality that exists now.<br></div><div><br></div><div>Cheers,</div><div>Tim<br></div><div><br></div><div><br></div></div></div></div>
_______________________________________________<br>
firefox-dev mailing list<br>
<a href="mailto:firefox-dev@mozilla.org" target="_blank">firefox-dev@mozilla.org</a><br>
<a href="https://mail.mozilla.org/listinfo/firefox-dev" rel="noreferrer" target="_blank">https://mail.mozilla.org/listinfo/firefox-dev</a><br>
</blockquote></div></div>