Skip to content
Submitted over 2 years ago

built with flexbox

Fai•140
@Fsanea
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

please have a look on my code and share feedback.

Community feedback

  • Rio Cantre•9,690
    @RioCantrePosted over 2 years ago

    Hello there!Nice work in completing this project. Looking at your solution, I would like to recommend the following for you...

    • Clean the whitespaces in the code
    • Import the attribution style in CSS file and remove style tag
    • Create a section rule set and add padding: 0.5rem;
    • Remove height: 100%; and border-radius: none; in .container rule set

    Above all, the design in well implemented. Hope this helps and Keep going!

    Marked as helpful
    0
    Fai•140
    @FsaneaPosted over 2 years ago

    thanks @RioCantre for the feedback

    1

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