Bug 170560 - Scroll offset jumps after a programmatic scroll in an overflow container with scroll snapping
Summary: Scroll offset jumps after a programmatic scroll in an overflow container with...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Wenson Hsieh
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2017-04-06 13:03 PDT by Wenson Hsieh
Modified: 2017-12-13 17:26 PST (History)
9 users (show)

See Also:

Attachments
Reduced test case adapted from <http://output.jsbin.com/fiwiqo> (2.27 KB, text/html)
2017-04-06 13:03 PDT, Wenson Hsieh 		no flags Details
Patch (9.48 KB, patch)
2017-04-06 13:32 PDT, Wenson Hsieh 		thorton: review+
Details | Formatted Diff | Diff
Patch for landing (9.33 KB, patch)
2017-04-06 16:58 PDT, Wenson Hsieh 		no flags Details | Formatted Diff | Diff
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 Wenson Hsieh 2017-04-06 13:03:30 PDT 
Created attachment 306413 [details]
Reduced test case adapted from <http://output.jsbin.com/fiwiqo>

See whsieh.github.io/examples/scroll-snap-fixed-div.html on an iOS device.
Comment 1 Wenson Hsieh 2017-04-06 13:04:42 PDT 
This is the bug in <https://bugs.webkit.org/show_bug.cgi?id=169800> pertaining to overflow scrolling.
Comment 2 Radar WebKit Bug Importer 2017-04-06 13:06:10 PDT 
<rdar://problem/31484693>
Comment 3 Wenson Hsieh 2017-04-06 13:32:44 PDT 
Created attachment 306422 [details]
Patch
Comment 4 Wenson Hsieh 2017-04-06 16:58:41 PDT 
Created attachment 306438 [details]
Patch for landing
Comment 5 WebKit Commit Bot 2017-04-06 17:38:45 PDT 
Comment on attachment 306438 [details]
Patch for landing

Clearing flags on attachment: 306438

Committed r215075: <http://trac.webkit.org/changeset/215075>
Comment 6 Sriram Krishnan 2017-05-31 15:48:38 PDT 
@Wenson Hsieh - looks like the patch failed to apply on IOS  - could some one take a look at this please?
Comment 7 Wenson Hsieh 2017-05-31 16:30:32 PDT 
(In reply to Sriram Krishnan from comment #6)
> @Wenson Hsieh - looks like the patch failed to apply on IOS  - could some
> one take a look at this please?

The "Failed to apply" message is only on an iOS EWS bot.