dewslyse• 3,025
@dewslyse
Posted
Hello Marialba! Congrats on completing your fourth challenge. Some things to note:
- The page isn't quite fully responsive. It's not optimized for mid-sized devices. You could set the max-width on the current media query higher. Personally, I would lose the
height: 50%on the.containersince it's constraining the height of the card. I would then space the card elements with either padding/margin. E.g.:.section a { margin-top: 50px; }. I would also probably lose thewidth: 50%and then set amax-widthrather (maybe 1110px). - The icons are decorative and as such should have
alt="". - Remember to also wrap the content between the
<body>in amainfor improved accessibility<main class="container">.
All the best
Marked as helpful
1
Marialba Córdova• 60
@Marialba16
Posted
Thanks for your comment @dewslyse, I'll take it into account. I will check my code and correct. Thanks!
0
