Huddle landing page with alternating feature blocks with Html and Css

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 3 years ago

  👋Hi Edy! My name is Vanza!

  I have some feedback on this solution:

  • The telephone number and the email address should be wrapped inside anchor tag that has tel and mailto
  • For the alt text inside this img tag, you can just say Huddle since on screen reader it will say image of Huddle.

  <img id="imagen-logo" src="./images/logo.svg" alt="Huddle logo" />
  

  • For all styling I recommend to use class only rather than id, since it's considered as a best practice.
  • For the illustration images or any decorative images you can just leave the alt="" empty like this (don't remove it).
  • I don't think an icon should be inside a p tag. My recommendation is to use span in this case.

  <p><ion-icon name="call"></ion-icon></p>
  

  • Try to use rem or em units rather than % unit, since it can create the value is relative on the width screen which means on very small screen it can be too small or in very large screen it can be too big.

  That's it! Hopefully this is helpful!

  1