Skip to content
OverviewSolutions
6
Comments
1
P
Samuel Hernandez
@S067130H

All comments

  • Samuel Hernandez•80
    @S067130HPosted 9 months ago

    Hey there! Great job on the solution, it looks great!

    One thing I noticed though was your use of the <main> HTML element. This is what is known as a semantic element and is used to encompass the main section of your website. I noticed you attached a container class to it. Since there can ever only be one main element tag inside a document, applying classes should not be necessary. Instead, you might want to consider a structure like so:

    <body>
  <main>
      <div class="container">
         [...]
      </div>
  </main>
</body>

    This helps with accessibility when it comes to things like screen readers. Other than that, great job! :)

    Marked as helpful
    0