Skip to content

Remove group as allowed child of tree - #2094

Merged
pkra merged 1 commit into
mainfrom
allowed-tree-children
Feb 14, 2025
Merged

pkra merged 1 commit into
mainfrom
allowed-tree-children

Conversation

@mcking65

@mcking65 mcking65 commented Dec 17, 2023 •

Copy link
Copy Markdown
Contributor

Closes #2014

Removes group as an allowed accessibility child of tree to fix inconsistency with definition of treeitem. Direct children of tree can only be parent nodes or end nodes. Leaf nodes must be a child of a group that is a child of a parent node treitem. Even in a virtualized tree, if the elements that represent the ancestry of rendered nodes were not present in the DOM, the tree would be inoperable.

PR tracking

Check these when the relevant issue or PR has been made, OR after you have confirmed the
related change is not necessary (add N/A). Leave unchecked if you are unsure. Read the
Process Document or
Test Overview for more information.

  • Related Core AAM Issue/PR: not required
  • Related AccName Issue/PR: not required
  • Related APG Issue/PR: not required

Implementation tracking


Preview | Diff

@giacomo-petri

Copy link
Copy Markdown
Contributor

Since I approved the PR without adding any notes (and changes seem great based on the rationale of #2315), I'm assuming I mistakenly applied the agenda label to the wrong GitHub PR. Now, it would be interesting to figure out where I missed applying the agenda label... 🥲

@spectranaut spectranaut removed the Agenda label Oct 8, 2024
@pkra pkra added the Agenda label Feb 9, 2025
@spectranaut

Copy link
Copy Markdown
Contributor

Discussed in today's meeting: https://www.w3.org/2025/02/13-aria-minutes.html#6a80

@giacomo-petri

Copy link
Copy Markdown
Contributor

Opened issue in ACT repo as requested: act-rules/act-rules.github.io#2309

@pkra
pkra merged commit f062faa into main Feb 14, 2025
@pkra
pkra deleted the allowed-tree-children branch February 14, 2025 08:49
YusukeHirao added a commit to markuplint/markuplint that referenced this pull request Apr 14, 2025
…IA spec change

Aligns with W3C ARIA specification change (w3c/aria#2094).

The 'tree' role no longer requires 'group > treeitem' as direct children

according to the ARIA specification update. Direct children of a 'tree'

must be 'treeitem'.

See: w3c/aria#2094
YusukeHirao added a commit to markuplint/markuplint that referenced this pull request Apr 14, 2025
…IA spec change

Aligns with W3C ARIA specification change (w3c/aria#2094).

The 'tree' role no longer requires 'group > treeitem' as direct children

according to the ARIA specification update. Direct children of a 'tree'

must be 'treeitem'.

See: w3c/aria#2094
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

treeitem scope should allow group with accessibility parent tree

7 participants