Design comparison
Solution retrospective
Is it needed to put the image inside a div? It still confuses me a bit.
I feel like I'm learning something. It only took me about 20 minutes to build it.
Thanks!
Community feedback
- @TechNechPosted over 2 years ago
Hope you are doing good. I would suggest you these few quick fixes on your component site.
-
In your
.container
you don't need that much ofmargin
you can do is to simplify that property tomargin: 0 auto
-
Then add these properties
height: 100vh; display: flex; flex-direction: column; justify-content: center;
to yourbody
css, which will center your component. -
In
.qr-text
simplifypadding: 1rem;
-
In your
.qr-text__p
addmax-width: 25ch;
Marked as helpful1@PaliTriesToDesignPosted over 2 years ago@TechNech I really appreciate your help. Thanks!
0@TechNechPosted over 2 years ago@PaliTriesToDesign Anytime! I would love to see those changes on the live site. Make those changes and push it on github.
Happy coding :)
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