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.

Sign up for GitHub

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

Jump to bottom

IDL element references attributes names #1724

Closed
mrego opened this issue Apr 18, 2022 · 5 comments · Fixed by #1729
Closed

IDL element references attributes names #1724

mrego opened this issue Apr 18, 2022 · 5 comments · Fixed by #1729
Assignees
@cookiecrook
Milestone
2023

Comments

@mrego
Copy link
Member

mrego commented Apr 18, 2022

See the non-normative section at https://w3c.github.io/aria/#idl_attr_disambiguation:

Any attribute name referencing concepts that are combinations of two or more words (such as "described by") becomes a camel-cased IDL attribute capitalizing each word boundary. For example, aria-describedby becomes ariaDescribedBy with both the D and B capitalized.

This doesn't match Chromium implementation, neither the AOM explainer where this is named ariaDescribedByElements.

I've filled an AOM issue WICG/aom#191, depending on what's agreed there ARIA spec might need to get updated.
@mrego mrego mentioned this issue Apr 18, 2022
Closed
@jnurthen jnurthen added this to the ARIA 1.3 milestone Apr 21, 2022
@cookiecrook
Copy link
Contributor

Yeah, I think that example predates the element reflection stuff, so it should be updated to use a different example attribute.

@cookiecrook
Copy link
Contributor

I'll use ariaValueText

cookiecrook added a commit that referenced this issue Apr 21, 2022
@cookiecrook
Replace irrelevant ariaDescribedBy example with ariaValueText
81a1b95 
Resolves #1724, ariaDescribedBy is not relevant to include in the prose because its awaiting FrozenArray<Element> reflection via ariaDescribedByElements
@cookiecrook cookiecrook mentioned this issue Apr 21, 2022
Merged
@cookiecrook
Copy link
Contributor

Thanks @mrego

@mrego
Copy link
Member Author

mrego commented Apr 22, 2022

What would be the expected name for aria-describedby: ariaDescribedBy or ariaDescribedByElements? Thanks.

pkra pushed a commit that referenced this issue Apr 29, 2022
@cookiecrook
Replace irrelevant ariaDescribedBy example with ariaValueText (#1729)
f215561 
Resolves #1724, ariaDescribedBy is not relevant to include in the prose because its awaiting FrozenArray<Element> reflection via ariaDescribedByElements
@mrego
Copy link
Member Author

mrego commented Apr 29, 2022

What would be the expected name for aria-describedby: ariaDescribedBy or ariaDescribedByElements? Thanks.

As this question wasn't answered, I've opened #1732 to clarify this topic.

jnurthen pushed a commit that referenced this issue Oct 10, 2023
@cookiecrook @jnurthen
Replace irrelevant ariaDescribedBy example with ariaValueText (#1729)
042c594 
Resolves #1724, ariaDescribedBy is not relevant to include in the prose because its awaiting FrozenArray<Element> reflection via ariaDescribedByElements
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cookiecrook cookiecrook

Labels
None yet
Projects
None yet
Milestone
2023
Development

Successfully merging a pull request may close this issue.

Replace irrelevant ariaDescribedBy example with ariaValueText w3c/aria
3 participants
@mrego @cookiecrook @jnurthen