Design comparison
SolutionDesign
Community feedback
- @correlucasPosted over 2 years ago
๐พHello Rhenrique, congratulations for your solution!
Pay attention, because you've add too much unnecessary info inside the container, you have a quick fix using
grid
withgrid-template-column: 1fr 1fr
andgrid-template-column: 1fr
for the mobile version, see the code fixes below:".caja { background-color: var(--White); border-radius: 5px; display: grid; grid-template-columns: 1fr 1fr; /* grid-template-columns: repeat(1,320px); */ /* grid-template-rows: repeat(2,360px); */ } .caja { background-color: var(--White); border-radius: 5px; /* display: grid; */ /* grid-template-columns: repeat(2,265px); */ /* grid-template-rows: repeat(1,400px); */ }
Hope it helps, happy coding!
Marked as helpful1 - @shaswatsingh19Posted over 2 years ago
Hi, Congratulations on completing this project
Everything is looking fine somethings I have noticed
- For smaller screens you should remove the margin of 130px from the image
- Background size should be cover and position should be center and it's look amazing
Marked as helpful1
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