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

Show loader upon submit #1

Closed
wants to merge 1 commit into from
Closed

Show loader upon submit #1

wants to merge 1 commit into from

Conversation

mmikhan
Copy link

@mmikhan mmikhan commented Feb 3, 2025

Liked your project a lot and decided to use it in a project. The first thing I have noticed is the loading spinner is missing upon submitting the form. Not sure if this is what you prefer but I always find it a good user feedback to show a loading spinner or some sort of status upon submitting a form in the in-between moment.

So, this PR adds a loading spinner and loading text upon submitting the auth form.

Note: I couldn't test it since you are using a custom version of better auth from your local disk. So, I am blindly submitting this PR. It might break stuff. So, please feel free to take a brief look at it, make the necessary changes (if needed) or discard if it's not up to the mark 🙏

@daveycodez
Copy link
Contributor

Hmm I’m using Lucide to show a spinner on the Button on submit, specifically Loader2

do you have Lucide installed?

@daveycodez
Copy link
Contributor

Specifically, the action button with Log In or Sign Up changes to the Loader. The other buttons get “disabled” state. Which button specifically are you trying to show a loader for?

@mmikhan
Copy link
Author

mmikhan commented Feb 3, 2025

The spinner doesn't work for the social auth. I saw the spinner is working for the email, password auth though

@daveycodez
Copy link
Contributor

I’ll fix that and merge it in a few hrs

@daveycodez
Copy link
Contributor

@mmikhan Just published 1.0.26 which will show the Action spinner on signIn with provider & passkey. LMK if this resolves the issue for you

@mmikhan
Copy link
Author

mmikhan commented Feb 3, 2025

It does, thanks a lot 🫡

@mmikhan mmikhan closed this Feb 3, 2025
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.

2 participants