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

QR-Component

JuanferGG 150

@JuanferGG

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


QR-Component

Community feedback

EthanAmato 130

@EthanAmato

Posted

Hey there! First of all I want to say fantastic job! It looks absolutely great! If you want to address some of the warnings that the accessibility report gave you, check out the following:

"Document should have one main landmark" + "All page content should be contained by landmarks" Essentially, it's telling you to make use of semantic HTML (documentation here) to make the site easier to read for screen readers. Essentially, replace the <div class="container"> with a <main/> tag and you should be all set!

2 "Page should contain a level-one heading"

I believe this error is coming from a lack of an H1 tag in your code. Try replacing the H3 tag with an H1 (and swap the associated styling)

Overall, fantastic job and I hope you enjoy your journey making websites :)

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