Responsive 3 Column Preview Card Component

Community feedback

• 

  darryncodes•6,370

  @darryncodesPosted almost 3 years ago

  Hi Ekrem,

  Really good solution - well done!

  Minor things you could consider:

  • adding cursor: pointer; to your button styles
  • adding max-width: 400px; (px just a guide here) margin: 0 auto; to .card when your design changes to a column for the mobile layout so it doesn't stretch all the across the viewport
  • swapping <div class="main"> <main class="main"> to clear your accessibility report. Using descriptive mark up is really important

  All the best!

  Marked as helpful

  1

  ---

  

  Ekrem İlkan•90

  @ekremilkanPosted almost 3 years ago

  @darryncodes Thank you very much Darryn. I will be more careful to write clean code. That was really good feedback for me.

  0