Skip to content
Submitted over 3 years ago

QR code component solution

Maciej•280
@dreamspice
A solution to the QR code component challenge
Preview (opens in new tab)Code (opens in new tab)
Code

Please log in to post a comment

Log in

Community feedback

  • Maciej•280
    @dreamspice

    Hi, thank you for reply and advices :)

    I forgot to use semantic tags. Yeah, 'main' is probably the best for this case.

    I use 'h2' because i dont' want to change font-size but now i know it's bad practice.

    I will upgrade that and add media queries for 1440px width.

    All the best.

  • Anosha Ahmed•9,300
    @anoshaahmed

    Hey good job on this challenge! Your report above shows a few issues. You can fix them by

    • wrap everything in your body in <main> ... OR use semantic tags! .... you can also give role="" to the direct children of your <body> but that's a little frowned upon... Read more here
    • have at least one <h1> in your code

    Hope this helps :)

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