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

[TreeView] Investigate if aria-activedescendant attribute is necessary #12494

Closed
LukasTy opened this issue Mar 19, 2024 · 3 comments · Fixed by #12867
Closed

[TreeView] Investigate if aria-activedescendant attribute is necessary #12494

LukasTy opened this issue Mar 19, 2024 · 3 comments · Fixed by #12867
Assignees
Labels
accessibility a11y component: tree view TreeView, TreeItem. This is the name of the generic UI component, not the React module!

Comments

@LukasTy
Copy link
Member

LukasTy commented Mar 19, 2024

Created from: #12226 (comment)

It looks like the tree role is not listed to support aria-activedescendant attribute.
W3C examples seem to confirm it by not having this attribute in their examples.

To be confirmed with other libraries and checking the behavior after removing it.

Search keywords:

@LukasTy LukasTy added accessibility a11y component: tree view TreeView, TreeItem. This is the name of the generic UI component, not the React module! labels Mar 19, 2024
@flaviendelangle
Copy link
Member

flaviendelangle commented Apr 18, 2024

Found in https://www.w3.org/WAI/ARIA/apg/patterns/treeview/

The tree role supports the aria-activedescendant property, which provides an alternative to moving DOM focus among treeitem elements when implementing keyboard navigation. For details, see Managing Focus in Composites Using aria-activedescendant.

If a component container has an ARIA role that supports the aria-activedescendant property, it is not necessary to manipulate the tabindex attribute and move DOM focus among focusable elements within the container. Instead, only the container element needs to be included in the tab sequence. When the container has DOM focus, the value of aria-activedescendant on the container tells assistive technologies which element is active within the widget.

So, from what I understand, it was needed before, but no longer is since we focus the item directly

@LukasTy
Copy link
Member Author

LukasTy commented Apr 18, 2024

So, from what I understand, it was needed before, but no longer is since we focus the item directly

Yes, makes total sense. 👍

Copy link

⚠️ This issue has been closed. If you have a similar problem but not exactly the same, please open a new issue.
Now, if you have additional information related to this issue or things that could help future readers, feel free to leave a comment.

@LukasTy: How did we do? Your experience with our support team matters to us. If you have a moment, please share your thoughts in this short Support Satisfaction survey.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility a11y component: tree view TreeView, TreeItem. This is the name of the generic UI component, not the React module!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants