Failed at the javascript part, do not know how to automatically deselect one rating when a new one is pushed.
Justine U.
@justine-135All comments
- @TheOneEnzoSubmitted over 1 year ago@justine-135Posted over 1 year ago
It would be efficient if you loop all the buttons from their parent container on your script instead of manually selecting all individual buttons.
At deselecting a button, you can set an initial boolean variable to each buttons then check if their value is true or not. If true ? highlight the button else : remove highlight.
I suggest putting your main-container inside a parent container, and styling it to full height of the screen, and adding flex to your main container to align it to the center of the screen.
I hope my feedback is helpful!
Marked as helpful1 - @BircanAliSubmitted over 1 year ago
The logical part was challange for me. I still work on effects of the numbers any suggestion about effect are welcome.
@justine-135Posted over 1 year agoFeedback:
- The component is not responsive on mobile device.
- The execution of event is confusing. I don't know when the output will occur. Sometimes it does, and it doesn't.
- Validation message from the month input is the same as the year input.
What you can study:
- Responsive design: learn about structuring your HTML better first so you can apply responsive styling easily.
- Javascript DOM events.
I hope my comment is helpful. Do more challenge and post more solutions to build you skills!
0