Submitted over 3 years ago
I used pure HTML and CSS with the approach of mobile first design.
@DeepPatel6128
Design comparison
SolutionDesign
Solution retrospective
Please try to and help me optimize the code.
Community feedback
- @tedikoPosted over 3 years ago
Hello, DEEP PATEL! 👋 Congrats on finishing another challenge! Your solution looks very good and also responds well. Here's my few tips:
- Change the
alt
attributes for the avatar images, as they don't add any extra context for screen reader users. Since your images are decorative youralt
text should be provided empty (alt="") so that they can be ignored by assistive technologies. - You repeat your HTML code with stars images which is unnecessary. Find the way to use only one star image for a
span
with stars images. You can do it easily by usingbackground-repeat
approach in your css. Your approach is also correct, so you don't have to change that. Just wanted to show you other way to do it.
Good luck with that, have fun coding! 💪
0 - Change the
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