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

#bem#sass/scss

@hassanidris

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


I would love to have your feedback guys.

Community feedback

P
AK 6,700

@skyv26

Posted

Hi! Hassen, well done, nice solution, I checked your design responsiveness too.

Everything seems so pretty but I have suggestions for you.

  1. Your design is not small as compared to the preview design, I mean look at your desktop design, there is lot of space at both sides (left and right). Try to make your design in such a way that it look more better and cover full width.

  2. In mobile I saw that your last card touching the bottom viewport and I would suggest you to add margin-bottom: 2rem; in below code (I added)

.wrapper {
    display: flex;
    justify-content: center;
    align-items: center;
    margin: 0 auto;
    margin-bottom: 2rem;
    max-width: 1100px;
}

I hope you understand.

Best Of Luck

Marked as helpful

0

@hassanidris

Posted

@skyv26 Wow... Thanks for your feedback that's was so valuable. I had adjusted my files.

1

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