Design comparison
SolutionDesign
Community feedback
- @kaamiikPosted 3 days ago
- headings are not part of the header and mostly we use logo of the site or nav items that are repeating contents inside the header. Your
h1
tag is not a repeating contents. It should be inside yourmain
.
- No need to add
aria-hidden
forimg
.
- Try to use a proper CSS reset at the start of your CSS style. Andy Bell and Josh Comeau both have a good one. You can simply search on the internet to find them.
Both your HTML structure and CSS are good. Keep on the great work!
Marked as helpful0 - headings are not part of the header and mostly we use logo of the site or nav items that are repeating contents inside the header. Your
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