@bene-volent
Posted
Hi there, congrats on completing the solution. The first time is always hard.
Few recommendations if you won't mind:
- As the following is a component (assuming a part of larger component), consider enclosing it in a div or an article in itself for increasing modularity.
- As you are using button[type=submit], you should consider using a form where the ratings are actually radio buttons with each one storing a rating value and using attached label to show the values instead of button as buttons are only for pressing and not having a [Pressed or Active or Checked] state. check out my solution if you need an example. I also learnt it from someone else.
- Try validating the form, for example, I can just press submit and it allows me to submit. Try validating whether a rating is already selected or not. If not don't show second screen and tell the user that a rating should be selected. This can be done by disabling by lightening the color of submit button or having an animation on it.
Try them one at a time if you have time, and re evaluate your solution. Everyone learns one step at a time
Marked as helpful
@JuHnr
Posted
Thank you for your recommendations @bene-volent ! My first idea was to use radio buttons and a form, but I didn't find a solution without a PHP script so I changed that (except the submit button). I will check out your solution and try again :)
@bene-volent
Posted
@JuHnr you don't need a php script for this. If I am right about what you are saying, you don't actually need to submit the form, you can handle the onsubmit event in the form and use the preventDefault() on the submit event to stop it from reloading/actualy try to submit. Once you do that, everything is in your control and you can do whatever you want with the html. 😊 Good luck
Marked as helpful
