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

⌨ Manual accessibility testing for Notifications #10443

Closed
7 tasks done
gobyrne opened this issue May 22, 2024 · 6 comments
Closed
7 tasks done

⌨ Manual accessibility testing for Notifications #10443

gobyrne opened this issue May 22, 2024 · 6 comments
Assignees
Labels
accessibility Issues related to accessibility
Milestone

Comments

@gobyrne
Copy link
Member

gobyrne commented May 22, 2024

⌨ Accessibility Issue

Work with @esizer to perform manual accessibility testing on the notification feature.

Paths for testing

  1. View notification on nav and activate link
  2. Mark all as read
  3. View notifications on nav and click to "view all notifications"
  4. On notification page, view read vs unread?
  5. Account settings page, toggle on/off for families
  6. Read an email - send one to team? best effort

✅ Acceptance Criteria

A set of assumptions which, when tested, verify that the issue was addressed.

@gobyrne gobyrne added the accessibility Issues related to accessibility label May 22, 2024
@gobyrne gobyrne added this to the Email verification milestone May 22, 2024
@gobyrne gobyrne added review in refinement Ready to be looked at and pulled into "ready to dev" and removed review in refinement Ready to be looked at and pulled into "ready to dev" labels May 23, 2024
@petertgiles
Copy link
Contributor

Plan to look at this on Friday

@esizer
Copy link
Member

esizer commented Jun 11, 2024

Completed my tests, it went very well except for the instructions. I had a lot of difficulty with them and believe they may cause confusion for non-sighted users.

Created a ticket #10630

@vd1992
Copy link
Contributor

vd1992 commented Jun 11, 2024

Gave the testing a go, asked Eric some questions now and there. Thanks for answering

Issues
#10637
#10638
#10640

@mnigh
Copy link
Contributor

mnigh commented Jun 11, 2024

I have completed the testing on macOS Safari + VoiceOver and created issues:

@yonikid15
Copy link
Contributor

Testing done, looked pretty good overall.

Issues:
#10646
Found same issue #10638
Found same issue #10637. When tabbing again the focus should start on the next notification, not the panel. Same with entering "mark all as read".

@esizer
Copy link
Member

esizer commented Jun 12, 2024

Looks like everyone has completed their test at this point. Are we good to close this now?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility Issues related to accessibility
Projects
Archived in project
Development

No branches or pull requests

6 participants