@mattstuddert
Posted
Hey Simeon, sorry for my delay in looking at this! You've done a really good job. Overall your HTML structure is great. I'd just recommend a few changes on the accessibility side:
- Your
alt
text for the logoimg
says "the logo". This doesn't actually give any context as to what the image really is. If it's a company logo, I'd recommend just putting the company name as the alt text. So it would just say "Huddle". - The same goes for your
alt
text for the illustration of the mockups where you've currently got the text "huddle photo". I'd go for something more descriptive like "Illustration of a platform dashboard on desktop and mobile screens". When writing alt text, put yourself in the position of someone who cannot see the image themselves and then ask yourself if the text you've written actually represents the image. - You've added
aria-label="Register Button"
to thebutton
element that has the text of "Register" inside it. In this instance, thearia-label
isn't needed as the screen reader would read out that the element is abutton
and the content is "Register" by default.
Let me know if you have any questions. Keep up the great work!