@Bayoumi-dev
Posted
Hey Pedro,
Congratulations on completing this challenge... You have accessibility issues
that need to fix.
Document should have one main landmark
, Contain the component with<main>
.
<main>
<div class="grid">
//...
</div >
</main>
-
Heading levels should only increase by one
, Useh2
instead ofh3
...Ordered headings
make it easier to navigate and understand when using assistive technologies. -
I suggest you put the
Why Us
content into thelist item
to add moresemantics
to your project.
<ul class="why-us-list">
<li>Tutorials by industry experts</li>
<li>Peer & expert code review</li>
<li>Coding exercises</li>
<li>Access to our GitHub repos</li>
<li>Community forum</li>
<li>Flashcard decks</li>
<li>New videos every week</li>
</ul>
I hope this is helpful to you... Keep codingπ