1867737 - [popover] Popover wpt test failures due to lack of Anchor positioning support

Status: NEW

(Core :: Layout, defect)

Description

[zsun]

We have the following popover wpt tests that use anchor positioning:

popover-anchor-change-display.tentative.html
popover-anchor-display-none.tentative.html
popover-anchor-display.tentative.html
popover-anchor-idl-property.tentative.html
popover-anchor-multicol-display.tentative.html
popover-anchor-nested-display.tentative.html
popover-anchor-scroll-display.tentative.html
popover-anchor-transition.tentative.tentative.html
popover-stacking.html
popover-light-dismiss-with-anchor.tentative.tentative.html
popover-shadow-dom.html(one sub-test)

Comment 3

[Jonathan Watt [:jwatt]]

Gosh, we have tests already? Don't Google and Apple still need to resolve their disagreements about how anchor positioning should work?

And presumably this should live in the 'Layout' component.

Comment 4

[Jonathan Watt [:jwatt]]

Most of the current popover-anchor-* tests now pass. The list of currently failing tests is:

• popover-anchor-backdrop-transition.html
• popover-anchor-change-display.tentative.html
• popover-anchor-display.tentative.html
• popover-anchor-idl-property.tentative.html
• popover-anchor-inset-rule-display.tentative.html
• popover-anchor-multicol-display.tentative.html
• popover-anchor-nested-display.tentative.html
• popover-anchor-scroll-display.tentative.html

Comment 5

[Jonathan Watt [:jwatt]]

(In reply to Jonathan Watt [:jwatt] from comment #4)

• popover-anchor-backdrop-transition.html

We don't support animations/transitions.

• popover-anchor-change-display.tentative.html

Uses the non-standard "anchor" attribute, which only Chrome has (behind a flag). Unsurprisingly Firefox and Safari Tech Preview both fail this in almost the same way.

• popover-anchor-display.tentative.html

Uses the non-standard "anchor" attribute. Unsurprisingly Firefox and Safari Tech Preview both fail this in almost the same way.

• popover-anchor-idl-property.tentative.html

Depends on the non-standard "anchor" attribute and on the non-standard anchorElement IDL property.

• popover-anchor-inset-rule-display.tentative.html

Depends on the non-standard "anchor" attribute. Also seems to fail in the same way in Chrome Canary and Safari Tech Preview.

• popover-anchor-multicol-display.tentative.html

Again, depends on the non-standard "anchor" attribute.

• popover-anchor-nested-display.tentative.html

Again, depends on the non-standard "anchor" attribute.

• popover-anchor-scroll-display.tentative.html

Again, depends on the non-standard "anchor" attribute.

Comment 6

[Jonathan Watt [:jwatt]]

Full list of popover failing tests:

https://searchfox.org/firefox-main/source/testing/web-platform/meta/html/semantics/popovers

Comment 7

[Jonathan Watt [:jwatt]]

https://searchfox.org/firefox-main/source/testing/web-platform/tests/html/semantics/popovers/popover-alignment-002.html

Probably needs bug 1909339 and bug 1991211.

Comment 8

[Jonathan Watt [:jwatt]]

Still fails, but also using the "anchor" attribute:

https://searchfox.org/firefox-main/source/testing/web-platform/tests/html/semantics/popovers/popover-light-dismiss-with-anchor.tentative.html