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

Submitted

Frontend Mentor QR code component Darioriverosp

@darioriverosp

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

P

@DarrickFauvel

Posted

Hi @darioriverosp, 👋

Congratulations on posting your first solution! 🥳

To fix the accessibility issues in the accessibility report (above ⬆️), you need to use at least one semantic html element. I would suggest wrapping your .card element with a <main></main> element. Also, you can do the same with <footer></footer> around the .attribution element.

I would also recommend moving the .container to inside these "landmarks". Try something like this:

<main>
    <div class="container">
      <div class="card"></div>
    </div>
</main>

<footer>
    <div class="container">
      <div class="attribution"></div>
    </div>
</footer>

I hope this is helpful. 😊

Marked as helpful

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord