Simplify popover=hint behaviours & related spec fixes by jakearchibald · Pull Request #12345 · whatwg/html

jakearchibald · 2026-04-08T13:45:02Z

Commit message:

Simplify popover=hint behaviors

- Showing a hint popover doesn't hide auto popovers.
- Showing a hint popover hides non-parent hint popovers.
- Clicking outside popovers consistently hides them.
- Hiding an auto popover does not hide unrelated hint popovers.
- Showing an auto popover as a child of a hint popover downgrades the auto popover to a hint.
- Showing a popover during the show or hide of another popover is no longer allowed.

At least two implementers are interested (and none opposed):
- Mozilla
- Google
Tests are written and can be reviewed and commented upon at:
- Implement new/revised popover=hint behavior web-platform-tests/wpt#59237
Implementation bugs are filed:
- Chromium: https://crbug.com/499019927 (Also, PRs are already up.)
- Gecko: https://bugzilla.mozilla.org/show_bug.cgi?id=2034661
- WebKit: Not required. This will be picked up with general popover hint work.
Corresponding HTML AAM & ARIA in HTML issues & PRs: Not required.
MDN issue is filed: Not required.
The top of this comment includes a clear commit message to use.

(See WHATWG Working Mode: Changes for more details.)

Fixes #12304.

Fixes #12346.

Fixes #12355.

Fixes #12356.

Add stuff to infra & use it
Decide on & spec auto-in-hint behaviour
Decide on & spec what happens with popovers shown during beforetoggle-hide.
Review new tests & add more if needed
Apply wrapping

/form-elements.html ( diff )
/infrastructure.html ( diff )
/interactive-elements.html ( diff )
/popover.html ( diff )

jakearchibald · 2026-04-08T14:01:09Z

I'll sort out the wrapping after reviews & changes.

lukewarlow · 2026-04-08T14:14:41Z

Btw https://github.com/domfarolino/specfmt is quite useful for the wrapping.

jakearchibald · 2026-04-08T15:20:38Z

@lukewarlow haha your deleted comment did reveal a bug! It wasn't the return false that was wrong, but I was missing a return true.

jakearchibald · 2026-04-08T15:29:29Z

Edit: I've broken this out into #12355

av1.mp4

@mfreed7 sorry for another video, but I don't think I'd be able to describe it well. What do you see as the intended behaviour here?

https://random-stuff.jakearchibald.com/bug-repros/popover-show-on-hide-warning/ - here's the demo page.

Edit: I'm not convinced my spec totally solves this either, as hiding a popover hides child popovers, which would include the new ones added, and they'd be hidden with events. Maybe.

jakearchibald · 2026-04-09T12:00:41Z

@mfreed7 can you create a PR for your tests so far?

For whatwg/html#12345.

keithamus · 2026-04-10T09:04:38Z

+       <span>"<code>InvalidStateError</code>"</span> <code>DOMException</code>.</p></li>
+
+       <li><p>Return false.</p></li>
+      </ol>


See this discussion for more context: whatwg/html#12304 whatwg/html#12345 This CL implements a simplified behavioral model for popover=hint and popover=auto, cleanly separating the autoRootedPopoverStack from the hintRootedPopoverStack and introducing a `hintStackParent` to track where the hint stack branches off the auto stack. The new behavior resolves the following inconsistencies (gated behind the `PopoverHintNewBehavior` experimental runtime flag): 1. Opening a hint popover will not hide unrelated auto popovers. 2. Opening a hint popover closes only other non-ancestor hint popovers. 3. Clicking outside consistently closes both auto and hint popovers. 4. Hiding an auto popover closes only its child popovers. 5. Opening an auto popover inside a hint popover is disallowed and will fail. A new WPT test (`popover-hint-hierarchy.html`) is added to explicitly assert these 5 rules, and existing popover WPTs are updated to reflect the new behavior. A virtual test suite (`popover-hint-old-behavior`) is also added to continue testing the legacy behavior. I'm not sure how to best handle the changed tests (and I guess the new test too) in the context of a spec PR that is forthcoming. I'd like to be able to publish (as a WPT PR at least) the changes, to help with the PR effort. Suggestions appreciated. A note about the diff: I made all of the feature flag checks look like `if (!feature enabled) {` in the hopes that the old code would show as unchanged in the diff, but gerrit's diff algorithm isn't great. It's at least a little better like this than if I put the new code first, so I left it. Bug: 4990199 Change-Id: I41d8d96be71ba60ec5e7aa640c935258d1e33431 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7727959 Auto-Submit: Mason Freed <masonf@chromium.org> Reviewed-by: David Baron <dbaron@chromium.org> Commit-Queue: David Baron <dbaron@chromium.org> Cr-Commit-Position: refs/heads/main@{#1615186}

@jakearchibald

