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

Responsive Social Proof Section

@firmansyahrizky

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


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

Community feedback

Renszo Camachoβ€’ 1,615

@RenszCamacho

Posted

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

@firmansyahrizky

Posted

πŸ–οΈ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
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