Insure 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:

  • If you use button/link without any text (or with only image), I suggest to add aria-label to the <button> (or <a>) tag to improve accessibility. You can read more about it here. For example:

  <a aria-label="my pinterest" href=""> <img src="images/icon-pinterest.svg" alt=""></a>
  

  • Also I have some weird hover effect when I hover over header's "View Plans" button.

  Keep it up! And good luck 🍀

  0

  ---

  

  Manoj Kumar Singh•970

  @dev-mksinghPosted almost 2 years ago

  @JaneMoroz hey thanks for the appreciation, yea i totally forgot about aria and to change to text color from view plans button in header. Gonna update it straight away. Thanks again.

  1