…ior, a=testonly Automatic update from web-platform-tests Implement new/revised popover=hint behavior See this discussion for more context: whatwg/html#12304 whatwg/html#12345 This CL implements a simplified behavioral model for popover=hint and popover=auto, cleanly separating the autoRootedPopoverStack from the hintRootedPopoverStack and introducing a `hintStackParent` to track where the hint stack branches off the auto stack. The new behavior resolves the following inconsistencies (gated behind the `PopoverHintNewBehavior` experimental runtime flag): 1. Opening a hint popover will not hide unrelated auto popovers. 2. Opening a hint popover closes only other non-ancestor hint popovers. 3. Clicking outside consistently closes both auto and hint popovers. 4. Hiding an auto popover closes only its child popovers. 5. Opening an auto popover inside a hint popover is disallowed and will fail. A new WPT test (`popover-hint-hierarchy.html`) is added to explicitly assert these 5 rules, and existing popover WPTs are updated to reflect the new behavior. A virtual test suite (`popover-hint-old-behavior`) is also added to continue testing the legacy behavior. I'm not sure how to best handle the changed tests (and I guess the new test too) in the context of a spec PR that is forthcoming. I'd like to be able to publish (as a WPT PR at least) the changes, to help with the PR effort. Suggestions appreciated. A note about the diff: I made all of the feature flag checks look like `if (!feature enabled) {` in the hopes that the old code would show as unchanged in the diff, but gerrit's diff algorithm isn't great. It's at least a little better like this than if I put the new code first, so I left it. Bug: 499019927 Change-Id: I41d8d96be71ba60ec5e7aa640c935258d1e33431 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7727959 Auto-Submit: Mason Freed <masonf@chromium.org> Reviewed-by: David Baron <dbaron@chromium.org> Commit-Queue: David Baron <dbaron@chromium.org> Cr-Commit-Position: refs/heads/main@{#1615186} -- Apply suggestion from @jakearchibald Co-authored-by: Jake Archibald <jaffathecake@gmail.com> -- Undo changes to this file -- Spaces -- Apply suggestion from @jakearchibald Co-authored-by: Jake Archibald <jaffathecake@gmail.com> -- wpt-commits: cf48cdceb7740ad09f521fb72b259d51cbb3f489, 36834da0c9a8370330208d84eed5abe092c11f11, b4d10cb3e66c1598e2f02a8d7106d3b34587b777, d1ee0b8ac21c4986aa60ef465e530ef0b76e7546, 7c5c87139d8080facb2095438372aa003b15e1f1 wpt-pr: 59237

A large change was made to popover=hint to fix weird behaviours, so the current implementations don't match the spec. - Spec issue: whatwg/html#12304 - Spec PR: whatwg/html#12345 --------- Co-authored-by: Claas Augner <495429+caugner@users.noreply.github.com>

