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
- @maicondguerianPosted about 1 year ago
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 helpful0@marcelo0101Posted about 1 year ago@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 GitHubJoin 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