Introduction
All CSS in a codebase should be consistent and give the impression it was written by a sole developer. It should be understandable and simple to work with.
Syntax
Consistency is the most important factor when contributing to existing CSS. Generally the following structure should be used:
- 4 space indentation (no tabs)
- 1 declaration per line
- A space after the colon separating the property and value pair
- Lowercase strings
- Always apply a semicolon to the last declaration
- Each selector should have it’s own line
- Include a space before the opening brace of a declaration block
- Closing braces of a declaration block should be on a separate line
- Quote attribute selectors with single quotes
- Separate rules by new lines
// Do not format like this
.selector, .another-selector[attr=""]{
border:0; line-height:inherit;
width:auto}
// Format like this
.selector,
[attr=''] {
border: 0;
line-height: inherit;
width: auto;
}
###Strings Prefer single quotes over double quotes. Once a standard has been set, stick with it.
.selector {
background-image: ('images/super-awesome-image.jpg');
font-family: 'Helvetica';
}
###Declaration Order Alphabetical ordering is preferred. Although a very debatable subject, this style suits an unknown team size because of the following:
- Consistency is the most important factor
- Everyone knows the alphabet
- Not all css contributors can distinguish between box-model properties, positioning, visual and typography
- No learning curve required for custom ordering preferences
// Define declarations alphabetically
.selector {
border: 1px solid rgba(0, 0, 0, .15);
border-radius: 3px;
box-shadow: inset 0px 1px 3px rgba(0, 0, 0, .1);
color: #fff;
width: auto;
}
###Units Values of 0 should be unitless. Favour rem over px. Favour seconds over milliseconds. Use relative units whenever possible such as on line-height. Avoid unnecessary leading zero’s.
// Do not do this
.selector {
background-color: rgba(255, 255, 255, 0.4);
border: 0px;
font-size: 16px;
line-height: 24px;
}
// Use relative units
.selector {
background-color: rgba(255, 255, 255, .4);
border: 0;
font-size: 1rem;
line-height: 1.5;
}
If fallback values are required use a mixin when using Sass, otherwise add a fallback in a browser specific file.
###Colours Colours should be written as lowercase hexademical values and in shorthand when possible. If transparency is required favour rgba. Sass can parse hex values for rgba so colour variables and map values can always be stored in hex.
// Do not do this
color: #FF6600;
// Do this
color: #f60;
###Shorthand Group properties into shorthand use when appropriate to avoid adding multiple properties:
// Avoid this
.selector {
margin-bottom: 1rem;
margin-left: 2rem;
margin-right: 2rem;
margin-top: 1rem;
}
// Do this instead
.selector {
margin: 1rem 2rem;
}
Ensure that shorthand is not used repeatedly when it is not needed. If only one value needs changing do not use shorthand:
// Don't do this
body {
font: normal 16px/1.5 'Helvetica';
}
.another-selector {
font: normal 2rem/1.5 'Helvetica';
}
// Do this
body {
font: normal 16px/1.5 'Helvetica';
}
.another-selector {
font-size: 2rem;
}
###Important rule Never use this. Rework selectors instead. You will reap the benefits in the long term.
###Vendor Prefixes Prefixes should never be used in Sass files and should be added via a build tool to ensure only required prefixes are added. IE specific vendor prefixes should be added to an IE only file or through the use of an IE mixin.
Selectors
###Naming Class and variable names should be lowercase and hyphen separated rather than camelcase or snakecase.
// Do not do this
.formList {}
.form_list {}
// Use hyphenated lowercase
.form-list {}
###ID’s Never use ID’s for selectors. If you are forced to reference an ID then use an attribute selector instead. This has the same specificity as a class and so can be overwritten without increasing specificity levels.
// Never use ID selectors
#anidselector {}
// Prefer an ID attribute if it is absolutely necessary
[id='anidselector'] {}
###Qualifying Never tag-qualify. By qualifying selectors specificity becomes a problem. Always rework your markup and class structure to address inheritance issues instead of qualifying.
// Instead of this
input[type='checkbox'] {}
// Use this
[type='checkbox'] {}
###JS-Classes Any classname prefixed with js- should not have any presentational properties attached. These are used as references for JS Classes. Instead is- prefixed classes should be used as state classes.
// Don't do this
.js-navigation {
display: block;
}
// Use this
.is-active {
&.navigation {
display: block;
}
}
This allows for the removal of JS without interfering with presentation.
Box Model
A document should use 1 box-model type consistently. Once a box-sizing value has been set there should not be any other elements using an alternate value.
// Do not do this
html,
body {
box-sizing: border-box;
}
.element {
box-sizing: content-box;
}
###Float Containment Clearing should be performed through a pseudo-class clearfix. Overflow clearing should not be used.
// Don't do this
.container {
overflow: auto;
}
.child-element,
.child-element-2 {
float: left;
width: 50%;
}
// Do this
.container {
&:after {
clear: both;
content: '';
display: table;
}
}
.child-element,
.child-element-2 {
float: left;
width: 50%;
}