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
Request path contains unescaped characters
Not Found
Not Found
Not Found
Not Found

Submitted

Social Proof Section Master Challenge with HTML, CSS (FLEXBOX)

@Nishkarsh01

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


Need a code review!

Community feedback

P
tediko 6,580

@tediko

Posted

Hello, Nishkarsh Dubb! 👋

Congrats on finishing another challenge! Here are few suggestions from me:

  • Read about semantic. Semantic elements lead to more consistent code, they are easier to read and improve accessibility.
  • You repeat your HTML code with stars images which is unnecessary. Find the way to use only one star image for a stars. You can do it easily by using background-repeat approach in your css. Instead of these images use one div and apply this property to it.
  • .text-primary shouldn't be a heading element. Use normal div instead.

Good luck with that, have fun coding! 💪

2

@Nishkarsh01

Posted

@tediko thank you for the valuable feedback, notes have been taken.

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