@Islandstone89
Posted
Hey there, great job! Only a few minor suggestions to improve it even further...
HTML:
-
The alt text also needs to say where it leads (frontendmentor.io).
-
.attribution
should be a<footer>
. -
Footer text needs to be wrapped in a
<p>
.
CSS:
-
It's good practice to include a CSS Reset at the top.
-
As fas as I can see, you don't need the
aspect-ratio
on the image. Also, adddisplay: block
and replacewidth
withmax-width: 100%
.
@Crimson-repere
Posted
@Islandstone89 thank you for your suggestion