@whimsicurl-creations
Posted
Mateusz, nice job with your solution. A few quick comments:
- The layout for tablets (between 680px and 975px) is unexpected because of having the three columns of content. I recently completed this challenge, but I approached it with CSS grid, so my code was a bit different.
- The use of ems seems to be okay, but your font-size and line-height for your h1 and h2 are competing. You only have 1.5em for the line height which can't allow for 2.3em of font size. This is fixed with a larger font size on larger screens. I always like using CSS Tricks for reference, and this is a link on their site about REMs and EMs: https://css-tricks.com/confused-rem-em/ It's brief, but it also has a link to another article with a more in-depth look between REM/EM/PX.