In order to improve the clarity, quality and development time it is worth considering the following principles whenever possible:
- Keep Sass Simple, which means KISS (Keep It Simple, Stupid) may override DRY (Don't Repeat Yourself) in some cases
- Single responsibility selectors
-
Airbnb CSS / Sass Styleguide is partially being followed in our code base.
-
CSS with BEM is partially being followed in our code base.
-
Most styling issues will be caught by stylelint, so before pushing your changes remember to run
grunt stylelint
to catch and fix any issues that it finds. -
Check below for the rules that are not caught by styling but should be followed.
Selectors: Selectors should follow the BEM Two Dashes style: block-name__elem-name--mod-name--mod-val
.
.button {}
.button--disabled {}
Remember to follow the Single responsibility principle.
Variables: Sass variables should be in uppercase and have a meaningful prefix.
$COLOR_RED: #e31c4b;
// Light theme
$COLOR_LIGHT_BLACK_1: rgba(0, 0, 0, 0.8);
// Dark theme
$COLOR_DARK_BLUE_1: #0b0e18;
Keep all common variables in the constants.scss file.
Flexibility: If flexibility is needed, for example for font-size, use units such as rem
, vh
, vw
, fr
, and only use px
if it's supposed to be a fixed value.
em
is typically used in padding and margin to maintain the vertical rhythm. If a user resizes the text, theem
unit will be scaled proportionately.em
size is always relative to the font-size of the element.
// For example: `span` with font-size of 14px and padding of 8px.
// The padding in `em` should be `14px/8px * 1em ~ 0.571em`.
span {
font-size: 1.4em;
padding: 0.571em;
}
px
is used to define a fixed value such as forbox-shadow
,border-radius
andborder-width
.
- Since the base font-size is set to be
10px = 1rem
, convertpx
toem
by dividing thepx
value by 10.
.balloon {
padding: 1.6em; // 16px;
}
- Or use the
@toEm($property, $px-to-be-converted, $font-size)
mixin. This is particularly helpful when you want to convert the padding/marginpx
values of an element that also has afont-size
.
// Converts padding 10px into `em` value
p {
font-size: 1.4em;
@include toEm(padding, 10px, 1.4em); // font-size in em
}
- Or any online converter tool.
The @typeface($var, $text-transform)
mixin can be used to style any text element. Simply pass in a typeface $var
name to the mixin.
The $var
name is in the format --$FONT_SIZE-$TEXT_ALIGN-$FONT_WEIGHT-$COLOR
.
Refer to typography.scss
for a list of valid font-sizes, text-align, font-weights & colors.
// Define bold red title, align to the left
h1 {
@include typeface(--title-left-bold-red);
}
The optional second argumant in the @typeface
mixin sets the text-transform
.
// Define an uppercased paragraph with color orange and font-weight 300
p {
@include typeface(--paragraph-center-light-orange, uppercase);
}
To define new typefaces, add the name and value in the $FONT_SIZES
, $FONT_WEIGHTS
or $COLORS
maps accordingly in typography.scss
file.
Mixin: use mixins wherever possible to standardize the colours used in different themes and reduce repetition.
@mixin link {
color: $COLOR_WHITE;
&:hover, &:active {
text-decoration: none;
}
}
.sidebar {
background: $COLOR_LIGHT_GRAY;
a {
@include link;
display: block;
}
}
Template versus Sass: Add SVG
s as components if you need to add classes to modify them in different themes. Otherwise you may import them in Sass, or you may import the SVG
directly to a Component
from src/images
:
import SomeIconSvg from 'Assets/SvgComponents/folder_name/some_icon.svg';
<SomeIconSvg width='15px' height='15px' />
Theme: Use declared classes such as color1-fill
to handle colouring of SVGs between different themes instead of adding extra Sass for each new image. If the existing classes don't cover what you need, create more here.
Explanations: Feel free to add comments to explain any styling that is confusing.
To do: Use TODO: ...
comments anywhere that needs consideration or attention in the future.