Fix password manager auto complete #5165
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description Of Changes
I added the
autocomplete
attribute to the password input field on the login for so that password managers got the correct hint for whether to show a "new password" form fill or an "existing password" form fill.https://developer.1password.com/docs/web/compatible-website-design/#password-change-and-reset-forms
https://developer.mozilla.org/en-US/docs/Web/HTML/Attributes/autocomplete
Before:
After:
Code Changes
autocomplete
to password input on login page.autocomplete
field toCustomTextInput
component.Steps to Confirm
Pre-Merge Checklist
CHANGELOG.md
main
downgrade()
migration is correct and works