Responsive site built with FlexBox

Solution retrospective
Hey guys, this is my solution for this challenge. Thank You!
Please log in to post a comment
Log in with GitHubCommunity feedback
- @vanzasetia
Hi, Attrams! 👋
Good effort on this challenge! 👍
Here are some suggestions for improvements.
- Alternative text should not contain any words that are related to the word "image" such as logo and mockup. The semantic meaning of the
img
element is good already. - For the logo, the alternative text should be the company name, "Huddle".
- Avoid using
br
elements for presentational purposes. Read the "Accessibility concerns" part of the MDN documentation forbr
. - I recommend taking a look at "Quick tip: Using alt text properly - The A11Y Project".
- Always specify the
type
of thebutton
. In this case, set the type of it astype="button"
. It will prevent the button from behaving unexpectedly. - Anchor tag should have
href
attribute. Otherwise, it is not a link. (HTML Standard #the-a-element) - Anchor tag (and any other interactive elements) must have an accessible name. For the social media link, I recommend using
aria-label
to label each link.
I hope this helps! Happy coding!
Marked as helpful - Alternative text should not contain any words that are related to the word "image" such as logo and mockup. The semantic meaning of the
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord