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

Social proof section - Using HTML y Sass

Kevin Moralesβ€’ 185

@kefer16

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


Hello! This is already my third challenge, any advice or recommendation of good practices when coding is welcome :)

Community feedback

Bonreyβ€’ 1,130

@Bonrey

Posted

Hello, Kevin! Wow, your site looks really awesome. Congrats on this one! πŸ₯³ I especially liked the animation in the beginning. 😊

Just a couple of things I'd suggest:

  1. Get rid of your validation error by changing your section.rateds to div. I think that'd be more semantically correct, since every section tag should contain a heading, and here there's none.
  2. Well, that's just my opinion, but I'd probably avoid stacking stars on top of each other on smaller screens. In one row, they look somewhat better ⭐⭐⭐⭐⭐

Everything else is smooth and responsive. Great work and good luck with your coding! πŸ‘¨β€πŸ’»

1

Kevin Moralesβ€’ 185

@kefer16

Posted

@Bonrey Hi, thank you very much. 😊 I took your suggestions, I already fixed the HTML semantics error (I changed the "section" tag to "div") and I modified the stars so they wouldn't stack on top of each other. Thank you so much for helping me improve my coding. πŸ’»

0
Bonreyβ€’ 1,130

@Bonrey

Posted

@kefer16 Great! I'm glad to hear that πŸ˜€

1

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