Skip to content
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

Accessibility improvements #61

Closed
adamwoodnz opened this issue Feb 14, 2023 · 4 comments · Fixed by #62
Closed

Accessibility improvements #61

adamwoodnz opened this issue Feb 14, 2023 · 4 comments · Fixed by #62
Assignees
Labels
accessibility Fix or enhancement related to accessibility bug Something isn't working
Milestone

Comments

@adamwoodnz
Copy link
Contributor

Issues found:

  1. No focus styling on profile page Account action buttons
  2. Password page 'show password' button has no accessible name
    Screen Shot 2023-02-15 at 10 55 14 AM
  3. No focus styling on 2FA page QR code or cancel button
  4. 2FA page QR code has no discernible text
    Screen Shot 2023-02-15 at 10 50 23 AM
  5. Success notifications when focused say 'Dismiss this notice, button' but they aren't dismissable via mouse or keyboard
    Screen Shot 2023-02-15 at 11 17 31 AM

Questions

  1. Should we add UI which is keyboard accessible for easy copy and paste of backup codes?
  2. Should the 'password best practices' link open in a new tab? Feels disruptive to navigate directly to an info link.
@adamwoodnz adamwoodnz added the bug Something isn't working label Feb 14, 2023
@iandunn
Copy link
Member

iandunn commented Feb 14, 2023

Success notifications when focused say 'Dismiss this notice, button' but they aren't dismissable

🤔 , IMO they shouldn't be dismissable, so the fix would be removing the label. I'd assume isDismissible=true param for the SnackBar component should do that.

Should we add UI which is keyboard accessible for easy copy and paste of backup codes?

Do you mean adding new UI elements? What would they look like/do?

Should the 'password best practices' link open in a new tab? Feels disruptive to navigate directly to an info link.

Usually I'd say it's best to let the user decide, but this may be an exception. I'd lean towards keeping it how it is, but don't feel strongly

https://www.nngroup.com/articles/new-browser-windows-and-tabs/

@iandunn iandunn added this to the Iteration 1 milestone Feb 15, 2023
@adamwoodnz
Copy link
Contributor Author

Do you mean adding new UI elements? What would they look like/do?

I was thinking something like

image

Depends if we're really pushing users to print. In that case perhaps a print button might be helpful.

@adamwoodnz
Copy link
Contributor Author

Usually I'd say it's best to let the user decide, but this may be an exception. I'd lean towards keeping it how it is, but don't feel strongly

https://www.nngroup.com/articles/new-browser-windows-and-tabs/

Good read, thanks. Yeah I can't say this info is reference material that should be left open, so you're probably right.

@iandunn
Copy link
Member

iandunn commented Feb 15, 2023

Ah, yes, those UI elements are planned in #45 . Feel free to pick that one up if you're interested 👍🏻

@adamwoodnz adamwoodnz linked a pull request Feb 16, 2023 that will close this issue
5 tasks
@adamwoodnz adamwoodnz added the accessibility Fix or enhancement related to accessibility label Feb 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility Fix or enhancement related to accessibility bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants