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 - Ensure Manager View Calendar Changes Are Announced to Screen Readers #48

Open
mdarrik opened this issue Jul 21, 2019 · 0 comments

Comments

@mdarrik
Copy link
Contributor

mdarrik commented Jul 21, 2019

I asked this question in chat just before you hit that bug where the calendar view wasn't actually updating (#44 ). But there needs to be some announcement for screen readers when the ManagerView data actually changes.

So I see two potential solutions going forward:

  1. Change each calendar view into a dynamically generated page.
    • Pro: Makes return navigation to a specific range easier.
    • Pro: Can use a link instead of a button and don't have to worry about aria-states.
    • Pro: Changes are expected by SR users, so don't have to worry about announcing things have changed if it's clear that they are clicking a link.
    • Con: May have to refactor the page/component some to make work
    • Con: Have to be careful if want to use the MangerView as more than just a page-component
  2. Use aria-live or the HTML "output" element for the calendar. Scott O'Hara has a great write-up on the html output element here: "output: HTML's native live region element"
    • Pro: Don't have to configure routing & links.
    • Pro: Keeps with the current logic of the page/component
    • Con: Aria-states can be prone to errors
    • Con: Output may not work since a calendar may be better semantically written as an ordered list.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant