Skip to content

Commit

Permalink
Merge pull request #18206 from martenson/autopass
Browse files Browse the repository at this point in the history
play nice with password managers
  • Loading branch information
mvdbeek authored May 23, 2024
2 parents 6e7160b + 1214da0 commit cd779a4
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 4 deletions.
6 changes: 3 additions & 3 deletions client/src/components/Login/ChangePassword.vue
Original file line number Diff line number Diff line change
Expand Up @@ -50,15 +50,15 @@ async function submit() {

<BCard header="Change your password">
<BFormGroup v-if="expiredUser" label="Current Password">
<BFormInput v-model="current" type="password" />
<BFormInput v-model="current" type="password" autocomplete="current-password" />
</BFormGroup>

<BFormGroup label="New Password">
<BFormInput v-model="password" type="password" />
<BFormInput v-model="password" type="password" autocomplete="new-password" />
</BFormGroup>

<BFormGroup label="Confirm password">
<BFormInput v-model="confirm" type="password" />
<BFormInput v-model="confirm" type="password" autocomplete="new-password" />
</BFormGroup>

<BButton type="submit">Save new password</BButton>
Expand Down
3 changes: 2 additions & 1 deletion client/src/components/Login/LoginForm.vue
Original file line number Diff line number Diff line change
Expand Up @@ -195,7 +195,8 @@ function returnToLogin() {
required
:state="passwordState"
name="password"
type="password" />
type="password"
autocomplete="current-password" />

<BFormText v-if="showResetLink">
<span v-localize>Forgot password?</span>
Expand Down
2 changes: 2 additions & 0 deletions client/src/components/Login/RegisterForm.vue
Original file line number Diff line number Diff line change
Expand Up @@ -144,6 +144,7 @@ async function submit() {
v-model="password"
name="password"
type="password"
autocomplete="new-password"
required />
</BFormGroup>

Expand All @@ -153,6 +154,7 @@ async function submit() {
v-model="confirm"
name="confirm"
type="password"
autocomplete="new-password"
required />
</BFormGroup>

Expand Down

0 comments on commit cd779a4

Please sign in to comment.