Social Proof Section

Community feedback

• 

  Samson Sham•140

  @samsonshamPosted over 2 years ago

  Hello Kally!👋 You did a great job on this challenge🎉 The responsive design is nice, especially the tablet view. I like it very much.👍

  For the scrolling, I found it caused by the margin in .container that you are applying 5% on margin top and bottom. I am afraid if it would be too much for the total page height. As percentage is relative unit and it would be a bit hard to calculate and control if you put it to height. You may reset it to 0 then try some other ways.

  • A quick and dirty way is to add overflow: hidden; to body so no more scrolling (remember to undo it in mobile view otherwise you can't scroll in mobile).
  • If you only want some head space you might only add padding-top or margin-top to avoid unwanted bottom space.
  • If you want to make your container align centre vertically, you can set display: flex; align-items:center; in body.

  Hope it would help! 💪

  1

  ---

  

  Karlisha•170

  @al-lattePosted over 2 years ago

  @samsonsham Thank you! 😊. I'll give the solutions a try 😀

  0
• 

  Kamasah-Dickson•5,590

  @Kamasah-DicksonPosted over 2 years ago

  Your solution looks great on mobile besides I didn't experience any side scrolling on mobile. But if you do it maybe the cause of a width you applied on something. Good job, keep coding👍

  1

  ---

  

  Karlisha•170

  @al-lattePosted over 2 years ago

  @Kamasah-Dickson Thank you! 😊

  0