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

Submitted

QR code component

@marcelo0101

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Although a simple project, if anyone can help I would like to know if using the Flexbox property in this project was the best choice.

Community feedback

maicond 230

@maicondguerian

Posted

Hey, how's it going? Pretty nice work!

You almost got it right, but forgot some things:

.container {
    display: flex;
    flex-direction: column;
    align-items: center; 
    height: 100vh; /* to set your view height at 100vh of the screen */
    justify-content: center; /* to set your content horizontally in the middle of the screen */
} 
.qrcode {
    background: hsl(0, 0%, 100%);
    /* margin-top: 40px; // remove this line */
    /* padding-top: 20px; // remove this line */
    /* padding-left: 20px; // remove this line */
    /* padding-right: 20px; // remove this line */
    /* padding-bottom: 40px; // remove this line */
    padding: 20px 20px 40px; /* you can avoid the verbosity by declaring all sides padding by passing the properties in PADDING: the first argument is about top, second: right, third: bottom, fourth: left */
    border-radius: 20px;
    width: 250px;
}

Marked as helpful

0

@marcelo0101

Posted

@maicondguerian

Hi Maicon, thank you very much for your feedback and for the tips to improve my design.

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