Design comparison
Solution retrospective
Not so far :)
Community feedback
- @correlucasPosted about 2 years ago
๐พHello Borcho, congratulations for your new solution!
1.Add
min-height: 100vh;
anddisplay: flex; align-items: center; justify-content: center;
to make your component have the proper vertical alignment:body { min-height: 100vh; padding: 0; margin: 0; font-family: Inter, sans-serif; background: var(--body-bg); display: flex; align-items: center; justify-content: center; }
2.You need to fix the
container size
that is a bit off, this challenge uses the standard container size that ismax-width: 1110px
.3.To make the image have a better fit inside of it, make the component image responsive with
display: block
andmax-width: 100%
(this makes the image fit the column/div size) and respect the component size while it scales down. To make it crop while scaling useobject-fit: cover
.img { display: block; object-fit: cover; max-width: 100%; }
โ๏ธ I hope this helps you and happy coding!
Marked as helpful0
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