Editorial: add valuetype column to ariamixin correspondance table #2105
Conversation
| <tr><td><dfn>ariaPressed</dfn></td><td><sref>aria-pressed</sref></td></tr> | ||
| <tr><td><dfn>ariaReadOnly</dfn></td><td><pref>aria-readonly</pref></td></tr> | ||
| <tr><th>IDL Attribute</th><th>Reflected ARIA Content Attribute</th><th>Value type</th></tr> | ||
| <tr><td><dfn data-dfn-for="ARIAMixin">role</dfn></td><td><a href="#introroles">role</a></td><td><a href="#valuetype_token_list">token list</a></td></tr> |
There was a problem hiding this comment.
role was actually quite difficult to look up because we don't have a table describing the value type of role, and also the reference was missing (so I've added it here).
spectranaut
requested review from
spectranaut
and removed request for
jnurthen
There was a problem hiding this comment.
Hi @keithamus ! I really appreciate you looking into this, a few requests. Also, maybe the html "types" should be included here as well, for ease of reference.
@spectranaut shall I do that in this PR or a follow up? |
If you also think it would be useful, then I would recommend doing it in this PR. It's a little repetitive of the previous section but personally I think repetitivity in specs -- if it helps the consumer of the spec -- is fine. Would be nice to hear an editor weigh in though, @pkra ? :) |
|
@spectranaut @keithamus editorially this change looks good to me. As an aside. It made me wonder if this table should be automatically generated, i.e., moving the IDL Attribute into the characteristics tables and have aria.js create the table. But that would have to wait until aria.js is in better shape. |
pkra
added
the
editorial
|
@keithamus could you resolve the merge conflicts so we can merge this? |
|
@pkra @keithamus Great idea! Since there is outstanding work to migrate many ARIA attributes to a new value type, i.e., enumerated (#2281), could you please hold off on merging this to better coordinate with the broader IDL updates? I anticipate that the "6.2.4 Value" section will change as part of #2281. I've also added this PR to the IDL project. |
Refs #1110
While looking through #1110 to figure out what we can do, I started enumerating all of the ariamixing properties and their corresponding value types, which I then realised is probably a good reference to have around, and so I added the value type as an additional column to the table that maps ARIAMixin properties to their corresponding attributes.
This information is duplicated in each of the attributes, but it's a little trickier to hunt for the table and navigate around the spec a lot. This table allows us to more easily see at-a-glance what value types map to what properties in the IDL.
Preview | Diff