Design comparison
SolutionDesign
Solution retrospective
I was here in hurry. I hope I did it well. If you have any comments, feel free to write to me so that I can make corrections. Thank you again, until next time! 👌
Community feedback
- @danielmrz-devPosted 10 months ago
Hello @Velingd!
Your solution looks great!
I have a suggestion for improvement:
- For semantic reasons, use
main
to wrap the main content instead of adiv
. 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!
1 - For semantic reasons, use
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