-
Notifications
You must be signed in to change notification settings - Fork 55
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
Enable autoFocus of input fields for registration, login, and password reset. #2469
Enable autoFocus of input fields for registration, login, and password reset. #2469
Conversation
…firm signup, and setup 2FA.
Once PR #2364 is merged, I will resolve any conflicts and mark this ready for review. |
I'm testing these changes using a screen reader to rule out potential accessibility issues. |
…age visibility in development.
…' of github.com:cisagov/crossfeed into 2260-focus-input-field-for-login-page-and-2fa-component
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
🗣 Description
Add auto focus to first field of components related to sign in, registration, and password reset.
Update placeholder text to improve site accessibility when using a screen reader.
Delete AuthContextOld.tsx, commented out code, and unused imports.
💭 Motivation and context
Closes issue #2260
🧪 Testing
✅ Pre-approval checklist
in code comments.
to reflect the changes in this PR.
✅ Pre-merge checklist
✅ Post-merge checklist