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

mobile first using html and css

Degoke 210

@Degoke

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


Your feedback would be appreciated

Community feedback

@MiltonChung

Posted

Hey Degoke!

Nice start! I like your approach of using grid-template-areas! Overall, you did pretty well. Some things to consider:

  • the top left texts are left aligned and the big texts are supposed to be bigger
  • the rating bars on the right should end and be aligned with the bottom testimonial box
  • when I stretch the browser horizontally across 2 monitors, everything stretches out too much, so considering adding a max-width to your container/wrapper
  • your body tag doesn't take up the full view height, so the background image doesn't cover all the white areas. Try adding min-height: 100vh in your body tag.

Great job! :)

2

Degoke 210

@Degoke

Posted

@MiltonChung Thank you, i'll make the changes

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