Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • P
    Samuel Hernandez• 80

    @S067130H

    Posted

    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