Comment 11 for bug 1977554

Revision history for this message
Scott Angel (scottangel1) wrote (last edit ):

Alright gang, I think I got it! I learned so much along the way.

Here's my branch
https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/scottangel/LP1977554-add-password-visibility-toggle-login-screen

Changed the <span> tags to <button> tags so it can be reached via
keyboard navigation.

Added a few more attributes to the password input tag.
autocapitalize="none"
spellcheck="false"
aria-description so screen readers can describe to the user what
the current state is.

Added aria-checked to the button for screen readers

Added a new css file for the login.component with a little styling.

I used NVDA as my testing screen reader as it seems to be the most popular. Not tested w/JAWS.