Submitted over 3 years ago
Social Proof Section Master Challenge with HTML, CSS (FLEXBOX)
@Nishkarsh01
Design comparison
SolutionDesign
Solution retrospective
Need a code review!
Community feedback
- @tedikoPosted over 3 years ago
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@Nishkarsh01Posted over 3 years ago@tediko thank you for the valuable feedback, notes have been taken.
0
Please log in to post a comment
Log in with GitHubJoin 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