Skip to content

[DRAFT] "User" styling#70

Merged
3mcd merged 8 commits into
mainfrom
gs/user-styles
Sep 8, 2023
Merged

[DRAFT] "User" styling#70
3mcd merged 8 commits into
mainfrom
gs/user-styles

Conversation

@gabestein
Copy link
Copy Markdown
Member

@gabestein gabestein commented Sep 7, 2023

Issue(s) Resolved

Resolves #48 by giving some styling to login, logout, and forgot password forms.

This is good enough for now, and I think I'd like to push it so I can upgrade to using the shadcn styles @3mcd is introducing in #77.

Test Plan

Visit /login, /signup and /forgot

Screenshots (if applicable)

Optional

Notes/Context/Gotchas

Supporting Docs

@render
Copy link
Copy Markdown

render Bot commented Sep 7, 2023

Copy link
Copy Markdown
Contributor

@kalilsn kalilsn left a comment

Choose a reason for hiding this comment

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

Looks a lot better!

@gabestein
Copy link
Copy Markdown
Member Author

gabestein commented Sep 7, 2023

Thanks! Not quite done yet...sorry, I forgot to call it a draft. I don't think I'm blocking anyone, so I'm gonna keep it open for now. LMK if you want me to merge.

@gabestein gabestein changed the title "User" styling [DRAFT] "User" styling Sep 7, 2023
@isTravis isTravis temporarily deployed to gs/user-styles - integration-evaluations PR #70 September 8, 2023 18:51 — with Render Destroyed
@isTravis isTravis temporarily deployed to gs/user-styles - integration-submissions PR #70 September 8, 2023 18:51 — with Render Destroyed
@gabestein gabestein requested a review from 3mcd September 8, 2023 18:51
@gabestein
Copy link
Copy Markdown
Member Author

@3mcd -- no rush, but take a look when you get a sec. I think it's good enough and could merge and then incorporate some of the shad styles.

@3mcd 3mcd merged commit f60704c into main Sep 8, 2023
@3mcd 3mcd deleted the gs/user-styles branch September 8, 2023 18:57
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.

4 participants