Bug 142706

Summary: scroll snap points do not properly account for zoomed pages
Product: WebKit Reporter: Brent Fulgham <bfulgham>
Component: Layout and RenderingAssignee: Brent Fulgham <bfulgham>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, cmarcelo, commit-queue, jamesr, luiz, thorton, tonikitoo, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=142705
Attachments:
Description Flags
Patch andersca: review+

Brent Fulgham
Reported 2015-03-14 21:07:14 PDT
Zooming a page with scroll snap points causes the snap point logic to do the wrong thing. We should be properly dealing with zoomed state when searching for the nearest snap point, and when animating to the target location.
Attachments
Patch (4.27 KB, patch)
2015-03-14 23:01 PDT, Brent Fulgham 		andersca: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2015-03-14 21:09:41 PDT
<rdar://problem/20165771>
Brent Fulgham
Comment 2 2015-03-14 22:58:27 PDT
When computing snap point targets, we need to account for the current page zoom. This is currently ignored.
Brent Fulgham
Comment 3 2015-03-14 23:01:21 PDT
Created attachment 248673 [details] Patch
Brent Fulgham
Comment 4 2015-03-15 12:14:07 PDT
Committed r181510: <http://trac.webkit.org/changeset/181510>
Note You need to log in before you can comment on or make changes to this bug.