@Islandstone89
Posted
Good job, Neven!
A few suggestions:
HTML:
-
Every webpage needs a
<main>
that wraps all of the content, except for<header>
andfooter>
. This is vital for accessibility, as it helps screen readers identify a page's "main" section. Wrap the card in a<main>
. -
The alt text must also say where it leads(frontendmentor website).
CSS:
-
Add around
1rem
ofpadding
on thebody
, so the card doesn't touch the edges on small screens. -
Remove
overflow: hidden
, it is not needed. -
max-width
on the card should be inrem
. Change it to20rem
, which equals320px
.
Marked as helpful