Flexbox, Responsive design, Mobile first, bootstrap 5

Community feedback

• 

  darryncodes•6,370

  @darryncodesPosted almost 3 years ago

  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-jonelPosted almost 3 years ago

  @darryncodes thank you for reviewing my work, i will do your recommendations. i have a nice day.

  0