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
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

Kundai 20

@KundaiTS

Posted

Great job Ariha! According to accessibility guidelines, having a single <h1> (level-one heading) on a page is the best practice. The <h1> is generally used to represent the main heading or title of the page, and it should logically come before other headings like <h2>, <h3>, etc. In this case this should be an h1 "<h2>Improve your front-end skills by building projects</h2>"

1

@Preshy2016

Posted

Hey, Nicely done. Some advice instead of hard-coding padding and margins in multiple places, consider using CSS variables or centralizing styles to make the design more scalable and maintainable.

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