Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Interactive rating component solution

Kyle Kasprzykβ€’ 260

@kylekasprzyk

Desktop design screenshot for the Interactive rating component coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hello πŸ‘‹,

This is my solution for the interactive rating component challenge. πŸ‘¨β€πŸ’»

This was my first time completing a challenge with JS. I learned the importance of script placement within HTML.

Thank you for taking the time to view my solution, and for any feedback provided!

Community feedback

@VCarames

Posted

Hey there! πŸ‘‹ Here is some feedback to further improve your code:

  • The HTML for the ratings needs to be rewritten as it was done incorrectly ❌:

To ensure that the "rating buttons" are fully accessible πŸ’―, they need to be built using a form ⚠️.

  1. Everything will be wrapped inside a fieldset which will have a legend that is visually hidden using CSS.
  2. Inside, there should be five input radios and each input should have a label attached to it to make the β€œratings” accessible.
  3. The last thing you will want to include will be a button so users can submit their choice.

More Info: πŸ“š

MDN: Form Element

MDN: <input type="radio">

MDN: Field Set Element

  • Once the top is implemented , for your JS, the eventListener should be on the form ⚠️ as a submit.

More Info:πŸ“š

Click vs. Submit EventListeners

  • Currently, you "thank you" content is not being announced ⚠️ to screen readers. So you need to wrap it it in another div and give it a aria-live="polite" (it should only have this and nothing more).
  • The h4 in your "thank you" content should be a p instead, since it is not a heading.

If you have any questions or need further clarification, feel free to reach out to me.

Happy Coding! πŸ‘Ύ

Marked as helpful

0

Kyle Kasprzykβ€’ 260

@kylekasprzyk

Posted

@vcarames thank you for all of the help!

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join 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