…PT (#59955) * Attempt to clean up the WPT export of popover=hint from Chromium to WPT This PR pushes the latest state of `popover=hint` tests from Chromium (prior to the landing of https://crrev.com/c/7851723, which was a temporary stopgap to unbreak the infra) to WPT. This one PR should completely replace any PRs that come out of any still-unlanded PRs in this list: whatwg/html#12345 (comment) * Update test condition descriptions Change-Id: I363b411aa8f6979fbde3766c27afe229703dfc1f Change-Id: Ibd6af063310585b9044de63a06e36ffd79d011d4 Change-Id: I917bf939614f9811470444e9f13c942e5c63b1b1

…pover=hint from Chromium to WPT, a=testonly Automatic update from web-platform-tests Attempt to clean up the WPT export of popover=hint from Chromium to WPT (#59955) * Attempt to clean up the WPT export of popover=hint from Chromium to WPT This PR pushes the latest state of `popover=hint` tests from Chromium (prior to the landing of https://crrev.com/c/7851723, which was a temporary stopgap to unbreak the infra) to WPT. This one PR should completely replace any PRs that come out of any still-unlanded PRs in this list: whatwg/html#12345 (comment) * Update test condition descriptions Change-Id: I363b411aa8f6979fbde3766c27afe229703dfc1f Change-Id: Ibd6af063310585b9044de63a06e36ffd79d011d4 Change-Id: I917bf939614f9811470444e9f13c942e5c63b1b1 -- wpt-commits: 3dbd6f953f3099729b1fd2973bce27b0b7884212 wpt-pr: 59955

…pover=hint from Chromium to WPT, a=testonly Automatic update from web-platform-tests Attempt to clean up the WPT export of popover=hint from Chromium to WPT (#59955) * Attempt to clean up the WPT export of popover=hint from Chromium to WPT This PR pushes the latest state of `popover=hint` tests from Chromium (prior to the landing of https://crrev.com/c/7851723, which was a temporary stopgap to unbreak the infra) to WPT. This one PR should completely replace any PRs that come out of any still-unlanded PRs in this list: whatwg/html#12345 (comment) * Update test condition descriptions Change-Id: I363b411aa8f6979fbde3766c27afe229703dfc1f Change-Id: Ibd6af063310585b9044de63a06e36ffd79d011d4 Change-Id: I917bf939614f9811470444e9f13c942e5c63b1b1 -- wpt-commits: 3dbd6f953f3099729b1fd2973bce27b0b7884212 wpt-pr: 59955 UltraBlame original commit: a06780bbcfd57a4e6a05d6172f16d29ed039308f

jakearchibald added 4 commits April 8, 2026 13:45

Show popover fixed

04c3483

The rest of the owl

f4714ea

No need to set hideUntil to document

3bca74f

Another document set

dc41ca8

jakearchibald commented Apr 8, 2026

View reviewed changes

Comment thread source Outdated

jakearchibald marked this pull request as ready for review April 8, 2026 14:01

jakearchibald requested review from keithamus, lukewarlow and mfreed7 April 8, 2026 14:01

Missing return true

4518fd0

jakearchibald mentioned this pull request Apr 8, 2026

What should happen if you open an auto popover 'within' a hint popover? #12346

Closed

annevk reviewed Apr 9, 2026

View reviewed changes

This was referenced Apr 9, 2026

Add list slice whatwg/infra#706

Merged

Add list reverse whatwg/infra#707

Merged

jakearchibald commented Apr 9, 2026

View reviewed changes

Comment thread source Outdated

jakearchibald added 5 commits April 9, 2026 11:38

Minor changes

e345ff0

Hint first, and make it clear the conditions are exclusive

8f4f8fd

Actually remove the TODO lol

31a54ee

Create algo for closing hint then auto popovers

afc6003

Fixing silly mistakes

d2fd3f5

jamesnw mentioned this pull request Apr 9, 2026

Update popover=hint to match spec changes oddbird/popover-polyfill#275

Open

keithamus mentioned this pull request Apr 9, 2026

Should the popoverList in "hide popover stack until" and "close entire popover list" be dynamically updated? #11007

Open

annevk pushed a commit to whatwg/infra that referenced this pull request Apr 10, 2026

Add list reverse

7f8c8ee

For whatwg/html#12345.

Don't hide autos if endpoint is in the hint stack

657f638

keithamus reviewed Apr 10, 2026

View reviewed changes

keithamus reviewed May 5, 2026

View reviewed changes

Comment thread source Outdated

Fix case where "showing popover" would be left true

61e7dfc

lukewarlow reviewed May 13, 2026

View reviewed changes

Comment thread source Outdated

lukewarlow reviewed May 13, 2026

View reviewed changes

Comment thread source Outdated

lukewarlow reviewed May 13, 2026

View reviewed changes

Comment thread source Outdated

lukewarlow reviewed May 13, 2026

View reviewed changes

Comment thread source Outdated

lukewarlow reviewed May 13, 2026

View reviewed changes

Comment thread source Outdated

jakearchibald and others added 7 commits May 13, 2026 17:20

Responding to AI feedback

6c6d1a3

Update wrapping

642c567

nits

a05511f

Fix regression in validity check

6cf279d

Fix wrapping

337bc8e

more nits

8ffd53b

more nits

9f36339

annevk merged commit 59e3d96 into main May 14, 2026
2 checks passed

annevk deleted the jakearchibald/popover-hint-simpler-behaviours branch May 14, 2026 09:37

jakearchibald mentioned this pull request May 14, 2026

Note inconsistent popover=hint behaviors in implementations mdn/browser-compat-data#29676

Merged

jonathan-j-lee mentioned this pull request May 15, 2026

Fix a bug in new popover=hint stacking behavior, and update tests web-platform-tests/wpt#59915

Closed

mfreed7 mentioned this pull request May 18, 2026

Attempt to clean up the WPT export of popover=hint from Chromium to WPT web-platform-tests/wpt#59955

Merged

This was referenced May 19, 2026

Editorial: export Document's showing hint and auto popover lists #11140

Closed

Invalid use of "find the topmost popover ancestor" whatwg/fullscreen#245

Closed

lukewarlow mentioned this pull request May 20, 2026

Implement popover=hint WebKit/WebKit#62557

Open

Conversation

jakearchibald commented Apr 8, 2026 • edited by pr-preview Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

jakearchibald commented Apr 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

lukewarlow commented Apr 8, 2026

Uh oh!

jakearchibald commented Apr 8, 2026

Uh oh!

jakearchibald commented Apr 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

jakearchibald commented Apr 9, 2026

Uh oh!

keithamus Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Milestone

Development

Uh oh!

5 participants

jakearchibald commented Apr 8, 2026 •

edited by pr-preview Bot

Loading

jakearchibald commented Apr 8, 2026 •

edited

Loading

jakearchibald commented Apr 8, 2026 •

edited

Loading