Frontend-mentor-rating-app

Community feedback

• 

  Fazza Razaq Amiarso•2,340

  @fazzaamiarsoPosted over 2 years ago

  Hello Aman! Nice work!

  I have some improvements on your code.

  • On accessibility, you can make your alt text clearer by changing <img src={Star} alt="icon" /> to <img src={Star} alt="star" />
  • You can place const arr=[1,2,3,4,5] above the component as it doesn't depends on any state or props. This ensures React to not re-creating the array every render.

  I hope it helps! Cheers!

  Marked as helpful

  0

  ---

  

  Aman Deep•190

  @amandeep2603Posted over 2 years ago

  @fazzaamiarso thanks for your feedback 😊.

  0
• 

  Kamasah-Dickson•5,590

  @Kamasah-DicksonPosted over 2 years ago

  Great job there but.

  • Your solution is a bit small as compared to the actual design. The texts too are also small.

  • You can reduce the card's background color's opacity to mach the designs.

  • Try to fix the accessibility issues reported as well.

  • You can add a transition to the final rating section when a user clicks the button.

  Besides good job there👍 Have a nice day and a happy coding👍💻

  Marked as helpful

  0

  ---

  

  Aman Deep•190

  @amandeep2603Posted over 2 years ago

  @Kamasah-Dickson Thanks for your feedback 😊

  0