Skip to content
Submitted over 1 year ago

QR code component

Ahmed abdulhamedā€¢130
@Ahmed-abdulhamed
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

that's my solution, any advice are welcome!

Community feedback

  • Daniel šŸ‘Øā€šŸ’»ā€¢980
    @Daniel-BilodidPosted over 1 year ago

    Hi, my congratulations you did a great job šŸŽ‰

    š’šØš¦šž š­š¢š©š¬ šŸšØš« š²šØš®š« šœšØššž šŸ› 

    šŸ”¹ You need to put a landmark in your code

    simply wrap your code with <main class='qr'> landmark instead of <div class='qr'>

    šŸ“š The <main> HTML element is intended for the main content (content) of the <body> of the document (page).

    šŸ”¹ And you need to have h1 tag in your code

    šŸ“š Every site must want at least one h1 element identifying and describing the main content of the page

    I hope it was helpful, you are great, keep up the good work šŸ‘

    Marked as helpful
    0
    Ahmed abdulhamedā€¢130
    @Ahmed-abdulhamedPosted over 1 year ago

    @Daniel-Bilodid thank you

    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