Design comparison
Community feedback
- @garcialexcoPosted almost 2 years ago
Hello! Great job completing the challenge! π₯
There's a couple parts of your code that I might be able to help with:
-
Make sure to use semantic HTML and use div sparingly. You can change
<div class="container">
to<main class="main">
. This will make your website more accessible to screen readers and search engines! -
No need to use multiptle
h1
's, you can use headers in levels of importance. H1 being most important h2 second most etc. You can usep
. -
Also make sure to use classes instead of id's. Id's you'll only be able to apply that style once, whereas classes you can apply to different elements and they can be reused to cut down on redundant code.
Hope this helps, and keep up the great work! π 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