Skip to content
Submitted over 2 years ago

Social-Proof with Flexbox

Kim•410
@Mismisty
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

I would love any and all feedback on this challenge, from the overall look to the code itself. Please let me know if there is any code that is unnecessary or could have been shorten somehow. Thanks, and have a great week!

Community feedback

  • Juan Bonilla•610
    @juanpb96Posted over 2 years ago

    Hey Kim! 👋

    Your solution looks good! 🥳

    I have some tips for you to improve your project:

    • I think you can change your div class="attribution" to footer class="attribution" this way you add a more semantic tag in your HTML 😉
    • I'm not sure why you used display: flex in your body but this is causing two columns, so the layout looks a bit weird.
    • Stars in .review are using inline styles, I consider you can include a class or play with selectors to easily add the margin that you need. Maybe something like .review > img { margin-right: 9px } could work.

    Hope this helps! 😃

    Marked as helpful
    0

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