Skip to content

errata: [htmlaam] popovertarget expanded state calculation shouldn't refer to command attribute. - #2884

Open
lukewarlow wants to merge 1 commit into
w3c:mainfrom
lukewarlow:popovertarget-fix
Open

errata: [htmlaam] popovertarget expanded state calculation shouldn't refer to command attribute.#2884
lukewarlow wants to merge 1 commit into
w3c:mainfrom
lukewarlow:popovertarget-fix

Conversation

@lukewarlow

Copy link
Copy Markdown
Member

Closes #????

Corrects the popovertarget expanded state section to refer to elements with the popovertarget attribute instead of command attribute.

And a few other todo items (delete this section after performing them):

  • For every spec that this PR edits, please add the appropriate spec:<spec_name> label. If you don't have privileges to do this, editors will do it for you.
  • If the change is editorial, please add "Editorial:" at the start of your PR name, and delete the "Test, Documentation and Implementation tracking" section below.

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:

@netlify

netlify Bot commented Sep 1, 2026

Copy link
Copy Markdown

Deploy Preview for wai-aria ready!

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

@lukewarlow lukewarlow changed the title popovertarget expanded state calculation shouldn't refer to command attribute. [htmlaam] popovertarget expanded state calculation shouldn't refer to command attribute. Sep 1, 2026

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

Good catch. 👍

@jnurthen jnurthen changed the title [htmlaam] popovertarget expanded state calculation shouldn't refer to command attribute. errata: [htmlaam] popovertarget expanded state calculation shouldn't refer to command attribute. Sep 3, 2026
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.

3 participants