3 Column preview card with FlexBox

Community feedback

• 

  Fidel Lim•2,775

  @fidellimPosted about 3 years ago

  Hi Dawid,

  congrats on finishing your first FEM project! Well done! It looks great for mobile and desktop views. Just a suggestion I would like to share is that you can add hover and active states to your buttons. This will help users checking your site that those elements are clickable.

  I hope it helps :)

  Marked as helpful

  0
• 

  Marlon Passos•940

  @MarlonPassos-gitPosted about 3 years ago

  congratulations for your first challenge, some suggestions: Add this code in the body to center the card min-height: 100vh; display: grid; place-intens: center; .

  • Each of the sections could be inside a <section> tag and all the content could be inside a <main> tag, this would make the code more semantic and organized.
  • Seeing that you styled css using standard HTML tags, I would recommend you always use classes to style as this helps to avoid errors in the future

  Marked as helpful

  0