3 column preview card component

Community feedback

• 

  Ahmed El_Bald•1,020

  @Ahmed-ElbaldPosted over 2 years ago

  Hey Rick, I've some notes on your solution. I hope you find them helpful.

  • It's always a good practice to have a <main> landmark in your page (only one). You are doing that actually but it can't be a child of another element as it's a landmark so it could be a replacement to your div.container.
  • It's also a good practice to have at least an h1 in the page so you can replace your h2s with h1s
  • and finally a p tag shouldn't be inside a button tag. You can replace it with a span.

  Thank you

  1