
Please log in to post a comment
Log in with GitHubCommunity feedback
- @pheight-89
🎉 Great job on your first project with maintaining the project design. 🎉
👩🎓Some things to keep in mind:
All the content of your page should be located within landmark elements such as
<main>, <header>, <footer>, <nav>, <aside>
as they provide semantic meaning to your page's structure. This is important to keep in mind because:-
Accessibility for visually impaired users to allow screen readers to navigate your page more efficiently.
-
Search Engine Optimization to allow search engines to better understand the content of your website and page structure, which could lead to higher search rankings.
-
Maintainability for developers as it is immediately clear what purpose each section of the page serves.
Looking at your code, you have commented
!--Main--
and created a<div>
called main. I would suggest using the<main>
tag rather than the<div class="main">
. Additionally heading tags<h1></h1>, <h2></h2>, <h3></h3>
etc. should be used in order and styled with CSS as necessary. For instance you should use the<h1></h1>
instead of the<h3></h3>
to adhere to semantic HTML.Solid foundation, continue practicing and keep semantic HTML in mind when structuring your webpages.
Marked as helpful -
Join 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