Bug 224380 - [Scroll snap] proximity scroll snapping is broken
Summary: [Scroll snap] proximity scroll snapping is broken
Status: RESOLVED WORKSFORME
Alias: None
Product: WebKit
Classification: Unclassified
Component: Scrolling (show other bugs)
Version: Safari Technology Preview
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Martin Robinson
URL: https://codepen.io/argyleink/pen/qBRpdEr
Keywords: InRadar
Depends on: 224326
Blocks:
  Show dependency tree / graph
 
Reported: 2021-04-09 11:13 PDT by Simon Fraser (smfr)
Modified: 2021-12-22 06:01 PST (History)
9 users (show)

See Also:

Attachments
Patch (33.46 KB, patch)
2021-05-05 07:25 PDT, Martin Robinson 		no flags Details | Formatted Diff | Diff
Patch (45.50 KB, patch)
2021-05-13 11:17 PDT, Martin Robinson 		no flags Details | Formatted Diff | Diff
Patch (42.89 KB, patch)
2021-05-13 13:04 PDT, Martin Robinson 		no flags Details | Formatted Diff | Diff
Patch (38.31 KB, patch)
2021-05-14 07:06 PDT, Martin Robinson 		no flags Details | Formatted Diff | Diff
Show Obsolete (4) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Simon Fraser (smfr) 2021-04-09 11:13:44 PDT 
This demo doesn't work correctly in Safari.
Comment 1 Simon Fraser (smfr) 2021-04-09 11:14:40 PDT 
Hard to get away from the first snap point. The scrolling stops dead between two snap points.
Comment 2 Radar WebKit Bug Importer 2021-04-09 11:15:34 PDT 
<rdar://problem/76461059>
Comment 3 Martin Robinson 2021-05-05 07:25:50 PDT 
Created attachment 427764 [details]
Patch
Comment 4 Martin Robinson 2021-05-13 11:17:57 PDT 
Created attachment 428538 [details]
Patch
Comment 5 Martin Robinson 2021-05-13 13:04:55 PDT 
Created attachment 428551 [details]
Patch
Comment 6 Martin Robinson 2021-05-14 07:06:18 PDT 
Created attachment 428618 [details]
Patch
Comment 7 Martin Robinson 2021-06-24 00:13:49 PDT 
I've reconsidered the previous patch. I think it's more likely that momentum scrolling on Mac should *always* escape the snap point. This matches behavior on iOS now.
Comment 8 Simon Fraser (smfr) 2021-12-20 10:15:20 PST 
Is this bug still valid?
Comment 9 Martin Robinson 2021-12-22 06:01:00 PST 
I think this one can be closed. The linked demo seems to work fine for me now.