1332624 - [tracking] CSS linting errors

Status: RESOLVED FIXED

(www.mozilla.org :: Pages & Content, defect)

Description

[Alex Gibson [:agibson]]

We recently added Stylelint [1] to our CI on bedrock [2] to help us automatically pick out CSS errors and styling preferences in pull requests.

[1] http://stylelint.io/
[2] https://github.com/mozilla/bedrock/pull/4567

There are currently a lot of 'warning' class linting errors in the existing Less and Sass files that make up the pages on mozorg.

This is a tracking bug for fixing those linting errors. These dependent bugs will be classed as "good first bugs" for contributors to take on.

Comment 1

[Rachel]

looking at the current css (as well as fixes to the linting errors), i've noticed that there's no standardisation of whether single or double quotes should be used. 

Should this be added to the stylelint? if yes,what would be the preference? single or double?

Comment 2

[Alex Gibson [:agibson]]

(In reply to Rachel from comment #1)
> looking at the current css (as well as fixes to the linting errors), i've
> noticed that there's no standardisation of whether single or double quotes
> should be used. 
> 
> Should this be added to the stylelint? if yes,what would be the preference?
> single or double?

Good suggestion - feel free to open a pull request to add the rule! (you can also file a bug for it if you wish, blocking this tracking bug)

Comment 3

[Alex Gibson [:agibson]]

Preference would be single quotes looking at what we use most often.

Comment 4

[Alex Gibson [:agibson]]

All dependent bugs have been resolved, thanks to everyone who contributed :)

We removed all warning-level Stylelint settings in https://github.com/mozilla/bedrock/pull/4669, so going forward builds will error if any rules are broken. We're going to see how much noise this creates, and can always re-evaluate if it becomes too troublesome .