Skip to content

Add mappings for CSS display property. - #2818

Draft
jcsteh wants to merge 8 commits into
mainfrom
css-display
Draft

Add mappings for CSS display property.#2818
jcsteh wants to merge 8 commits into
mainfrom
css-display

Conversation

@jcsteh

@jcsteh jcsteh commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

🚀 Netlify Preview:
🔄 this PR updates the following sspecs:

@jcsteh
jcsteh requested a review from ZoeBijl June 16, 2026 07:48
@netlify

netlify Bot commented Jun 16, 2026

Copy link
Copy Markdown

Deploy Preview for wai-aria ready!

Name Link
🔨 Latest commit b064ff5
🔍 Latest deploy log https://app.netlify.com/projects/wai-aria/deploys/6a7a996db532990008f7b3fc
😎 Deploy Preview https://deploy-preview-2818--wai-aria.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions

Copy link
Copy Markdown
Contributor

🚀 Deployed on https://deploy-preview-2818--wai-aria.netlify.app

@github-actions
github-actions Bot temporarily deployed to pull request June 16, 2026 07:49 Inactive
@jcsteh

jcsteh commented Jun 16, 2026

Copy link
Copy Markdown
Contributor Author

From an editorial standpoint, I'm quite unsure of the right CSS classes, correct cross-referencing, etc. here.

Comment thread css-aam/index.html
<tr>
<th>[[WAI-ARIA-1.2]]</th>
<td>
<span class="core-mapping">Element SHOULD be exposed</span>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If some ARIA spec language is desirable here, two existing options might be

The latter doesn't appear to be referenced anywhere in the ARIA repository, so it could also be rephrased to fit here.

Otherwise, a new term might be usfeul

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@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.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.).

@github-actions
github-actions Bot temporarily deployed to pull request June 30, 2026 06:30 Inactive
@ZoeBijl

ZoeBijl commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Notes from the css-aam sync today:

  • Reviewed PR Add mappings for CSS display property. #2818
  • Should there be a definition for “Included” in ARIA?
  • TODO: Correctly link to definitions and specs with ReSpec
  • RESOLUTION: set maximum table of contents level to three.
  • RESOLUTION: each property will have a generic “top” level (<h4>) with the various values nested inside (<h5>)

@github-actions
github-actions Bot temporarily deployed to pull request June 30, 2026 10:23 Inactive

@ZoeBijl ZoeBijl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm as discussed during review :)

Comment thread css-aam/index.html
</section>
<section id="css-property-mappings">
<h3>CSS Property Mappings</h3>
<h4 id="prop-display">`display` property</h4>

@ZoeBijl ZoeBijl Jun 30, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Comment thread css-aam/index.html Outdated
Co-authored-by: Keith Cirkel <keithamus@users.noreply.github.com>
@github-actions
github-actions Bot temporarily deployed to pull request August 11, 2026 03:40 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants