Social Proof Section Master Solution

Community feedback

• 

  Elaine•11,400

  @elaineleungPosted over 2 years ago

  Hi Samantha, well done on this challenge, and I believe you did great also. Your code is clean and easy to read, and I like how you used descriptive names for your classes. I just got two quick suggestions:

  • Right now there's some empty space at the bottom of the page at desktop view; usually we don't notice this when we use a background color but since you're using an image, that empty space is quite noticeable. To fix that, just add min-height: 100vh to your body tag and the background image should cover the whole page.

  • Instead of using fixed widths (e.g., width: 340px) try responsive properties such as min-width, max-width; I like to use width: min() and clamp() also, which I find super useful when you want to have both min and max widths in one property.

  Hope to see more great work!

  Marked as helpful

  0

  ---

  

  Samantha•30

  @samanthascarcellaPosted over 2 years ago

  @elaineleung Thank you so much! That was incredibly helpful. I will have to start playing with the min() and clamp() and update my code accordingly:)

  1