Design comparison
SolutionDesign
Solution retrospective
Completed another challenge. Looking for comments and suggestions from the community members.
Community feedback
- @MelvinAguilarPosted 12 months ago
Hello there ๐. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
- For star icons used decoratively, like in this challenge, it's best to keep the alt attribute empty. This prevents screen readers from repetitively reading the same description, which can be bothersome for users with visual impairments. So, you can simplify the code by leaving the alt attributes for the star icons empty.
- Avoid using the
<div>
tag just for text.<div>
is a generic container used for grouping content and applying styles, but it doesn't convey any semantic meaning. Similarly, avoid using<span>
for entire blocks of text,<span>
is an inline HTML element used to apply styles to specific parts of text without changing the structure or meaning, but this tag doesn't provide the same semantic value as other tags like paragraphs.
- Remember to change the alt attribute for images to the person's name.
I hope you find it useful! ๐
Happy coding!
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