Social Proof Section: mobile-first, with grid and flexbox
Design comparison
Solution retrospective
I would love to hear your criticism. Go nuts!
Community feedback
- @hardy333Posted almost 3 years ago
No criticism whatsoever, You killed it.
One small thing - on small screen size
.container
element becomes smaller than screen size and background images does not take whole space on screen, so maybe moving those background images tobody
element might be better I think .Marked as helpful2@nogyuuuPosted almost 3 years ago@hardy333 thanks so much, i will do that right away! do you think it would be best to put it directly into the body element or to create a new div wrapping the whole thing?
1@hardy333Posted almost 3 years ago@nogyuuu Both ways will be fine. But creating wrapper container as you have already done will be a little bit more robust and modular I guess, in case if you want to copy and paste html and css in other pages.
Sometimes I also using
section
element inmain
element and thissection
element takes all the viewport width space and that case I will use background properties in section element.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