- check your work using basic recommendations here
- follow DRY (Don't Repeat Yourself)
- make sure you used flexbox in this task
- check the styles and make sure you don't write default styles (like display: block for
div
or others) - if you have 3 or more attributes, place each one on the new line (including the first one)
- font family is usually written for
html
tag and inherited by others - use alternative font-family in case the main one doesn't work like this
- we preferably use classes for styling, not tags, ids, attributes or something else
- don't increase specificity of selectors about selectors specificity
- if you want
box-sizing: border-box
property, use it with*
-selector because all elements need it so that you don't have to calculatewidth - border
- check the demo page with different screen sizes (including those bigger than 1700px)
- do not use overcomplicated media conditions (try to follow desktop-driven or mobile-driven development)
- are you sure you've polished your code? :)