Closed Bug 1928128 Opened 1 year ago Closed 11 months ago

Implementation bug for addition of options to popover APIs and connection to anchor pos

Categories

(Core :: DOM: Core & HTML, enhancement)

Product:
Core
Component:
DOM: Core & HTML
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: masonf, Assigned: keithamus)

References

(Blocks 2 open bugs,
URL
)

Details

(Whiteboard: [anchorpositioning:m2])

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/132.0.0.0 Safari/537.36

Steps to reproduce:

See https://github.com/whatwg/html/pull/10728 for details.

Two changes are being made to the popover API in that PR:

  • The showPopover() and togglePopover() methods now include an options bag that allows setting the popover invoker implicitly.
  • Popover invokers (set declaratively or imperatively) now create an implicit anchor element reference for that popover.
Blocks: popover
Component: Untriaged → DOM: Core & HTML
Product: Firefox → Core
Depends on: 1936411
Assignee: nobody → mozilla
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
See Also: → 1924210
Depends on: 1981121
Points: --- → 3
Whiteboard: [anchorpositioning:m2]
Depends on: 1924210
No longer depends on: 1924210
Blocks: 1924210

We implemented imperative invokers in bug 1936411. Bug 1924210 tracks the layout piece for implicit anchors, so I think the necessary work is done here.

My understanding is that the "implicit anchor" will always be the invoker. Therefore the API would be GetPopoverData()->GetInvoker() (https://searchfox.org/firefox-main/source/dom/base/PopoverData.h#77).

Status: ASSIGNED → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Points: 3 → ---
You need to log in before you can comment on or make changes to this bug.