Design comparison
Solution retrospective
Which areas of your code are you unsure of?
- The div sizes and paddings
Community feedback
- @DeanogitPosted 11 months ago
Hello @ChristianDriz,
This solution looks great!
I took a look at the
style.css
file to check something, you might be interested in, when I check the mobile layout in chrome dev tools on my desktop, the top of the.left
div is not visible.I changed the
height: 100dvh;
tomin-height: 100dvh
and this looks like its working as intended.I also noted the div with the class
.attribution
is positioned absolutely, when the viewport is less than 641px wide, this floats ontop of the.right
div.In order to fix this, its possible to
- remove the
position: absolute;
from the.attribution
and - add
flex-direction: column
to the body.
I hope this helps, your solution is already looking great, just some minor tweaks
Marked as helpful0 - remove the
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