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

Solution for "Social-proof-section" challenge using html/css

@YousefKhalid-iq

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


any feedback would be appreciated, thanks :)

Community feedback

Dušan Lukić 1,660

@dusanlukic404

Posted

Hi Yousef, I have a few suggestions for you. This challenge should be a landing page so it is unnecessary to have a vertical scrollbar. You have to avoid it with reducing padding that you added to containers. Also, in my opinion much easier way to do this challenge and have a responsive landing page is to have one main container and inside of them top and bottom section. You wrap everything in main element.

On mobile devices it looks very nice so well done!

I suggest you to take a look on my solution :)

Marked as helpful

1

@YousefKhalid-iq

Posted

@dusanlukic404 Thanks for the feedback and sorry for being late on the response it's probably the time zone difference. so i changed the page so that it's no longer scrollable in desktop mode, and for the container design i decided to leave it as is for this challenge but I'll apply the method you mentioned in any small pages I make in the future. Thanks again for the feedback ;)

0
Dušan Lukić 1,660

@dusanlukic404

Posted

@YousefKhalid-iq nice. Keep grinding! 😁

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