Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Social Proof Section using SCSS, Flexbox and Grid

@mohamadadithya

Desktop design screenshot for the Social proof section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Your feedback is useful for me :).

Community feedback

P
David Turner 4,110

@brodiewebdt

Posted

I would lower your breakpoint between mobile and desktop, the review text length is too long. I would put it at about 768px.

You also need to add some spacing between the copy and the reviews at desktop resolution. The reviews start to cover the text at about 1100px.

Other than that this looks very good. Nice job.

Marked as helpful

0

@mohamadadithya

Posted

@brodiewebdt Ok, thanks for your feedback/advice, it's very useful and I will fix my solution later ;).

0
Peter 170

@PeterJan285

Posted

Wow, you solved layout between "copy" and "reviews" with grid so nicely, im struggling to do it with flexbox. I noticed body background image does not sit well on fullscreen

0

@mohamadadithya

Posted

@PeterJan285 Thanks for your feedback, yes it was a bit tricky to set the background, but I'll fix that later ;).

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join 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