Design comparison
Community feedback
- @peanutbutterjllyPosted over 1 year ago
hey 👋,
I have a couple of suggestions, it won't make it perfectly like the design, but will get you going in the right direction.
It looks like the site doesn't work as it gets squished down to mobile, to help with that, get your div with a class of page and put on
flex-wrap: wrap
. also, there's no need to put on aflex-direction: row
since that's the default (there's nothing wrong with doing that, just not needed).next, I'd grab your class of .img and take off the width, set
max-width: 100%
and putflex: 1 300px;
. that flex shorthand is basically saying grow/shrink normally but dont go smaller than 300px.last thing is to get your .wrapper class and add the same flex properties as your .img class
flex: 1 300px;
.one last thing to note is the social icons are missing.
Other than those small details/fixes, your solution looks great!
Marked as helpful1@ishitaraina1807Posted over 1 year agoThank you alot! I needed this feedback :) @peanutbutterjlly
1
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