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 using js pure

Mohamed Shaheenβ€’ 320

@mohamedshaheen26

Desktop design screenshot for the Interactive rating component coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Nelsonβ€’ 2,380

@nelsonleone

Posted

HELLO......congrats on completing this challenge....well done 🎊 🎊 🎊

I have some tips you might want to hear. Firstly, the star icon, it doesn't send a message(just for design) don't you think? So...it would be nice to set alt="" and aria-hidden="true" so that screen-readers(AT) won't stress to know what it is.

Secondly, you can perfectly position your content in the middle of the page by setting

body{
 display:flex;
 align-items:center;
 justify-content:center;
 width:100vw;
 min-height:100vh;
}

**Hope this comment was helpful and you found meaning in i ** Have fun coding

Marked as helpful

0

@VCarames

Posted

Hey there! πŸ‘‹ Here are some suggestions to help improve your code:

  • The β€œicons/illustrations” in this component are purely decorative. ⚠️ Their alt tag should be left blank and have an aria-hidden=β€œtrue” to hide them from assistive technology.

More Info: πŸ“š

[MDN Aria-Hidden] (https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Attributes/aria-hidden#description)

  • 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: <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

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

Happy Coding! πŸŽ†πŸŽŠπŸͺ…

Marked as helpful

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