Bug 224380

Summary: [Scroll snap] proximity scroll snapping is broken
Product: WebKit Reporter: Simon Fraser (smfr) <simon.fraser>
Component: ScrollingAssignee: Martin Robinson <mrobinson>
Status: RESOLVED WORKSFORME    
Severity: Normal CC: cmarcelo, ews-watchlist, fred.wang, jamesr, luiz, mrobinson, simon.fraser, tonikitoo, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Safari Technology Preview   
Hardware: Unspecified   
OS: Unspecified   
URL: https://codepen.io/argyleink/pen/qBRpdEr
Bug Depends on: 224326    
Bug Blocks:    
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch none

Simon Fraser (smfr)
Reported 2021-04-09 11:13:44 PDT
This demo doesn't work correctly in Safari.
Attachments
Patch (33.46 KB, patch)
2021-05-05 07:25 PDT, Martin Robinson 		no flags
DetailsFormatted DiffDiff
Patch (45.50 KB, patch)
2021-05-13 11:17 PDT, Martin Robinson 		no flags
DetailsFormatted DiffDiff
Patch (42.89 KB, patch)
2021-05-13 13:04 PDT, Martin Robinson 		no flags
DetailsFormatted DiffDiff
Patch (38.31 KB, patch)
2021-05-14 07:06 PDT, Martin Robinson 		no flags
DetailsFormatted DiffDiff
Show Obsolete (4) View All Add attachment proposed patch, testcase, etc.
Simon Fraser (smfr)
Comment 1 2021-04-09 11:14:40 PDT
Hard to get away from the first snap point. The scrolling stops dead between two snap points.
Radar WebKit Bug Importer
Comment 2 2021-04-09 11:15:34 PDT
<rdar://problem/76461059>
Martin Robinson
Comment 3 2021-05-05 07:25:50 PDT
Created attachment 427764 [details] Patch
Martin Robinson
Comment 4 2021-05-13 11:17:57 PDT
Created attachment 428538 [details] Patch
Martin Robinson
Comment 5 2021-05-13 13:04:55 PDT
Created attachment 428551 [details] Patch
Martin Robinson
Comment 6 2021-05-14 07:06:18 PDT
Created attachment 428618 [details] Patch
Martin Robinson
Comment 7 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.
Simon Fraser (smfr)
Comment 8 2021-12-20 10:15:20 PST
Is this bug still valid?
Martin Robinson
Comment 9 2021-12-22 06:01:00 PST
I think this one can be closed. The linked demo seems to work fine for me now.
Note You need to log in before you can comment on or make changes to this bug.