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

fix: Follow aria-labelledby and aria-describedby if they point to elements in the same shadow root #827

Merged
merged 5 commits into from
Apr 23, 2022

Conversation

nolanlawson
Copy link
Contributor

Partially addresses #768. Fixes aria-labelledby and aria-describedby in the following situation:

  1. The two elements in question (i.e. the element with aria-*edby and the element referenced by that attribute) are both located in the same shadow root.
  2. The text contents of the second element are fully contained within its own DOM (not its shadow DOM).

I've added two tests to confirm the behavior as well.

@changeset-bot
Copy link

changeset-bot bot commented Apr 14, 2022

🦋 Changeset detected

Latest commit: aa1c3d9

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
dom-accessibility-api Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@nolanlawson

This comment was marked as resolved.

@eps1lon eps1lon changed the title fix: fix co-located elements in shadow DOM fix: Follow aria-labelledby and aria-describedby if they point to elements in the same shadow root Apr 23, 2022
Copy link
Owner

@eps1lon eps1lon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Much appreciated, thanks!

@eps1lon eps1lon merged commit a1daca5 into eps1lon:main Apr 23, 2022
@github-actions github-actions bot mentioned this pull request Apr 23, 2022
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

Successfully merging this pull request may close these issues.

None yet

2 participants