<div dir="ltr"><div dir="ltr"><div>Hey Matthew,</div><div><br></div><div>Thanks for showing me this CSS Tips page! I didn't see it before.</div><div><br></div><div>I put it on MDN in line with other mozilla-central coding style articles, see the right column of: <a href="https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Source_Code">https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Source_Code</a></div><div><br></div><div>Cheers,</div><div>Tim<br></div><div><br></div><div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Nov 19, 2018 at 11:06 PM Matthew N. <<a href="mailto:MattN@mozilla.com">MattN@mozilla.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 class="gmail_default" style="font-family:arial,helvetica,sans-serif">Thanks for working on this Tim. We may want to consolidate your new page with the existing related page at <a href="https://wiki.mozilla.org/Firefox/CSS_Tips" target="_blank">https://wiki.mozilla.org/Firefox/CSS_Tips</a></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">By the way, the document is probably better on the wiki as MDN is trying to focus on web tech documentation, not be Mozilla/Firefox-specific.</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Thanks,</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Matthew<br></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Nov 19, 2018 at 3:17 PM Jared Wein <<a href="mailto:jaws@mozilla.com" target="_blank">jaws@mozilla.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>Thanks Tim, this is great and much needed. Having some automated linting is really helpful (both are great, proactive and reactive).</div><div><br></div><div>From reading your document I learned about inset-inline-start and inline-start, etc. These can help remove some extra RTL handling that we do today, though our CSS will become even harder for a regular user to contribute to (I don't think that's a good enough reason to not push forward).</div><div><br></div><div>Cheers,</div><div>Jared<br></div></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Nov 19, 2018 at 2:51 PM Andrew Halberstadt <<a href="mailto:ahal@mozilla.com" target="_blank">ahal@mozilla.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>For anyone who has opinions on this, I filed:</div><div><a href="https://bugzilla.mozilla.org/show_bug.cgi?id=1508369" target="_blank">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" target="_blank">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>
_______________________________________________<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>
_______________________________________________<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>
</blockquote></div>