@AgataLiberska
Posted
Hi @joanpujol, that looks nearly pixel-perfect, woww! Very impressive :D
I personally just use rems instead of ems or px, I just calculate the size off the base font size and don't have to worry about the font size on the parent element at all (I don't have much experience but I copied it from Jessica Chan and I'm going with it! Also recommend her videos where she works on some projects from here)
Font sizes is something I would look at in this project - because for some of the text the font size comes to 9.1px and for some to 9.8px which is very hard to read. Your base font is set to 14px and I wouldn't go below that at all. (And if you're using rems, that's super simple - it's 1rem or higher :))
One more thing - clickable elements, like buttons - they could really use some hover styles to indicate that they're buttons. And the links in the nav, both at the top of the page and in the footer should really be <a>
tags, even though it's a one page project :)
Hope this helps :)