3 column component built with flex-box and media queries

Community feedback

• 

  P

  Fluffy Kas•7,715

  @FluffyKasPosted about 3 years ago

  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:

  1. For your component-container you could use a main instead of a div, and the attribution could go inside a footer.

  2. You're missing the border-radius on the component.

  3. 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 :)

  0