Skip to content
Submitted about 3 years ago

QR code component

sass/scss
LakiDev•80
@lakiidev
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

  • EHO•265
    @hermannleboss

    Hello nice work. You have accessibility issues. Your page content should be contained by landmarks. The landmark is something like that

     <body>
     <div role="banner">This is the header</div>
     <div role="navigation">This is the nav</div>
     <div role="main">This is the main</div>
     <div role="contentinfo">This is the footer</div>
 </body>

    The role attribute.

    It defines all sections of your content and is useful for screen readers.

    For your code you can encapsulate all the content in the body markup and add role main or role content.

    Marked as helpful

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