Design comparison
Community feedback
- @correlucasPosted about 2 years ago
๐พHello Aczos, congratulations for your new solution!
Your solution its fine, but need some adjustments, I've some tips for you:
Add rounded borders to you component with
border-radius: 12px
adding this also to the image.To manage the text content its better you use a single div to keep all them and add a padding to this div to make the same spacing for all the content and add
text-align: left
to make all the text aligned left, if you remove the other divs and clean your code you'll see how much is easier to deal with the content.You need to fix the
container size
that is a bit off, this challenge uses the standard container size that ismax-width: 1110px
. To make the image have a better fit inside of it, make the component image responsive withdisplay: 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!
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