social proof with a lot flex box

Community feedback

• 

  Benjo Quilario•1,810

  @benjoquilarioPosted about 3 years ago

  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
• 

  Jimmy Hoang•570

  @JimmyHoang296Posted about 3 years ago

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

  0

  ---

  

  Benjo Quilario•1,810

  @benjoquilarioPosted about 3 years ago

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

  0