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

[template] [ux] SignInPage improvements #4067

Closed
Tracked by #4153
bharatkashyap opened this issue Sep 10, 2024 · 4 comments · Fixed by #4370
Closed
Tracked by #4153

[template] [ux] SignInPage improvements #4067

bharatkashyap opened this issue Sep 10, 2024 · 4 comments · Fixed by #4370
Assignees
Labels
design: ux Design scope: toolpad-core Abbreviated to "core"

Comments

@bharatkashyap
Copy link
Member

bharatkashyap commented Sep 10, 2024

Current Proposed
Screenshot 2024-09-10 at 2 03 30 PM

Search keywords:

@zanivan
Copy link

zanivan commented Sep 10, 2024

Done!

image

@bharatkashyap
Copy link
Member Author

Done!

Appreciate the speed 😄 There is a little complexity though, the Passkey provider requires a text field as well, which is why I was wondering what the ideal UI would be for its placement within this list

@bharatkashyap bharatkashyap changed the title [core] [ux] SignInPage improvements [template] [ux] SignInPage improvements Sep 24, 2024
@aress31
Copy link

aress31 commented Oct 30, 2024

Adding a theme switcher would be great and make everything feel more complete. It would also be helpful to include customization options, especially for the outer box or component.

Copy link

github-actions bot commented Nov 8, 2024

This issue has been closed. If you have a similar problem but not exactly the same, please open a new issue.
Now, if you have additional information related to this issue or things that could help future readers, feel free to leave a comment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design: ux Design scope: toolpad-core Abbreviated to "core"
Projects
Status: Completed
Development

Successfully merging a pull request may close this issue.

4 participants