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

Correct WAI-ARIA integration for custom elements #8974

Merged
merged 2 commits into from
Mar 7, 2023

Conversation

annevk
Copy link
Member

@annevk annevk commented Mar 2, 2023

This requires corresponding changes in https://w3c.github.io/core-aam/ as discussed in w3c/core-aam#152.


@scottaohara @alice @spectranaut I think WAI-ARIA will want to define a similar contract around ARIA properties. Namely that for each of them you get a string, element, list of elements, or their default value.


/common-dom-interfaces.html ( diff )
/custom-elements.html ( diff )
/dom.html ( diff )

@annevk annevk added accessibility Affects accessibility a11y-tracker Group bringing to attention of a11y, or tracked by the a11y Group but not needing response. labels Mar 2, 2023
@annevk annevk requested a review from domenic March 2, 2023 19:27
@annevk
Copy link
Member Author

annevk commented Mar 2, 2023

If #8962 lands first this will need changes to account for that and vice versa.

source Show resolved Hide resolved
source Show resolved Hide resolved
source Show resolved Hide resolved
@@ -14249,6 +14253,19 @@ interface <dfn interface>DOMStringMap</dfn> {
<li><p>If <var>map</var>[<var>stateOrProperty</var>] <span data-x="map exists">exists</span>,
then return it.</p></li>

<li>
Copy link
Member

Choose a reason for hiding this comment

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

I think we should prioritize this higher than the internal content attribute map. Otherwise, for the element-valued properties you'll get the empty string.

Copy link
Member Author

Choose a reason for hiding this comment

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

‼️

@annevk annevk added the topic: reflect For issues with reflected IDL attributes and friends. label Mar 3, 2023
@annevk annevk requested a review from domenic March 3, 2023 19:00
@annevk annevk merged commit 4c10dbb into main Mar 7, 2023
@annevk annevk deleted the annevk/wai-aria-custom-elements branch March 7, 2023 01:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y-tracker Group bringing to attention of a11y, or tracked by the a11y Group but not needing response. accessibility Affects accessibility topic: reflect For issues with reflected IDL attributes and friends.
Development

Successfully merging this pull request may close these issues.

None yet

2 participants