We recently added Stylelint  to our CI on bedrock  to help us automatically pick out CSS errors and styling preferences in pull requests.  http://stylelint.io/  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.
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?
(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)
Preference would be single quotes looking at what we use most often.
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 .
Status: NEW → RESOLVED
Last Resolved: 11 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.