Design comparison
SolutionDesign
Community feedback
- @correlucasPosted about 2 years ago
๐พHello again Omar, congratulations for your new solution!
You can start to fix this solution by adding
<main>
to replace the div wrapping all the content. And also manage both images inside the<picture>
tag and use the html to code to set when the images should change setting the devicemax-width
depending of the device (phone / computer) Hereโs a guide about how to usepicture
. This way you've a clean code and remove the div.Add also
object-fit: cover
to make the image auto-crop when resizing inside the column:img { object-fit: cover; }
๐ 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