-
Notifications
You must be signed in to change notification settings - Fork 435
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
project survey week 6 #441
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You made a well-functioning survey with a logical structure. I think you did your components reusable, especially the Freetext-one. The design is clear and it looks very professional.
Good job on the project!
|
||
{step === 5 && ( | ||
<div> | ||
<FreeText |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the design of the FreeText-component, good reusability
<> | ||
<h2>How often per week do you purchase something to drink in a cafe?</h2> | ||
<form className="radio-buttons" id="radio-buttons"> | ||
{purchases.map((group) => ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like this, will be easier to add more options later and the radio buttons will still work
No description provided.