Design comparison
SolutionDesign
Solution retrospective
Any feedback will be appreciated :)
Community feedback
- @tedikoPosted over 3 years ago
Hello, Bub! 👋
Congrats on finishing another challenge. A few suggestions from me:
- You repeat your HTML code with stars images which is unnecessary. Find the way to use only one star image for a
.block
. You can do it easily by usingbackground-repeat
approach in your css. - Read about semantic. Semantic elements lead to more consistent code, they are easier to read and improve accessibility.
- Add some
padding
to.opinions
element on mobile so it doesn't stick to screen borders.
Good luck with that, have fun coding! 💪
0@voxx1Posted over 3 years ago@tediko Thank you for your reply! I added some semantics, and margin for mobile version, so it doesn't stick to borders anymore :)
0 - You repeat your HTML code with stars images which is unnecessary. Find the way to use only one star image for a
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