Skip to content
Submitted almost 4 years ago

Responsive Social Proof Section

Rizky Firman Syah•215
@rizfirsy-gh
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

If you found any problems around it please tell me, I'll fix that ASAP.

Community feedback

  • Renszo Camacho•1,615
    @RenszCamachoPosted almost 4 years ago

    Hiya 👋🏻 RizkyFirman.

    Good job mate 👏👏👏. You have done a fantastic job on this challenge 👌, I like the animation cards and it’s very responsive 💯.

    I have been digging into your code, you have used css-grid. Cool.

    You have a horizontal scroll on a small and medium screen. I would put on the body an overflow: hidden to fix that.

    You got many <div>. Just a suggestion in my humble newbie opinion. 😊

    • You could have a <main> tag involving your code. The <main> element represents the dominant content of the <body> of a document.

    • Your cards could be <article> instead of just <div>

    I hope, it helps.

    Happy coding🧑‍💻

    2
    Rizky Firman Syah•215
    @rizfirsy-ghPosted almost 4 years ago

    🖐️Hi @RenszCamacho Thanks for your concern about it. Now i know my bad. I'll dive it more.

    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