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

Update ARIA IDL to define AccessibilityRole and AriaAttributes instead of ARIAMixin #1726

Closed
mrego opened this issue Apr 20, 2022 · 6 comments
Assignees

Comments

@mrego
Copy link
Member

mrego commented Apr 20, 2022

ARIA IDL only defines ARIAMixin interface: https://w3c.github.io/aria/#idl-interface

While AOM has 2 interfaces AccessibilityRole and AriaAttributes: https://wicg.github.io/aom/spec/aria-reflection.html#idl-interface

Both Chromium and WebKit implementations match AOM regarding this:

Should ARIA spec get updated?

@jnurthen
Copy link
Member

@cookiecrook can you please let us know what to do with this

@cookiecrook
Copy link
Contributor

cookiecrook commented Apr 21, 2022

IIRC, I had originally proposed them separated (role has a broader applicability than the ARIA attrs) but combining the interfaces in ARIA may have been @domenic's suggestion sometime after the AOM proposal made it into the ARIA issue tracker. Practically speaking, it's not important to separate since implementations would likely want both.

Unless @domenic has a reason, I think it makes sense to update the ARIA spec to match the implementations.

@domenic
Copy link
Contributor

domenic commented Apr 21, 2022

It's important to merge them for HTML's features which treat them the same, namely ElementInternals.

It is unobservable whether they are a single mixin or two, so implementations don't have much weight here IMO. They can change to update the spec or not, or they could split it into 5 different mixins. It's an implementation choice and not relevant to specs.

@cookiecrook
Copy link
Contributor

I think this can be closed then. The original hand-off proposal from AOM to ARIA WG is no longer relevant IMO, but we could clean it up and have the AOM explainer just reference the new portion of the ARIA spec. What do you think @mrego?

@mrego
Copy link
Member Author

mrego commented Apr 22, 2022

I think this can be closed then. The original hand-off proposal from AOM to ARIA WG is no longer relevant IMO, but we could clean it up and have the AOM explainer just reference the new portion of the ARIA spec. What do you think @mrego?

Sounds good, thanks. I wasn't sure this was relevant or not, but I saw some inconsistency, thanks for the clarifications.

@mrego
Copy link
Member Author

mrego commented Jul 8, 2022

Ok, I've reported WICG/aom#194 to track the work on the AOM repo. Closing this for now.

Thanks for the feedback.

@mrego mrego closed this as completed Jul 8, 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

No branches or pull requests

4 participants