Preview card component

Community feedback

• 

  mubizzy•1,520

  @mubizzyPosted about 2 years ago

  Excellent job on this challenge! your report has a few issues though:

  1. wrap everything in your body in <main> or use semantics

  <a href=# class=ff-primary fs-100 fw-primary-regular remove the button element

  Hope it helps:).

  Marked as helpful

  1

  ---

  

  achrustowski•150

  @achrustowskiPosted about 2 years ago

  @mubizzy Thank you for a helpful feedback :) I shall edit those!

  0
• 

  Lucas 👾•104,400

  @correlucasPosted about 2 years ago

  👾Hello again @achrustowski, congratulations for your new solution!

  Again a great solution here, the cards are good in design and also fully responsive. Now something you can do is to replace the divs with <article> to improve the semantics and create a media query to give less padding and make the card content grow a bit more after max-width: 300px this will make the cards fit better the content and improve even more the responsiveness.

  👋 I hope this helps you and happy coding!

  0