Submitted almost 2 years ago
Interactive Rating Component (SASS and JS)
@LysitheaDarkKnight
Design comparison
SolutionDesign
Solution retrospective
Hello, I have finished this exercise!
Getting the rating selection down was a huge challenge. Part of it is because I group the ratings as ul
instead of radio
buttons. If someone can elaborote on which method to use (ul
or radio
buttons) that would be helpful!
Thank you for reading! Feedback would be appreciated!
Community feedback
- @VCaramesPosted almost 2 years ago
Hey there! 👋 Here are some suggestions to help improve your code:
- To not only improve your HTML code but to identify the main content of you page, you will want to wrap your entire component inside the
main
element.
More Info:📚
- The `headers element is not needed for this challenge. Since this is a component.
- Along with the blank
alt tag
, you also want to include thearia-hidden=“true”
to your “ images/icons” to fully remove them from assistive technology.
More Info:📚
https://www.w3.org/WAI/tutorials/images/
- The proper way to build this challenge is to create a
form
and inside of it, the “rating buttons” should be built using aninput radio
(wrap all of them inside afieldset
), since this will prevent additional buttons from being active at the same time, and it should have alabel
element attached to it for accessibility.
More Info:📚
MDN <fieldset>: The Field Set element
If you have any questions or need further clarification, feel free to reach out to me.
Happy Coding!🎄🎁
Marked as helpful0 - To not only improve your HTML code but to identify the main content of you page, you will want to wrap your entire component inside 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