@FluffyKas
Posted
Hey there, your solution looks very good! I really like the tablet view you made, it's a nice touch! ^^ There's a few small things you could improve:
-
For your component-container you could use a
main
instead of adiv
, and the attribution could go inside afooter
. -
You're missing the border-radius on the component.
-
I'd probably leave the car logo alt-texts empty, they're decorative images and don't really add much to the content. On a side note, if you do want to give something an alt-text, you can skip using words like image, picture, logo, etc. You're using a semantic <img> tag, that alone is descriptive enough for users to know it's an image :)