Design comparison
SolutionDesign
Community feedback
- @MelvinAguilarPosted almost 2 years ago
Hello there ๐. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
- Use company name as the alt attribute value for logos, the word "logo" is not necessary.
- Wrap the unordered list with navigation links (About us, What We Do, FAQ, Career, Blog, Contact Us) in a <nav> tag, this provides additional semantics for the group of links, indicating that it is a navigation section.
- In the
<header>
there is no need for any<nav>
tag since there is no list of links.
- Do the illustrations add context to the page, or are they just for decoration?; Use an empty alt attribute for decorative images. It indicates to screen readers that the image is not important and can be ignored, reducing unnecessary noise for users.
I hope you find it useful! ๐
Happy coding!
Marked as helpful1
Please log in to post a comment
Log in with GitHubJoin 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