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

Flexible social proof section

Kadheryna 200

@kadheryna

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


I'd like to receive feedback on my code :)

Community feedback

Kadheryna 200

@kadheryna

Posted

Thank you very much for your suggestions! :) I'm glad there is no as much to change as I was afraid of.

0
Rayane 1,935

@RayaneBengaoui

Posted

Hello Kadheryna,

Congrats for completing this challenge ! 🙂

I'd like to suggest :

  • Your background-size property is invalid. Instead of having background-position: top-left, bottom-rigth; , it should be background-position: left top, right bottom;. There was a typo on "right" and there is no need to add "-" between values.

  • Add min-height : 100vh to your body. It will make sure that your body covers the entire viewport, thus, your second background image can position bottom right correctly.

  • Modify the font-weight property of your <h1> and the text in the reviews to match better the design.

  • Use the max-width property on your boxes so that they don't overscale on big screens.

Overall, well done for the challenge and happy coding ! 😃

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