Huddle Landing Page

Community feedback

• 

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  Hi, Pradeep Saini! 👋

  Great work on this challenge! Your solution looks pretty good! 😀

  Some suggestions from me about accessibility.

  • Create a custom :focus-visible styling for any interactive elements (button, links, input, textarea). This will let people who use a keyboard to navigate the site know where they are currently on the page.
  • Remove the word "logo" from the alternative text of the logo. The screenreader would pronounce it as an image already.
  • Links must have text content in order to be accessible by screenreader users. So, I recommend using aria-label attribute to give some text content to all social media links.
  • Lastly, I suggest fixing all the issues that have been reported.

  I hope this helps! Keep up the good work! 👍

  Marked as helpful

  1

  ---

  

  Pradeep Saini•990

  @pradeeps4iniPosted over 2 years ago

  @vanzasetia Thanks for all the feedbacks. Issues reported were regarding the alt text i provided to the <a> links. But i think aria-label is what i should've used.

  0

  

  Vanza Setia•27,775

  @vanzasetiaPosted over 2 years ago

  @pradeeps4ini You are welcome! 👍

  You can't have an alt attribute on an anchor tag. It's invalid HTML. 🙂

  0