@VCarames
Posted
Hey there! 👋 Here are some suggestions to help improve your code:
- This LINK is a heading ⚠️ so it needs to wrapped in a
heading
element.
- The “icons/illustrations” in this component are purely decorative. ⚠️ Their
alt tag
should be left blank to hide them from assistive technology.
More Info:📚
https://www.w3.org/WAI/tutorials/images/
- Unfortunately, the HTML for the ratings needs to be rewritten as it was done incorrect ❌:
To ensure that the "rating buttons" are fully accessible 💯, they need to be built using a form
⚠️.
- Everything will be wrapped inside a
fieldset
which will have alegend
that is visually hidden using CSS. - Inside, there should be five
input radios
and eachinput
should have alabel
attached to it to make the “ratings” accessible. - The last thing you will want to include will be a
button
so users can submit their choice.
Here is how it looks like fully implemented: EXAMPLE
More Info: 📚
If you have any questions or need further clarification, you can always check out my submission and/or feel free to reach out to me.
Happy Coding! 👾
Marked as helpful
@harnettd
Posted
@vcarames
Thanks for the detailed feedback. Now that you mention it, it seems that I did indeed use javascript to re-invent the wheel and make radio buttons. I'll re-do that portion of the html and use radio buttons within a form instead.