Huddle landing page with a single section (Responsive)

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  Hello, Ahmed Abdelgawad! 👋

  Good job on this challenge! 👏

  Some recommendations from me.

  • I would recommend keeping the bg-mobile.svg for mobile users. I think without the background image, it makes the site looks empty. 😅
  • Links must have text content. In this case, for the social media links, I would recommend using aria-label to give text content to each link. For example, Twitter link would be aria-label="Twitter".
  • Alternative text for images should not contain any words that are related to image (e.g. picture, photo, logo, icon, graphic, avatar, etc). It's already an image element so the screen reader will pronounce it as an image.
  • The mock-up image is a decorative image. So, I would suggest leaving the alt="" empty. This way, screenreader users won't have to listen to the decorative element.

  Hope this helps. 😊

  Marked as helpful

  0

  ---

  

  Ahmed Abdelgawad•315

  @ahmedAgawadPosted over 2 years ago

  @vanzasetia Thanks a lot Vanza 😊 , Your suggestions helped me a lot this one of the most useful feedbacks i got on frontend mentor.

  happy coding 😄.

  0

  

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  @ahmedAgawad You're welcome, Ahmed! Happy coding too! 😄

  0