Skip to content

Commit

Permalink
Merge pull request #96 from sebadob/ux-password-manager-input-login-form
Browse files Browse the repository at this point in the history
better UX when using a password manager on login form
  • Loading branch information
sebadob authored Oct 25, 2023
2 parents 7a965a2 + 603a7ff commit 09d1d3a
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion frontend/src/routes/oidc/authorize/+page.svelte
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@
let showReset = false;
let showResetRequest = false;
let emailSuccess = false;
let emailAfterSubmit = '';
let formValues = {email: '', password: ''};
let formErrors = {};
Expand Down Expand Up @@ -175,6 +176,7 @@
// this will happen always if the user does the first try with a password-only account
// the good thing about this is, that it is a prevention against autofill passwords from the browser
needsPassword = true;
emailAfterSubmit = formValues.email;
} else {
err = t.invalidCredentials;
showResetRequest = true;
Expand All @@ -185,7 +187,7 @@
function onEmailInput() {
// this will basically remove the password input again if the user was asked to provide
// a password and afterward changes his email again
if (needsPassword) {
if (needsPassword && emailAfterSubmit !== formValues.email) {
needsPassword = false;
}
}
Expand Down

0 comments on commit 09d1d3a

Please sign in to comment.