Skip to content
Submitted over 3 years ago

CSS Flexbox and grid solution for the social proof challenge

Nebiyou Ersabo•185
@NebiyouErsabo
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Social proof section
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Howdy there, this is my fifth challenge out here. I used the mobile first approach, and had a very hard time rearranging it to the desktop view. Also, I feel like my CSS code is a bit redundant and that I could have used significantly less code to achieve the same thing. Any feedbacks, suggestions, and shortcuts are very welcome! Thanks!

Community feedback

  • Ritesh_Gupta•520
    @b4n1y4Posted over 3 years ago

    Hey Nebiyou, your layout looks good... But there are a few things that I would like to point out -

    1. The background in the desktop layout is missing.
    2. The background color for the star rating container is missing.. Rest is all awesome! Keep Coding 🔥🔥🔥
    0
    Nebiyou Ersabo•185
    @NebiyouErsaboPosted over 3 years ago

    @b4n1y4 I just noticed the background behind the ratings is missing. I'll fix that rq. But for the overall desktop background, I honestly saw no difference with the background or without it. Maybe I'll just put it there anyway.

    0
    Ritesh_Gupta•520
    @b4n1y4Posted over 3 years ago

    @Nebiyou12 Yeah the desktop background is very subtle, but then, it's all about those details, isn't it?

    1

Please log in to post a comment

Log in with GitHub

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