interactive-rating-component-main

Community feedback

• 

  Tux3er•780

  @Tux3er-IsmaPosted over 1 year ago

  Hi Renan

  Tux3er has reviewed your code 🐧

  Congrats 🎉, it looks nice, but I will give you some tips for your next project 💡:

  HTML 🦴:

  • In my opinion is not a good practice to add onclick in each element. You can do it with JS with the addEventListener()

  • Also you can put tabindex instead to use it after in css with :focus

  CSS 🎨:

  • As I said before you can use :focus in the numbers. Is like click in JS

  • You can also add a transition to all the hover states

  JS 🟡:

  • The function mark you can get it with only HTML and CSS (as I said before)

  • You have to put a return button to return to the homepage

  That's all, have a nice coding ⌨️!!!!

  Marked as helpful

  1

  ---

  

  Renan Do Vale•110

  @Renan-do-valePosted over 1 year ago

  @Tux3er-Isma Thanks for the advices. Initially, my goal was to use the addEventListener(), but I couldn't make it work, so I had to come up with a workaround to make it work.

  0