1938042 - [wpt-sync] Sync PR 49755 - [carousel] Focus should follow the active marker when scrolling

Status: RESOLVED FIXED

(Core :: Layout: Scrolling and Overflow, task, P4)

Description

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

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

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

Daniil Sakhapov <[e-mail]> wrote:

[carousel] Focus should follow the active marker when scrolling

In the spec, when updating the active marker the following is meant
to ensure that if the previously active marker was focused we update
the focus to the new active marker,

From https://drafts.csswg.org/css-overflow-5/#active-scroll-marker
"If active marker was the last-focused element of the group,
focus active marker".

Fixed: 384391190
Change-Id: If629a73fae1613f0c12b0f9cff796e46407890c0
Reviewed-on: https://chromium-review.googlesource.com/6097240
WPT-Export-Revision: 05218b8396c985e4eb7746dfd39d1958e4e7da30

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=c2ed530d5a86001da8f400f1cbbde5edfa19375d

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 1 tests and 2 subtests

Status Summary

Firefox

OK : 1
FAIL: 2

Chrome

OK : 1
PASS: 1
FAIL: 1

Safari

OK : 1
FAIL: 2

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

• /css/css-overflow/scroll-markers-focus-on-scrolling.html [wpt.fyi]
  • active ::scroll-marker doesn't have focus on scroll if previous ::scroll-marker didn't have it: FAIL (Chrome: PASS, Safari: FAIL)
  • active ::scroll-marker saves focus on scroll if previous ::scroll-marker had it: FAIL (Chrome: FAIL, Safari: FAIL)

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2685c21afbb5
[wpt PR 49755] - [carousel] Focus should follow the active marker when scrolling, a=testonly
https://hg.mozilla.org/integration/autoland/rev/a8f1e6a715cf
[wpt PR 49755] - Update wpt metadata, a=testonly

Comment 4

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/2685c21afbb5
https://hg.mozilla.org/mozilla-central/rev/a8f1e6a715cf