Rating component for Frontend Mentor with Tailwind CSS
Design comparison
Solution retrospective
Do you consider that hiding a radio button with opacity to have HTML5 form alerts is correct? and If no, What will you recommend instead? Do you think the colors match the theme given? In which parts? Do you consider that the component adapts well in mobile devices? That's all, and finally, thank you!!
Community feedback
- @Kamasah-DicksonPosted over 2 years ago
Your solution looks great and responsive. To help you with the accessibility issues.
-
Whenever you use a section tag there should be an h tag after it. Don't use sections if there is no h at that level, you can use a div instead.
-
Don't just mix your h tags they should decrease by one level.
-
There should be at least a semantic tag in your solution. Eg <main> <aside> e.tc. but in this solution wrap the card in a <main> semantic tag or change the parent div into main .
Besides good job there👍 Happy coding👍💻
Marked as helpful2 -
Please log in to post a comment
Log in with GitHubJoin 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