6
3%
master: 3%

Ran 27 Mar 2017 07:10PM UTC

Files 369

Run time 31s

Badge

Embed ▾

Committed 26 Mar 2017 11:57PM UTC coverage: 91.45% (+0.3%) from 91.123%

Job # TEST_SUITE=backend

Build Type

push

travis-ci

Committed by

showell

Commit Message

css: Enforce one selector per line.

While it's sometimes nice to put a few selectors on the same line,
it is generally better to have a consistent way of formatting our
selectors, and most of our code up until now lists them vertically.
This change fixes the linter to enforce one selector per line, and
it cleans up the places in the CSS where we had multiple selectors
on the same line.

The advantages of one-per-line are as followers:
    * cleaner diffs
    * easier to see when multiple areas of the app may have the
      same format
    * less likely to go over 80 cols
    * makes it more clear where we have deep nesting in the
      individual selectors
    * makes it easier for our linting tools to enforce
      whitespace violations

This also fixed an old bug where we had ".landing_page h2, h4", which
sets "h4" styles outside of the landing page.

Run Details

29937 of 32736 relevant lines covered (91.45%)

0.91 hits per line

zulip / zulip / 536 / 6
3%
master: 3%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 536.6 (TEST_SUITE=backend)

zulip / zulip / 536 / 6 3% master: 3%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 536.6 (TEST_SUITE=backend)

zulip / zulip / 536 / 6
3%
master: 3%

README BADGES
x