Interactive rating component

Community feedback

• 

  P

  Kamran Kiani•2,780

  @kaamiik

  Posted 5 months ago

  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 a h2 here. Never contain text inside div.

  ---

  • 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 and max-width should be in rem unit not px. You can read this article about it and why you should not use px as a font-size.

  11

  Marked as helpful
• 

  adam Chiha•210

  @ChihaAdam

  Posted 5 months ago

  excellent . but if you want it perfect you can add transition effect in css for smoother animation

  11
• 

  Ilham Bouktir•500

  @ilham-bouk

  Posted 5 months ago

  The page looks good and the code is organized and clear, nice job!

  10
• 

  P

  EfthymiosK•390

  @EfthymiosK

  Posted 5 months ago

  Excellent! Keep up the good work!

  10