Skip to content
Submitted almost 2 years ago

QR code page with HTML, CSS, Flexbox

Giorgi Gogilashvili•30
@Notoriousgg
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...QR code component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Everything is perfect

Community feedback

  • Sinisa Vukmirovic•1,690
    @SinisaVukmirovicPosted almost 2 years ago

    Hello!

    No, everything is not perfect! It might be "perfect looking" but the code is not perfect.

    You didn't use semantic elements in HTML. Error in your report are about that.

    This "Document should have one main landmark" means you are missing <main> semantic element in your html. Landmark just means element.

    This "All page content should be contained by landmarks" means all non-semantic elements should be inside of semantic elements, and not directly inside of <body> element.

    More about semantic elements

    Fix those errors to make it perfect. Good luck!

    0

Please log in to post a comment

Log in with GitHub

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