[whatwg/fullscreen] Invalid use of "find the topmost popover ancestor" (Issue #245)

theIDinside created an issue (whatwg/fullscreen#245)

Recently a new PR was pulled in:  [Add popover=hint #9778](https://github.com/whatwg/html/pull/9778)

It adds another argument to the [find the topmost popover ancestor](https://html.spec.whatwg.org/multipage/popover.html#topmost-popover-ancestor):

1. a [Node](https://dom.spec.whatwg.org/#interface-node) newPopoverOrTopLayerElement
2. a [list](https://infra.spec.whatwg.org/#list) popoverList <- this one
3. an [HTML element](https://html.spec.whatwg.org/multipage/infrastructure.html#html-elements) or null invoker
4. a boolean isPopover

However, to [fullscreen an element](https://fullscreen.spec.whatwg.org/#fullscreen-an-element) we are still only using the old three argument-version.

@josepharhar wrote the pull request for the new popover attribute value so I guess my question is what would be the parameter for the list in the Fullscreen API scenario? I'm not well versed in how popover works or is intended to work, but I take that the specific change for the PR is to add tooltip-like stuff, and so, I wonder, should all these be auto closed and only have the "manual" popovers left open or should it be auto? But that's just my initial guesses after having peaked at it for 5 minutes.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/fullscreen/issues/245
You are receiving this because you are subscribed to this thread.

Message ID: <whatwg/fullscreen/issues/245@github.com>