Repository navigation
Clarify setSize and posinSet calculation for radios - #2919
spectranaut wants to merge 3 commits into
Conversation
✅ Deploy Preview for wai-aria ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
🚀 Deployed on https://deploy-preview-2919--wai-aria.netlify.app |
|
Maybe also take a look at: w3c/html-aam#598 (comment) |
|
The ARIA Working Group just discussed The full IRC log of that discussion<jugglinmike> subtopic: HTML-AAM: Clarify setSize and posinSet calculation for radios<jugglinmike> github: https://github.com//pull/2919 <jugglinmike> spectranaut_: I opened this one. I would love some reviewers! <jugglinmike> HaTheo: I'm happy to look at it. Microsoft has a lot of experience here <jugglinmike> spectranaut_: It'd be nice to have one more! <jugglinmike> front-endian-jane: I can take a look <jugglinmike> spectranaut_: Awesome |
|
ping @scottaohara @HaTheo @front-endian for review :) |
|
agree with the content of the changes. arguably, since this now calls out the there's this (the separation of all the |
HaTheo
left a comment
There was a problem hiding this comment.
re-review as Scott's point makes sense, only NIT feedback on actual changes.
|
Thanks @scottaohara and @HaTheo! I've update the spec by adding a "name" mapping and removed a bunch of prose in favor of pointing to it. I also added changes to Core-AAM... I realized the mapping is a little more complicate than redirecting to Actually, Theo, maybe you would know if Core-AAM is incorrect for posinset and setsize... it says to map them to |
|
Yes, I am not sure I have an authoritative answer but @douggeoffray or @benbeaudry might know the history here. What is stated here in the spec and mapping for aria properties in UIA and in the Core-AAM doesn't align with what I thought we did which was map to the native properties when they exist. I wonder if there is some particular history here I am missing or if things are dated, it's a good callout. For those unfamiliar with the history(likely not anyone in the conversation): In UI Automation(UIA) the accessibility mapping API in Windows, Microsoft added a table that would allow them to expose all of the properties and every time a property was added so the solution was to add a ARIA table which can accept all ARIA properties. This allows AT to have a pure/updated understanding the ARIA properties in the case that it was not in UIA without Microsoft having to update UIA everytime, in many cases using the available native UIA property is preferred(what chrome is doing), but also in those cases they are cross-referenced in this case in particular UIA chose to not map ARIA's |
🚀 Netlify Preview:
🔄 this PR updates the following specs: