バレンタイン 😈• 64,190
@VCarames
Posted
Hey there! 👋 Here are some recommendations for enhancing your code:
- The logo’s
alt tag
description needs to be improved upon ⚠️; it should ALWAYS and only state the company’s name.
- This
<ul class="main-nav"
should be ⚠️ anav
element.
- This is NOT❌ how to build a responsive
nav
. You only need one navigation menu and you will CSS to style it for each layout.
- The
nav
toggle button should be built ⚠️ using abutton
element and it should have anaria-label
,aria-expanded
andaria-control
.
- The main article (The Bright Future of Web 3.0) should be wrapped ⚠️ inside an
article
element.
- The "The Bright Future of Web 3.0?" Is not an
h1
heading ❌, it is another article heading. It should be anh2
heading ✅ and there should be a visibly hiddenh1
in the site.
- The "new" section should be built using an
unordered list
element ⚠️ and have anaria-label
.
- The bottom section should be built ⚠️ using an
ordered list
and have a visually hiddenh2
heading.
- The headings in your site are being used incorrectly❌ since the
h1
heading can only be used once ⚠️ per page.
- Numbers are never ❌ headings.
- The “image” / “illustrations in this component are purely decorative⚠️. Their
alt tag
should be left blank to hide them from assistive technology.
More Info:📚
https://www.w3.org/WAI/tutorials/images/
- There is no need to have separate stylesheets ❌, as it can affect site performance. Instead have one single stylesheet ✅.
If you have any questions or need further clarification, feel free to reach out to me.
Happy Coding! 👾
Marked as helpful
0
Winstein Martins• 150
@winstein27
Posted
@vcarames This is not a helpful comment, this is a perfect comment!!! Thank you very much, you showed me a lot to improve.
1
バレンタイン 😈• 64,190
@VCarames
Posted
@winstein27
I’m glad I was able to help!
0