Skip to content

FF153 Popover API docs for new hint/auto popover interaction model - #44603

Merged
hamishwillee merged 4 commits into
mdn:mainfrom
hamishwillee:ff153_popover_auto_hint_interactions
Jul 3, 2026
Merged

hamishwillee merged 4 commits into
mdn:mainfrom
hamishwillee:ff153_popover_auto_hint_interactions

Conversation

@hamishwillee

Copy link
Copy Markdown
Collaborator

The popover spec added support for some new popover hint/auto interaction in whatwg/html#12345 which will be deployed in FF1153, and likely in Chrome 150. In summary:

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

To address this I have added a summary of the auto/hint stack stuff in the using guide ## Popover open/close interaction rules and also updated the docs in various points to capture this.

@jakearchibald I would really appreciate your review of this. I've used a combination of review, testing, and chatting to Claude to work out how this works. Its the last part that concerns me - sometimes Claude makes statements that seem quite reasonable based on what you're seeing in other comments - but that miss a nuance or 10. I'll add a few inline comments too.

Related docs work can be tracked in #44461

@hamishwillee
hamishwillee requested review from a team as code owners June 30, 2026 07:25
@hamishwillee
hamishwillee requested review from chrisdavidmills and estelle and removed request for a team June 30, 2026 07:25
@github-actions github-actions Bot added Content:HTML Hypertext Markup Language docs Content:WebAPI Web API docs size/m [PR only] 51-500 LoC changed labels Jun 30, 2026
Comment thread files/en-us/web/api/popover_api/using/index.md
@github-actions

github-actions Bot commented Jun 30, 2026 •

Copy link
Copy Markdown
Contributor

Comment thread files/en-us/web/api/popover_api/using/index.md

@jakearchibald jakearchibald left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hope this clears things up! It's definitely a complicated feature, but it's simpler than it was previously.

Comment thread files/en-us/web/api/htmlelement/hidepopover/index.md Outdated
Comment thread files/en-us/web/api/htmlelement/showpopover/index.md Outdated
Comment thread files/en-us/web/api/htmlelement/togglepopover/index.md Outdated
Comment thread files/en-us/web/api/popover_api/using/index.md Outdated
Comment thread files/en-us/web/api/popover_api/using/index.md Outdated
Comment thread files/en-us/web/api/popover_api/using/index.md Outdated

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

@hamishwillee, all seems reasonable to me. Just one small suggestion for you.

Comment thread files/en-us/web/api/popover_api/using/index.md Outdated
hamishwillee and others added 3 commits July 3, 2026 10:17
Co-authored-by: Chris Mills <chrisdavidmills@gmail.com>
Co-authored-by: Jake Archibald <jaffathecake@gmail.com>
Co-authored-by: Jake Archibald <jaffathecake@gmail.com>
Co-authored-by: Hamish Willee <hamishwillee@gmail.com>
@hamishwillee

Copy link
Copy Markdown
Collaborator Author

Thanks @jakearchibald and @chrisdavidmills, I've accepted the changes and will merge. I would kind of like to have included the stack diagram you showed in https://github.com/mdn/content/pull/44603/changes#diff-0b8c2975de757a2dda2a803d20fbde1a7a7a5191123441f441816959b70f90d4R375 but I'm not sure I can do so without adding a lot more words that I am "not sure of". Best to get this in, and we might iterate in future.

@hamishwillee
hamishwillee merged commit 5829079 into mdn:main Jul 3, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:HTML Hypertext Markup Language docs Content:WebAPI Web API docs size/m [PR only] 51-500 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants