@DavidMorgade
Posted
Hello Jeremy, congrats on finishing the challenge, you did a great job!
Probably would add some cursor: pointers
to your rating buttons and also add it to the submit button, so it's easy for the user to understand that those are clickable buttons!
You could also add some tranistions to those rating buttons and the submit button just by adding transition: all 0.5s ease;
, it will have a smoother effect and it will look your component even more modern!.
For the JS is ok to at a class to select that specific item, you could also have selected all of your li
items and then looping through them in your for loop, you could have selected the buttons using the closest()
method. But adding an extra class is also acceptable!
Hope my feedback helps you! great job!