Skip to content
OverviewSolutions
2
Comments
2
Luis Miguel Casañ
@lcasan

All comments

  • Luis Miguel Casañ•60
    @lcasanPosted about 1 month ago

    Hi guy, there are a few points where improvements can be made, especially around semantic HTML:

    • Landmark Elements: Including semantic landmarks such as <header>, <main>, and <footer> can significantly improve accessibility. In your case, you can wrap the main content inside a <main> tag:
    <main class="container">
</main>

    This helps assistive technologies navigate your document better.

    • Heading Hierarchy: Ensure that your heading hierarchy is logical. You use <h1> and then directly jump to <h2>, which is fine if the content warrants it, but you might consider adding a more explicit structure to define sections if there are more subsections in the page.
    • Image Alt Text: The alt text for the QR code image (alt="QR code") can be more descriptive for accessibility. It’s recommended to explain what the QR code does:
    <img
   class="image-qr-code"
   src="./assets/image-qr-code.png"
   alt="Scan this QR code to visit
/>
    Marked as helpful
    1
  • Luis Miguel Casañ•60
    @lcasanPosted about 1 month ago

    Good code

    Marked as helpful
    0