Hey @blpeters 👋🏻
I have some suggestion to help you fix the accessibility and HTML issues.
- Instead of giving
body
thisrole="main"
, do this, change,<div class="">...</div>
to<main class="">...</main>
. Also, removerole='contentinfo'
because<footer>
tag is enough for the screen reader to determine the content in it. - Next, instead of
<section>
, I suggest using regular<div>
for a couple of reasons. First, when you use a<section>
you have to have a heading, likeh1-h6
. Next,<section>
is for bigger parts of layout, such as, contact us about us, image gallery, etc.
These will fix the most of the accessibility issues. Don't forget to generate a new repot once you fix the issues.
I hope this was helpful 👨🏻💻 overall, you did a great job, well done. Cheers 👾
Marked as helpful
@blpeters
Posted
@kens-visuals That took care of all the accessibility issues - thanks so much! Still learning how to properly use semantic html and your explanation makes sense