1843141 - [wpt-sync] Sync PR 40995 - [scroll-start-target] Apply scroll-start-target

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, task, P4)

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Sync web-platform-tests PR 40995 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/40995
Details from upstream follow.

David Awogbemila <[e-mail]> wrote:

[scroll-start-target] Apply scroll-start-target

scroll-start-target[1] makes it possible to specify a scroller's initial
scroll position using one of its descendants. For example:

\<div id="scroller">
\<div>\</div>
\<div id="target" style="scroll-start-target: auto">\</div>
\<div>\</div>
\</div>

Design doc at [2]

Bug:1439807
Change-Id: Ic0f9efbb327f6f670c9f0abae145424b300c6af4
[1]https://drafts.csswg.org/css-scroll-snap-2/#scroll-start-target
[2]https://docs.google.com/document/d/13-WsJ1uL20BQ5M9FTfLh9nCgoERf4neJ4HTkgxf_-BM/edit?resourcekey=0-Ksy-i8w-1xrk3MVlfYrDXg
Reviewed-on: https://chromium-review.googlesource.com/4681305
WPT-Export-Revision: 694ec3d075d8d9421291f88b9ab7b2209a079de1

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=dc064d293768a72fdc04746fd9d88631e97157bd

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

CI Results

Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 12 tests and 2 subtests

Status Summary

Firefox

OK : 11
PASS : 1
FAIL : 16
ERROR : 1
NOTRUN: 1

Chrome

OK : 11
PASS : 2
FAIL : 15
ERROR : 1
NOTRUN: 1

Safari

OK : 11
PASS : 2
FAIL : 15
ERROR : 1
NOTRUN: 1

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

Firefox-only Failures

• /css/css-scroll-snap-2/scroll-start-target/scroll-start-target-with-text-fragment-navigation.tentative.html [wpt.fyi]
  • scroll-start doesn't override text fragment navigation: FAIL

New Tests That Don't Pass

• /css/css-scroll-snap-2/scroll-start-target/scroll-start-target-display-toggled.tentative.html [wpt.fyi]
  • display:block scroll-start-target becomes display: none: FAIL (Chrome: FAIL, Safari: FAIL)
  • display:none scroll-start-target becomes display: block: FAIL (Chrome: FAIL, Safari: FAIL)
• /css/css-scroll-snap-2/scroll-start-target/scroll-start-target-nested-container.tentative.html [wpt.fyi]: ERROR (Chrome: ERROR, Safari: ERROR)
  • display:none scroll-start-target becomes display:block: FAIL (Chrome: FAIL, Safari: FAIL)
  • intermediate overflow:visible container becomes overflow:scroll: FAIL (Chrome: FAIL, Safari: FAIL)
  • inner scroll-start-target takes precedence over outer: FAIL (Chrome: FAIL, Safari: FAIL)
  • scroll containers can also be scroll-start-targets: NOTRUN (Chrome: NOTRUN, Safari: NOTRUN)
• /css/css-scroll-snap-2/scroll-start-target/scroll-start-target-root.tentative.html [wpt.fyi]
  • CSS Scroll Snap 2 Test: scroll-start-target*: FAIL (Chrome: FAIL, Safari: FAIL)
• /css/css-scroll-snap-2/scroll-start-target/scroll-start-target-rtl.tentative.html [wpt.fyi]
  • scroll-start-target reflects vertical rtl writing mode.: FAIL (Chrome: FAIL, Safari: FAIL)
• /css/css-scroll-snap-2/scroll-start-target/scroll-start-target-with-anchor-navigation.tentative.html [wpt.fyi]
  • scroll-start-target does not override anchor navigation.: FAIL (Chrome: FAIL, Safari: FAIL)
• /css/css-scroll-snap-2/scroll-start-target/scroll-start-target-with-scroll-snap.tentative.html [wpt.fyi]
  • CSS Scroll Snap 2 Test: scroll-start-target*: FAIL (Chrome: FAIL, Safari: FAIL)
• /css/css-scroll-snap-2/scroll-start-target/scroll-start-target-with-scroll-start-root.tentative.html [wpt.fyi]
  • CSS Scroll Snap 2 Test: scroll-start-target*: FAIL (Chrome: FAIL, Safari: FAIL)
• /css/css-scroll-snap-2/scroll-start-target/scroll-start-target-with-scroll-start.tentative.html [wpt.fyi]
  • CSS Scroll Snap 2 Test: scroll-start-target*: FAIL (Chrome: FAIL, Safari: FAIL)
• /css/css-scroll-snap-2/scroll-start-target/scroll-start-target-with-text-fragment-navigation.tentative.html [wpt.fyi]
  • scroll-start doesn't override text fragment navigation: FAIL (Chrome: PASS, Safari: PASS)
• /css/css-scroll-snap-2/scroll-start-target/scroll-start-target-with-user-programmatic-scroll.tentative.html [wpt.fyi]
  • scroll-start-target does not override user scroll: FAIL (Chrome: FAIL, Safari: FAIL)
  • scroll-start-target does not override programmatic scroll: FAIL (Chrome: FAIL, Safari: FAIL)
• /css/css-scroll-snap-2/scroll-start-target/scroll-start-target.tentative.html [wpt.fyi]
  • CSS Scroll Snap 2 Test: scroll-start-target*: FAIL (Chrome: FAIL, Safari: FAIL)
  • CSS Scroll Snap 2 Test: scroll-start-target* 1: FAIL (Chrome: FAIL, Safari: FAIL)

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/mozilla-central/rev/d4f642db1af4
[wpt PR 40995] - [scroll-start-target] Apply scroll-start-target, a=testonly
https://hg.mozilla.org/mozilla-central/rev/8dfe2883500f
[wpt PR 40995] - Update wpt metadata, a=testonly

Comment 4

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/d4f642db1af4
https://hg.mozilla.org/mozilla-central/rev/8dfe2883500f