-
Notifications
You must be signed in to change notification settings - Fork 0
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
Onboarding Part 1 #134
Onboarding Part 1 #134
Conversation
✅ Deploy Preview for motherhoodbeyondbars ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Everything looks really great besides a few small coding things and the things we Slacked about, I really appreciate the attention to detail and implementing the form to keep the onboarding data organized!
</span> | ||
</div> | ||
<div className="flex flex-col p-7 gap-[1.125rem]"> | ||
<h1 className="text-primary-text text-2xl font-bold font-opensans"> |
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.
The titles for Desktop views should be centered on the page using sm:
web/components/atoms/ProgressBar.tsx
Outdated
@@ -5,7 +5,7 @@ interface Props { | |||
|
|||
export default function ProgressBar({ progress }: Props) { | |||
return ( | |||
<div className="h-[6px] relative rounded-full bg-empty-gray"> | |||
<div className="h-[6px] w-[107px] lg:w-[250px] relative rounded-full bg-empty-gray"> |
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.
The progress bar looks SO GOOD especially the animation between pages :D The widths should be relative precents though for desktop vs mobile using sm
instead of set sizes so that it adjusts better to different screen sizes
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.
Looks great except for these small details!
</div> | ||
<CheckboxText label="I agree to the Liability Waiver" /> | ||
<div className="mt-3"> | ||
<TextInput label="Signature" /> |
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.
This page is missing error checking 😅 The user should agree to the waiver and sign it with the date before moving on
…OfGood/motherhood-beyond-bars into 122-isaiah-onboarding-pt1
…ond-bars into 122-isaiah-onboarding-pt1
What does this Pull Request change?
Creates to form for onboarding waiver, shipping address, and contact preference. Also creates a lot of the general infrastructure for onboarding. Closes #122
Important Changes
Important Notes
I'm not sure how we want to handle the waiver in the form, so could use some guidance on that. Also still working on completing the form sorry about that 😅