Design comparison
SolutionDesign
Community feedback
- @VCaramesPosted almost 2 years ago
Hey there! 👋 Here are some suggestions to help improve your code:
- The
nav
should be wrapped inside aheader
element for improved semantics.
- The "logo" is not part of the
nav
so it should be outside in theheader
.
- The logo is not not decorative, Its
alt tag
should not be blank. The description should state the company’s name.
- Your HTML should have a
main
element that will assist in identifying the main content of your site. In this, case themain
element would be wrapping everything after theheader
and before thefooter
.
- The company info in the
footer
should be wrapped inside anaddress
element.
- The
footer
company links should be wrapped inside anav
and be a single list, not two.
If you have any questions or need further clarification, feel free to reach out to me.
Happy Coding!🎄🎁
Marked as helpful1@GGKFPosted almost 2 years ago@vcarames I don't know how to thank you for these tips, thank you very much🙏🥰
1 - The
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