<div dir="ltr"><div>For anyone who has opinions on this, I filed:</div><div><a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1508369">https://bugzilla.mozilla.org/show_bug.cgi?id=1508369</a></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Nov 19, 2018 at 1:53 PM Dan Mosedale <<a href="mailto:dmosedale@mozilla.com">dmosedale@mozilla.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Automating CSS style curation with a linter sounds like an excellent<br>
plan.  CSSlint hasn't seen much action in a while; from what I can<br>
tell, much of the interesting CSS tooling stuff is going on with<br>
PostCSS-based stuff, which is written on top of NodeJS.  I haven't<br>
deeply investigated the various linting stuff (yet!), but<br>
<a href="https://github.com/stylelint/stylelint" rel="noreferrer" target="_blank">https://github.com/stylelint/stylelint</a> is one popular example.  If we<br>
can wait a bit more until we've started landing live node_modules in<br>
the tree, this might be a good way forward.  Activity-Stream is<br>
already tentatively planning land and use PostCSS in the tree, though<br>
that's likely to happen in the new year.<br>
<br>
Dan<br>
<br>
<br>
<br>
Am Mo., 19. Nov. 2018 um 10:39 Uhr schrieb Andrew Halberstadt<br>
<<a href="mailto:ahal@mozilla.com" target="_blank">ahal@mozilla.com</a>>:<br>
><br>
> Would it be helpful to stand up <a href="https://github.com/CSSLint/csslint" rel="noreferrer" target="_blank">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>
><br>
> Cheers,<br>
> Andrew<br>
><br>
> On Sat, Nov 17, 2018 at 8:52 PM Tim Nguyen <<a href="mailto:ntim.bugs@gmail.com" target="_blank">ntim.bugs@gmail.com</a>> wrote:<br>
>><br>
>> Hello everyone,<br>
>><br>
>> While CSS quality hasn't been too big of a problem in mozilla-central, I have sometimes noticed the following problems:<br>
>> * Explaining Firefox Desktop's CSS structure/practices to newcomers can be difficult<br>
>> * 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.<br>
>> * Big CSS changes/refactorings can be difficult despite sometimes being essential for moving forward<br>
>><br>
>> With those 3 problems in mind, I started writing the following MDN document with:<br>
>> <a href="https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/CSS_Guidelines" rel="noreferrer" target="_blank">https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/CSS_Guidelines</a><br>
>><br>
>> It is based on the Developer Tools guide: <a href="https://docs.firefox-dev.tools/contributing/css.html" rel="noreferrer" target="_blank">https://docs.firefox-dev.tools/contributing/css.html</a><br>
>><br>
>> 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>
>><br>
>> Hopefully this document will be helpful to make iterating over CSS easier and more efficient, while preserving the quality that exists now.<br>
>><br>
>> Cheers,<br>
>> Tim<br>
>><br>
>><br>
>> _______________________________________________<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>
><br>
> _______________________________________________<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>