Conversation
code-and-cats
left a comment
code-and-cats
left a comment
There was a problem hiding this comment.
Nice work. The survey works well, looks nice and meets the stretch goals. The code is easy to follow and makes sense. I really like the solutions in the code for progressing the stages!
Well done!
|
|
||
| return ( | ||
| <main> | ||
| {formStage > 0 && <ProgressBar progress={percent} />} |
There was a problem hiding this comment.
I really like this percent function, havent seen it before!
| <h1>Mentorship Sign Up</h1> | ||
| </div> | ||
| <div className="image-box"> | ||
| <picture> |
There was a problem hiding this comment.
is there a reason for calling 2 different pictures instead of resizing?
| </button> | ||
| )} | ||
| <form onSubmit={handleSubmit} required> | ||
| {formStage === 0 && ( |
There was a problem hiding this comment.
very nice solution on progressing the stages!
| Restart Form | ||
| </button> | ||
| )} | ||
| <form onSubmit={handleSubmit} required> |
There was a problem hiding this comment.
So cool that required works well! But is the "required" here needed as well as the required on the different inputs (for example row 105)? Are they both needed? If yes, what are they doing?
| )} | ||
| {formStage === 3 && ( | ||
| <div> | ||
| <h1>What time commitment can I make? /Hours a month/ *</h1> |
There was a problem hiding this comment.
super minor nitpicky thing, the user has been asked stuff as a "you" before this, and now an "I". Might be nice to be consistent :)
2 participants
https://legendary-froyo-ff20a9.netlify.app/