Add mappings for CSS display property. - #2818
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-2818--wai-aria.netlify.app |
|
From an editorial standpoint, I'm quite unsure of the right CSS classes, correct cross-referencing, etc. here. |
| <tr> | ||
| <th>[[WAI-ARIA-1.2]]</th> | ||
| <td> | ||
| <span class="core-mapping">Element SHOULD be exposed</span> |
There was a problem hiding this comment.
@pkra This is a bit of a tricky case because strictly speaking, display: contents has no impact on accessibility at all. There's no explicit mention anywhere that it should be excluded from the accessibility tree, which implicitly means that it should be included. The problem is that this is commonly misunderstood due to display: none, etc., so I think this should be explicit.
I think the best match in the ARIA spec is the concept of elements that must be included in the accessibility tree. However, that could be confusing because that section has a very explicit list of rules which doesn't have any allowance for extension via an AAM spec.
Alternatively, we could just say "not mapped" (since it doesn't map to any property in ARIA) and rely on the comments to make the tree inclusion explicit.
There was a problem hiding this comment.
ARIA 7.1 Excluding Elements from the Accessibility Tree explicitly talks about host language semantics which exclude things from the tree. That leads to another possibility here: we could add text to ARIA 7.2 Including Elements in the Accessibility Tree; e.g.
Elements, including their descendent elements, that have host language semantics specifying that the element has explicit semantic relevance, such as CSS display:contents.
Yet another possibility is that we just link to the 7. Accessibility Tree section of the ARIA spec, which talks more generally about why something might be included in the accessibility tree (because it can fire events, has a feature which needs to be exposed, etc.).
|
Notes from the css-aam sync today:
|
ZoeBijl
left a comment
There was a problem hiding this comment.
lgtm as discussed during review :)
| </section> | ||
| <section id="css-property-mappings"> | ||
| <h3>CSS Property Mappings</h3> | ||
| <h4 id="prop-display">`display` property</h4> |
There was a problem hiding this comment.
We decided on having a generic top level for each property. Should we add some text here about the property? Maybe link to the relevant CSS spec that defines the property?
RESOLUTION: we decided to not include any text, less is more.
Co-authored-by: Keith Cirkel <keithamus@users.noreply.github.com>
🚀 Netlify Preview:
🔄 this PR updates the following sspecs: