Interactive rating component

Solution retrospective
Any comments are helpful
Please log in to post a comment
Log in with GitHubCommunity feedback
- P@kaamiik
Some notes:
- This
<div class="title">How did we do?</div>
is completely wrong. You should use proper semantic tags that suitable for a text. Here this question seems like ah2
here. Never contain text insidediv
.
- This challenge html structure is really important. You have a fieldset with a legend and a group of 5 radio buttons. These numbers are not button they are radio buttons and need proper id, name, value and label too.
- Try to use a proper CSS reset at the start of your CSS style. Andy Bell and Josh Comeau both have a good one. You can simply search on the internet to find them.
- Your
font-size
andmax-width
should be inrem
unit notpx
. You can read this article about it and why you should not usepx
as a font-size.
Marked as helpful - This
- @ChihaAdam
excellent . but if you want it perfect you can add transition effect in css for smoother animation
- @ilham-bouk
The page looks good and the code is organized and clear, nice job!
- P@EfthymiosK
Excellent! Keep up the good work!
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