[wpt-sync] Sync PR 40929 - [anchor-position] Make 'anchor-scroll' internal
Categories
(Core :: CSS Parsing and Computation, task, P4)
Tracking
()
Tracking | Status | |
---|---|---|
firefox117 | --- | fixed |
People
(Reporter: wpt-sync, Unassigned)
References
(Blocks 1 open bug, )
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 40929 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/40929
Details from upstream follow.
Xiaocheng Hu <xiaochengh@chromium.org> wrote:
[anchor-position] Make 'anchor-scroll' internal
See https://github.com/w3c/csswg-drafts/issues/8675 for context.
This patch:
Changes the implementation to longer expose
anchor-scroll
as a
CSS property, but use the target ofanchor-default
for scrolling
purposesCoverts tests and UA style to stop using
anchor-scroll
. The usage
is either simply removed if there's already a default anchor, or
changed to useanchor-default
otherwise. The cases that assume
differentanchor-scroll
andanchor-default
targets are removed
since they will no longer be a possible.To minimize diff, this patch focuses on the implementation change.
A follow up patch will convert all in-code documentations and rename
existing classes/functions to stop referring to the functionality as
"anchor-scroll", and hence, stop the naming confusion.Bug: 1463085
Change-Id: I3cb2000de67b2b81b4edf8b9a1303de682946232
Reviewed-on: https://chromium-review.googlesource.com/4662290
WPT-Export-Revision: cd6a193d577fea5a41456dbe6dc4f3ef9873a2f8
Assignee | ||
Comment 1•1 year ago
|
||
PR 40929 applied with additional changes from upstream: 4733ae5e5590fec37db69778aaaf5ef4ce84ee61
Assignee | ||
Updated•1 year ago
|
Assignee | ||
Comment 2•1 year ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=deb6bcb1d2b61636691b70954ab311f1abd7911c
Assignee | ||
Comment 3•1 year ago
|
||
CI Results
Ran 7 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI
Total 41 tests
Status Summary
Firefox
OK
: 16
PASS
: 6
FAIL
: 68[GitHub] 69[Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview, Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-windows11-32-2009-qr-opt, Gecko-windows11-64-2009-qr-opt]
TIMEOUT
: 1
Chrome
OK
: 16
PASS
: 73
FAIL
: 2
Safari
OK
: 16
PASS
: 6
FAIL
: 69
Links
Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base
Details
New Tests That Don't Pass
- /css/css-anchor-position/anchor-position-top-layer-001.html [wpt.fyi]:
FAIL
(Chrome:PASS
, Safari:FAIL
) - /css/css-anchor-position/anchor-position-top-layer-002.html [wpt.fyi]:
FAIL
(Chrome:PASS
, Safari:FAIL
) - /css/css-anchor-position/anchor-position-top-layer-003.html [wpt.fyi]:
FAIL
(Chrome:PASS
, Safari:FAIL
) - /css/css-anchor-position/anchor-position-top-layer-004.html [wpt.fyi]:
FAIL
(Chrome:PASS
, Safari:FAIL
) - /css/css-anchor-position/anchor-position-top-layer-005.html [wpt.fyi]:
FAIL
(Chrome:PASS
, Safari:FAIL
) - /css/css-anchor-position/anchor-position-top-layer-006.html [wpt.fyi]:
FAIL
(Chrome:PASS
, Safari:FAIL
) - /css/css-anchor-position/anchor-scroll-001.html [wpt.fyi]:
FAIL
(Chrome:PASS
, Safari:FAIL
) - /css/css-anchor-position/anchor-scroll-composited-scrolling-006.html [wpt.fyi]:
FAIL
(Chrome:PASS
, Safari:FAIL
) - /css/css-anchor-position/anchor-scroll-fixedpos.html [wpt.fyi]:
FAIL
(Chrome:PASS
, Safari:FAIL
) - /css/css-anchor-position/anchor-scroll-nested.html [wpt.fyi]:
FAIL
(Chrome:PASS
, Safari:FAIL
) - /css/css-anchor-position/anchor-scroll-update-001.html [wpt.fyi]:
FAIL
(Chrome:PASS
, Safari:FAIL
) - /css/css-anchor-position/anchor-scroll-update-002.html [wpt.fyi]:
FAIL
(Chrome:PASS
, Safari:FAIL
) - /css/css-anchor-position/anchor-scroll-update-003.html [wpt.fyi]:
FAIL
(Chrome:PASS
, Safari:FAIL
) - /css/css-anchor-position/anchor-scroll-update-004.html [wpt.fyi]:
FAIL
(Chrome:PASS
, Safari:FAIL
) - /css/css-anchor-position/anchor-scroll-update-005.html [wpt.fyi]:
FAIL
(Chrome:PASS
, Safari:FAIL
) - /css/css-anchor-position/anchor-scroll-update-006.html [wpt.fyi]:
FAIL
(Chrome:PASS
, Safari:FAIL
) - /css/css-anchor-position/anchor-scroll-update-007.html [wpt.fyi]:
FAIL
(Chrome:PASS
, Safari:FAIL
) - /css/css-anchor-position/anchor-scroll-vlr.html [wpt.fyi]:
FAIL
(Chrome:PASS
, Safari:FAIL
) - /css/css-anchor-position/anchor-scroll-vrl.html [wpt.fyi]:
FAIL
(Chrome:PASS
, Safari:FAIL
) - /html/semantics/popovers/popover-anchor-scroll-display.html [wpt.fyi]:
FAIL
[Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview
,Gecko-android-em-7.0-x86_64-qr-debug-geckoview
,Gecko-android-em-7.0-x86_64-qr-opt-geckoview
,Gecko-linux1804-64-qr-debug
,Gecko-linux1804-64-qr-opt
,Gecko-windows11-32-2009-qr-opt
,Gecko-windows11-64-2009-qr-opt
],TIMEOUT
[GitHub
] (Chrome:FAIL
, Safari:FAIL
) - /css/css-anchor-position/anchor-scroll-002.html [wpt.fyi]
- target1 should scroll with anchor1:
FAIL
(Chrome:PASS
, Safari:FAIL
) - target2 should scroll with anchor2:
FAIL
(Chrome:PASS
, Safari:FAIL
) - target3 should scroll with anchor3:
FAIL
(Chrome:PASS
, Safari:FAIL
)
- target1 should scroll with anchor1:
- /css/css-anchor-position/anchor-scroll-003.html [wpt.fyi]
- Scrolling should work in fragmented containing block:
FAIL
(Chrome:PASS
, Safari:FAIL
)
- Scrolling should work in fragmented containing block:
- /css/css-anchor-position/anchor-scroll-004.html [wpt.fyi]
- Initial position of the targets:
FAIL
(Chrome:PASS
, Safari:FAIL
) - #target1 should scroll with #anchor1:
FAIL
(Chrome:PASS
, Safari:FAIL
) - #target2 should scroll with #anchor2:
FAIL
(Chrome:PASS
, Safari:FAIL
)
- Initial position of the targets:
- /css/css-anchor-position/anchor-scroll-005.html [wpt.fyi]
- Target should not scroll with viewport when anchor is in fixed-positioned scroller:
FAIL
(Chrome:FAIL
, Safari:FAIL
)
- Target should not scroll with viewport when anchor is in fixed-positioned scroller:
- /css/css-anchor-position/anchor-scroll-fallback-position-001.html [wpt.fyi]
- Should be above the anchor when at initial scroll position:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Scroll down until the top edge of #anchor touches container but not overflowing:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Scroll further down, making the first fallback position overflow by 1px:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Scroll back up to reuse the first fallback position:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Scroll bottom-right to make the first three fallback positions overflow:
FAIL
(Chrome:PASS
, Safari:FAIL
)
- Should be above the anchor when at initial scroll position:
- /css/css-anchor-position/anchor-scroll-fallback-position-002.html [wpt.fyi]
- Should use the first fallback position at the initial scroll offset:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Should use the second fallback position after scrolling left:
FAIL
(Chrome:PASS
, Safari:FAIL
)
- Should use the first fallback position at the initial scroll offset:
- /css/css-anchor-position/anchor-scroll-fallback-position-003.html [wpt.fyi]
- Should use the first fallback position at the initial scroll offset:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Should use the second fallback position after scrolling up:
FAIL
(Chrome:PASS
, Safari:FAIL
)
- Should use the first fallback position at the initial scroll offset:
- /css/css-anchor-position/anchor-scroll-fallback-position-004.html [wpt.fyi]
- Should use the first fallback position at the initial scroll offsets:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Should use the second fallback position after scrolling viewport down:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Should use the third fallback position after scrolling the vrl scroller left:
FAIL
(Chrome:PASS
, Safari:FAIL
)
- Should use the first fallback position at the initial scroll offsets:
- /css/css-anchor-position/anchor-scroll-fallback-position-005.html [wpt.fyi]
- Should use the first fallback position at the initial scroll offset:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Should use the second fallback position after scrolling left:
FAIL
(Chrome:PASS
, Safari:FAIL
)
- Should use the first fallback position at the initial scroll offset:
- /css/css-anchor-position/anchor-scroll-fallback-position-006.html [wpt.fyi]
- Should use the last fallback position initially:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Should use the third fallback position with enough space below:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Should use the second fallback position with enough space right:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Should use the first fallback position with enough space below and right:
FAIL
(Chrome:PASS
, Safari:FAIL
)
- Should use the last fallback position initially:
- /css/css-anchor-position/anchor-scroll-fallback-position-007.html [wpt.fyi]
- Should use the last fallback position initially:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Should use the third fallback position with enough space left:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Should use the second fallback position with enough space below:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Should use the first fallback position with enough space left and below:
FAIL
(Chrome:PASS
, Safari:FAIL
)
- Should use the last fallback position initially:
- /css/css-anchor-position/anchor-scroll-fallback-position-008.html [wpt.fyi]
- Should use the last fallback position initially:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Should use the third fallback position with enough space left:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Should use the second fallback position with enough space above:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Should use the first fallback position with enough space left and above:
FAIL
(Chrome:PASS
, Safari:FAIL
)
- Should use the last fallback position initially:
- /css/css-anchor-position/anchor-scroll-fallback-position-009.html [wpt.fyi]
- Should use the last fallback position initially:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Should use the third fallback position with enough space right:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Should use the second fallback position with enough space below:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Should use the first fallback position with enough space right and below:
FAIL
(Chrome:PASS
, Safari:FAIL
)
- Should use the last fallback position initially:
- /css/css-anchor-position/anchor-scroll-fallback-position-010.html [wpt.fyi]
- Should use the last fallback position initially:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Should use the third fallback position with enough space right:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Should use the second fallback position with enough space above:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Should use the first fallback position with enough space right and above:
FAIL
(Chrome:PASS
, Safari:FAIL
)
- Should use the last fallback position initially:
- /css/css-anchor-position/anchor-scroll-fallback-position-011.html [wpt.fyi]
- Should use the last fallback position initially:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Should use the third fallback position with enough space above:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Should use the second fallback position with enough space right:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Should use the first fallback position with enough space above and right:
FAIL
(Chrome:PASS
, Safari:FAIL
)
- Should use the last fallback position initially:
- /css/css-anchor-position/anchor-scroll-js-expose.html [wpt.fyi]
- Element.getBoundingClientRect() returns the actual rendered location:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Range.getBoundingClientRect() returns the actual rendered location:
FAIL
(Chrome:PASS
, Safari:FAIL
) - Element.offset* return adjusted offsets:
FAIL
(Chrome:PASS
, Safari:FAIL
)
- Element.getBoundingClientRect() returns the actual rendered location:
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/849e1d77375f [wpt PR 40929] - [anchor-position] Make 'anchor-scroll' internal, a=testonly https://hg.mozilla.org/integration/autoland/rev/780149048992 [wpt PR 40929] - Update wpt metadata, a=testonly
Comment 5•1 year ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/849e1d77375f
https://hg.mozilla.org/mozilla-central/rev/780149048992
Description
•