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

Sematic HTML & CSS (Flexbox layout)

Jonathan Goboliโ€ข 90

@jonexist

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Lucas ๐Ÿ‘พโ€ข 104,420

@correlucas

Posted

๐Ÿ‘พ Hello Jonathan, congratulations for your solution!

Your solution seems fine, I liked that you've used width: min-content; to the .wrapper, smart choice!

Pay attention only for the card padding that's not correct, this is due the heigh you've set to the .wrapper, note that you don't need to set height for the wrapper, the height will come from the padding/margins of the elements inside the wrapper, instead of that just apply an single padding for all the wrapper having an equal space between all the elements, see the fixes below:

    display: flex;
    justify-content: center;
    align-items: center;
    flex-direction: column;
    background: hsl(0, 0%, 100%);
    width: min-content;
    /* height: 500px; */
    padding: 16px;
    border-radius: 20px;
}

Hope it helps you, happy coding!

0

Jonathan Goboliโ€ข 90

@jonexist

Posted

@correlucas

The height should be in the container dang I messed up๐Ÿ˜…. But I will try this single padding that you mention thank you so much for feedback this will help me improve as a beginner๐Ÿ˜„

1
Lucas ๐Ÿ‘พโ€ข 104,420

@correlucas

Posted

@jonexist You're welcome Jonathan, happy to hear that was useful!

Marked as helpful

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