Skip to content

[html-aam] Add img w/controls attr mapping#2815

Open
rahimabdi wants to merge 1 commit into
mainfrom
rahim/update-controls-attr-mapping
Open

[html-aam] Add img w/controls attr mapping#2815
rahimabdi wants to merge 1 commit into
mainfrom
rahim/update-controls-attr-mapping

Conversation

@rahimabdi

@rahimabdi rahimabdi commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

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

Closes w3c/html-aam#601

This PR makes two changes:

  • Adds new mapping table for img element with a controls attribute, with a callout that empty alt should be ignored
  • Adds img element to controls attribute mapping (along with existing audio and video)

Test, Documentation and Implementation tracking

Once this PR has been reviewed and has consensus from the working group, tests should be written and issues should be opened on browsers. Add N/A and check when not applicable.

  • "author MUST" tests:
  • "user agent MUST" tests:
  • Browser implementations (link to issue or commit):
    • WebKit:
    • Gecko:
    • Blink:
  • ACT review?
  • Does this need AT implementations?
  • Related APG Issue/PR:
  • MDN Issue/PR:

@rahimabdi rahimabdi self-assigned this Jun 5, 2026
@netlify

netlify Bot commented Jun 5, 2026

Copy link
Copy Markdown

Deploy Preview for wai-aria ready!

Name Link
🔨 Latest commit f70193a
🔍 Latest deploy log https://app.netlify.com/projects/wai-aria/deploys/6a235925d58abe0008db866c
😎 Deploy Preview https://deploy-preview-2815--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

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

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

@github-actions github-actions Bot temporarily deployed to pull request June 5, 2026 23:19 Inactive
@scottaohara

Copy link
Copy Markdown
Member

@rahimabdi thanks again for getting this going.

from what we talked about, i think we still need to separate out the attribute mapping for the img[controls] from the other instances of [controls]. because this still needs to call out that the controls attr on an img needs to sprout a parent role=group in the a11y tree to contain the img and any controls that will be rendered as a sibling to the img

i.e., the img[controls] attribute mapping table should mimic the video element mappings, but specifically mention the mappings don't alter the img, but instead add a parent group / sibling toolbar of controls.

then the comments for img w/controls and the img[controls] attribute tables can reference each other in the spec. simply "see element" / "see attribute"

@scottaohara scottaohara left a comment

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.

see prior comment

@jnurthen jnurthen requested review from HaTheo and aardrian June 11, 2026 17:05
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.

New controls attribute for img element

2 participants