darryncodes• 6,430
@darryncodes
Posted
Hi Hatwell,
Overall a pretty solid solution.
I'd highly recommend taking a look at your accessibility report and the learn more links to find out tons of information regarding semantics. In summary:
- you should only have one
<h1>per page - swap this
<div class="wrapper container d-md-flex" role="main">for<main class="wrapper container d-md-flex">you don't need the role attribute if you use a<main> - semantic elements
All the best
Marked as helpful
0
Jonel Hatwell• 585
@hatwell-jonel
Posted
@darryncodes thank you for reviewing my work, i will do your recommendations. i have a nice day.
0
