huddle-landing-page- responsive design

Community feedback

• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted almost 2 years ago

  Hello there 👋. Good job on completing the challenge !

  I have some suggestions about your code that might interest you.

  • The alt text for the logo of the Hubble, "chat-icon", is not an appropriate description of the image because it does not accurately describe the content of the image. The logo is the representation of the company and the "chat-icon" doesn't provide any information about the company or the image. You can use the name of the company as the alt text.

  • You should use the aria-label attribute to describe the links if they don't have visible text, this will help screen reader users to understand the purpose of the link.

  e.g.

  <a href="#" aria-label="Facebook"><i class="fa-brands fa-facebook"></i></a>
  

  I hope you find it useful! 😄 Above all, the solution you submitted is great!

  Happy coding!

  Marked as helpful

  1

  ---

  

  mutasem•250

  @wadmortadaPosted almost 2 years ago

  @MelvinAguilar I love you suggestions for both logo's alt and link's attribute, thank you for feedback.

  1