バレンタイン 😈• 64,190
@VCarames
Posted
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
mainelement.
More Info:📚
- The “icons/illustrations” in this component serve no other purpose than to be decorative; They add no value. There
alt tagshould be left blank and have anaria-hidden=“true”to hide them from assistive technology.
More Info:📚
https://www.w3.org/WAI/tutorials/images/
- The proper way to build this challenge is to create a
formand 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 alabelelement 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 helpful
1
Nasyr Akhmadov• 170
@darrowv
Posted
@vcarames thank you for very usefull feedback. Used all your advices to improve my work. All of that makes perfect sense.
0
バレンタイン 😈• 64,190
@VCarames
Posted
@darrowv
I am glad that I was able to help out!
Keep it up!
1
