Bug 236244

Summary: No animation when scroll snap scroller is navigated with the keyboard
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: ScrollingAssignee: Simon Fraser (smfr) <simon.fraser>
Status: RESOLVED FIXED    
Severity: Normal CC: changseok, esprehn+autocc, ews-watchlist, fred.wang, glenn, kondapallykalyan, pdr, simon.fraser, thorton, webkit-bug-importer
Priority: P2 Keywords: BrowserCompat, InRadar
Version: Safari Technology Preview   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=234456
Attachments:
Description Flags
Patch
thorton: review+, ews-feeder: commit-queue-
Patch
ews-feeder: commit-queue-
Patch none

Simon Fraser (smfr)
Reported 2022-02-07 08:49:46 PST
https://css-tricks.com/css-scroll-snap-slide-deck/ describes a slide deck using scroll snap for the slides. In the result <https://codepen.io/5t3ph/full/yLzQeGr/652097fa9ce1150aeb5400637ab91b63>, moving between slides via the keyboard doesn't animate in Safari, but odes in Chrome and Firefox.
Attachments
Patch (2.52 KB, patch)
2022-02-25 17:19 PST, Simon Fraser (smfr)
thorton: review+
ews-feeder: commit-queue-
Patch (5.01 KB, patch)
2022-02-28 10:12 PST, Simon Fraser (smfr)
ews-feeder: commit-queue-
Patch (7.65 KB, patch)
2022-02-28 14:41 PST, Simon Fraser (smfr)
no flags
Radar WebKit Bug Importer
Comment 1 2022-02-07 08:50:12 PST
Simon Fraser (smfr)
Comment 2 2022-02-25 17:19:35 PST
Simon Fraser (smfr)
Comment 3 2022-02-26 10:43:28 PST
Simon Fraser (smfr)
Comment 4 2022-02-28 10:07:21 PST
Simon Fraser (smfr)
Comment 5 2022-02-28 10:12:48 PST
Simon Fraser (smfr)
Comment 6 2022-02-28 12:10:52 PST
I think the tests are passing for me because the default value for ScrollAnimatorEnabled comes from the NSScrollAnimationEnabled default.
Simon Fraser (smfr)
Comment 7 2022-02-28 12:38:56 PST
Also NSScrollAnimationEnabled is 1 when read in the web process, and 0 when read in the UI process (when there's a stale default left around)
Simon Fraser (smfr)
Comment 8 2022-02-28 14:41:36 PST
EWS
Comment 9 2022-02-28 19:01:24 PST
Committed r290625 (247898@main): <https://commits.webkit.org/247898@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 453433 [details].
Note You need to log in before you can comment on or make changes to this bug.