Design comparison
SolutionDesign
Community feedback
- @fienmsaPosted 8 months ago
Hello @codeyeskrish!
Your solution looks great!
I have a suggestion for improvement:
Use <main> to wrap the main content instead of <div>. Tags like <div> and <span> are typical examples of non-semantic HTML elements. They serve only as content holders but give no indication as to what type of content they contain or what role that content plays on the page.
š This tag change does not impact your project visually and makes your HTML code more semantic, improving SEO optimization as well as the accessibility of your project.
I hope it helps!
Other than that, great job!
0
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