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 with a lot flex box

@JimmyHoang296

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


Done but not satisfy with my solution. I think this will be hard to maintain

Community feedback

@benjoquilario

Posted

Hey @GiaVoNgu

I like your solution. There's only something need to remove and change to make looks more good

Some suggestion:

  • The wrapper should change to <main>. All element that has content are inside their respective landmark elements.
  • You must remove the width: 100vw; your wrapper class. Because this will only add the horizontal scroll bar. -It will better if you change the media queries to 768px.

Keep coding and Good luck!

Marked as helpful

1

@JimmyHoang296

Posted

thank you so much for your advice. I have just start the front end dev long journey, so many things to learn

0

@benjoquilario

Posted

@GiaVoNgu That's great just keep coding and keep learning.

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