Skip to content
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

[auth] Style onboarding screen #66

Merged
merged 18 commits into from
Mar 27, 2024
Merged

Conversation

jjstnlee
Copy link
Contributor

@jjstnlee jjstnlee commented Mar 14, 2024

What's new in this PR

I added the birthday input button, but it hasn't been implemented. Implemented the Skip For Now option, and styled the screen. The bottom buttons are starting to go off the screen because everything is kinda just one block right now. I think in the figma, the information input boxes are in their own scroll so it doesn't move the bottom? I was struggling to figure out how to do that though. I also made it so that the user is able to press Update Profile if they have filled out at least one of the four input boxes.

Relevant Links

Notion Sprint Task

https://www.notion.so/calblueprint/auth-Style-onboarding-screen-b5de52766dad41d586ed21635d95fa53

Online sources

Related PRs

How to review

Signup for a new account to get to the onboarding screen.

Next steps

Tests Performed, Edge Cases

Screenshots

IMG_3427

CC: @adityapawar1

Copy link
Collaborator

is this background the same on the figma? it looks more grey

@jjstnlee
Copy link
Contributor Author

is this background the same on the figma? it looks more grey

I think I fixed it!
IMG_3451

Copy link
Collaborator

@adityapawar1 adityapawar1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks really good! couple of styling changes to make everything easier to maintain in the future.

src/app/auth/login/index.tsx Outdated Show resolved Hide resolved
src/app/auth/onboarding/index.tsx Outdated Show resolved Hide resolved
src/components/UserStringInput/styles.tsx Outdated Show resolved Hide resolved
src/app/auth/onboarding/index.tsx Outdated Show resolved Hide resolved
src/app/auth/onboarding/index.tsx Outdated Show resolved Hide resolved
src/app/auth/onboarding/index.tsx Outdated Show resolved Hide resolved
src/app/auth/onboarding/styles.tsx Outdated Show resolved Hide resolved
src/app/auth/onboarding/styles.tsx Outdated Show resolved Hide resolved
Copy link
Collaborator

@adityapawar1 adityapawar1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice job looks great! kinda hijacked your pr to fix some bugs too lol

@adityapawar1 adityapawar1 merged commit 3f724dc into main Mar 27, 2024
2 checks passed
@adityapawar1 adityapawar1 deleted the justin/style-onboarding-screen branch March 27, 2024 07:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants