Rebecca Padgett• 1,760
@bccpadge
Posted
Hello @CodeWithDion. Congratulations on completing this challenge!! 🎉
I have a few suggestions to improve your code below.
Every website must have a landmark like <main> tag
- You can wrap your content using a
<main>tag instead of<section>tag
More info 📚
I see that you used a <ul> to wrap all rating buttons but that isn't accessible for screen readers.
- To fix this issue you can you use radio buttons
- Example here:
<input type="radio" class="visually-hidden" name="rating" value="1" /> - The visually-hidden class just hides the content visually but is available for screen readers.
Here is my solution if you want to check it out - Interactive rating component
Hope this helps you and don't hesitate to reach out to me if you have any questions
Marked as helpful
0
