- [STYLES] - make sure you used grid in this task
- [BEM] - Check your BEM structure using BEM-linter (
npm run lint
) and this list - [BEM] - Make sure to follow BEM naming convention for complex modifiers:
block-name--modifier-name--modifier-value
; - [BEM] - Create a separate file per each BEM block styles that have the same name as the block
- [SASS] - Make use of SASS nesting - write pseudo-class, pseudo-element selectors inside general selector. As well as media queries.
GOOD example:
&__buy-link {
display: flex;
margin-top: 20px;
&:hover {
color: blue;
}
}
BAD example:
&__buy-link {
display: flex;
margin-top: 20px;
}
&__buy-link:hover {
color: blue;
}
- [SASS] - use variables for the main values so that you'll be able to reuse them and give them descriptive names. But don't overuse them, don't create variable for the value that's used just once.
- [SASS] - Don't use SASS loops for styles that stay the same for all elements
of the group, e.g.
display
,position
font-size
etc. - [CODE STYLE] - Remember about styles properties order - (css order)
- [FUNCTIONALITY] - check the demo page with different screen sizes (including those bigger than 1700px)