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

Tree Pattern: Update selection guidance to include aria-checked #2121

Merged
merged 3 commits into from
Nov 15, 2021

Conversation

mcking65
Copy link
Contributor

@mcking65 mcking65 commented Nov 8, 2021

ARIA 1.3 clarifies when browsers may provide implicit selection.
Those changes to the ARIA specification enable APG to provide guidance on using ariachecked to represent selection in trees. This change updates the roles, states, and properties section of the tree view pattern accordingly.

Preview changes to tree pattern in RawGitHack

Note: revisions to listbox will be addressed separately.


Preview | Diff

@JAWS-test
Copy link

I haven't followed the entire discussion on the topic, but it seems odd to me that in the paragraph aria-checked and aria-selected can be used arbitrarily. Shouldn't there be some indication that primarily aria-selected should be used. Only if selected and checked are two different states that a treeitem has, both should be used?

@mcking65
Copy link
Contributor Author

mcking65 commented Nov 9, 2021

@JAWS-test wrote:

I haven't followed the entire discussion on the topic, but it seems odd to me that in the paragraph aria-checked and aria-selected can be used arbitrarily. Shouldn't there be some indication that primarily aria-selected should be used.

I'm planning to add a note related to the choice of selected or checked before this is ready for review.

Only if selected and checked are two different states that a treeitem has, both should be used?

In the spec, We included a set of 3 conditions that would have to be met by a tree that uses both checked and selected. Basically, we strongly advise against designing UI that would use both. It would be very difficult to ever meet all three conditions. I'm also planning to add a note about this as well.

@mcking65 mcking65 added this to the 1.2 Release 1 milestone Nov 9, 2021
@a11ydoer a11ydoer requested review from a11ydoer and jongund November 9, 2021 20:01
@mcking65 mcking65 force-pushed the tree-pattern-selected branch from 8ef3e8f to 2ea1610 Compare November 15, 2021 02:36
@mcking65 mcking65 marked this pull request as ready for review November 15, 2021 02:49
@mcking65 mcking65 merged commit c214552 into main Nov 15, 2021
@mcking65 mcking65 deleted the tree-pattern-selected branch November 15, 2021 20:50
@mcking65 mcking65 added documentation Example Page Related to a page containing an example implementation of a pattern labels Nov 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Example Page Related to a page containing an example implementation of a pattern
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants