@A-amon
Posted
Great work! It's responsive too~😀
Here are a few suggestions:
- Place all your
<section>
inside<main>
- As stated in the Accessibility Report, seems like you're jumping around the heading tags order quite a bit. To fix this, increase your heading tag by only 1 as the nesting goes deeper. For example:
<section>
<h2>Title</h2>
<div>
<h3>Sub-Title</h3>
...
<div>
<h4>Sub-sub-title</h4>
...
</div>
</div>
</section>
^ This is just my understanding though so do read up on it for more accurate explanation 😀
- Your .nav__hamburger 🍔 should be a
<button>
instead of a<div>
. - The
<a>
tags that wrap around the icons and logo should have a text describing where it links to (For screen reader users) 🔊. And since this text is hidden visually 👻, I usually would usearia-label
attribute on the<a>
tag. But you can also solve this by addingsr-only
class (and do its CSS) to hide the text Reference. - Oh, and seems like there are no hover effects on the links (only the 'Contact' button has it) 🤔
Marked as helpful