clipboard-landing-page

Community feedback

• 

  Eugenia•950

  @JaneMorozPosted almost 2 years ago

  Hey! Your solution to this challenge is awesome! ❤️

  The only things I've noticed:

  • You need to wrap the main content of the page into the <main> tag. It will solve all these landmark issues.
  • If you use button/link without any text (or just image/icon), I suggest to add aria-label to the <button>/<a> tag to improve accessibility. You can read more about it here. For example:

  <a href="#" aria-label="facebook"><img src="../images/icon-facebook.svg" alt="" class="duration-200 ficon"></a>
  

  • Also you might want to add cursor: pointer to the links.

  Keep it up! And good luck 🍀

  Marked as helpful

  1

  ---

  

  Davide Di Francesco•200

  @davdifrPosted almost 2 years ago

  @JaneMoroz Hey, thanks for the suggestions!

  0