Hey @blpeters 👋🏻
I have some suggestion to help you fix the accessibility and HTML issues.
- Instead of giving
role="main", do this, change,
<main class="">...</main>. Also, remove
<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, like
<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
@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