RESOLVED FIXED 236244
No animation when scroll snap scroller is navigated with the keyboard 
https://bugs.webkit.org/show_bug.cgi?id=236244
Summary No animation when scroll snap scroller is navigated with the keyboard
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-
DetailsFormatted DiffDiff
Patch (5.01 KB, patch)
2022-02-28 10:12 PST, Simon Fraser (smfr) 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Patch (7.65 KB, patch)
2022-02-28 14:41 PST, Simon Fraser (smfr) 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2022-02-07 08:50:12 PST
<rdar://problem/88574200>
Simon Fraser (smfr)
Comment 2 2022-02-25 17:19:35 PST
Created attachment 453279 [details] Patch
Simon Fraser (smfr)
Comment 3 2022-02-26 10:43:28 PST
https://trac.webkit.org/changeset/290548/webkit
Simon Fraser (smfr)
Comment 4 2022-02-28 10:07:21 PST
Reverted in https://trac.webkit.org/changeset/290604/webkit
Simon Fraser (smfr)
Comment 5 2022-02-28 10:12:48 PST
Created attachment 453402 [details] Patch
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
Created attachment 453433 [details] Patch
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.