Bug 237360 - [css-scroll-snap] Improve snapping behavior with incremental directional scrolls
Summary: [css-scroll-snap] Improve snapping behavior with incremental directional scrolls
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Martin Robinson
URL:
Keywords: InRadar
Depends on: 237589
Blocks:
  Show dependency tree / graph
 
Reported: 2022-03-02 04:17 PST by Martin Robinson
Modified: 2022-05-17 02:36 PDT (History)
10 users (show)

See Also:

Attachments
Patch (14.09 KB, patch)
2022-03-03 10:27 PST, Martin Robinson 		no flags Details | Formatted Diff | Diff
Patch (21.38 KB, patch)
2022-03-07 05:51 PST, Martin Robinson 		no flags Details | Formatted Diff | Diff
Patch (10.91 KB, patch)
2022-04-19 08:38 PDT, Martin Robinson 		no flags Details | Formatted Diff | Diff
Patch (13.61 KB, patch)
2022-04-20 00:38 PDT, Martin Robinson 		no flags Details | Formatted Diff | Diff
Show Obsolete (3) 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 Martin Robinson 2022-03-02 04:17:49 PST 
Under certain circumstances, when a snap area covers the viewport the user should be able to scroll into the middle of the area without triggering a snap. An exception to this rule is when doing a directional snap into the large scroll area. In that case, if there is an intermediate snap point (one from another element) that is between the start and end of the large snap area, the scroll should not skip past this intermediate point.
Comment 1 Martin Robinson 2022-03-03 10:27:33 PST 
Created attachment 453757 [details]
Patch
Comment 2 EWS Watchlist 2022-03-03 10:30:18 PST 
This patch modifies the imported WPT tests. Please ensure that any changes on the tests (not coming from a WPT import) are exported to WPT. Please see https://trac.webkit.org/wiki/WPTExportProcess
Comment 3 Martin Robinson 2022-03-04 02:58:02 PST 
Comment on attachment 453757 [details]
Patch

Clearing flags. The script used to focus elements for key events needs a bit more work.
Comment 4 Martin Robinson 2022-03-07 05:51:00 PST 
Created attachment 453968 [details]
Patch
Comment 5 Radar WebKit Bug Importer 2022-03-09 04:18:16 PST 
<rdar://problem/90024215>
Comment 6 Martin Robinson 2022-03-09 04:19:44 PST 
I have split out parts of this change into bug 237589 and will update the patch here once that lands.
Comment 7 Martin Robinson 2022-04-19 08:38:42 PDT 
Created attachment 457896 [details]
Patch
Comment 8 Martin Robinson 2022-04-20 00:38:24 PDT 
Created attachment 457966 [details]
Patch
Comment 9 Martin Robinson 2022-05-17 02:36:09 PDT 
Pull request: https://github.com/WebKit/WebKit/pull/668