3-column preview card component

Community feedback

• 

  Adriano•38,270

  @AdrianoEscarabotePosted about 1 month ago

  Hi Yosef Hayim, how are you doing? I really loved the outcome of your project, but I have a few suggestions that I think might be helpful:

  I noticed that you use more than one h1, this is not a good practice since it is recommended to have only one h1 per page to inform the main title of the page! so remove all h1 and put h2

  I noticed that you used a button in which case the best option would be an a, because in my head when a person clicks on a button written Learn More, he is not confirming a form, or something like, it will be redirected to another page, to Learn More about!

  to solve this problem do this:

  <a href="/">Learn More</a>

  The rest is excellent.

  I hope you find it useful. 👍

  0
• 

  P

  Lo-Deck•2,790

  @Lo-DeckPosted about 1 month ago

  Hi well done for this challenge,

  For this <div class="first-container"> you can use <section class="first-container">

  Use a div when there is no other choices, you can use section, article...

  It's better to use em or rem instead of px. FreeCodeCamp. FreeCodeCamp.

  Hope to be helpful.

  0