RESOLVED FIXED 146957
Scroll snapping to elements is broken for main frame scrolling 
https://bugs.webkit.org/show_bug.cgi?id=146957
Summary Scroll snapping to elements is broken for main frame scrolling
Wenson Hsieh
Reported 2015-07-14 21:26:41 PDT
Setting the mainframe to have scroll snap type mandatory and some of its children to have scroll snap coordinates does not result in scroll snapping behavior (even on a toy example).
Attachments
Patch (8.13 KB, patch)
2015-07-15 12:26 PDT, Wenson Hsieh 		simon.fraser: review+
DetailsFormatted DiffDiff
Patch (8.13 KB, patch)
2015-07-15 13:54 PDT, Wenson Hsieh 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Wenson Hsieh
Comment 1 2015-07-14 21:28:18 PDT
<rdar://problem/21823835>
Wenson Hsieh
Comment 2 2015-07-15 12:26:24 PDT
Created attachment 256849 [details] Patch
Simon Fraser (smfr)
Comment 3 2015-07-15 13:28:18 PDT
Comment on attachment 256849 [details] Patch r+ but please fix the broken builds.
Wenson Hsieh
Comment 4 2015-07-15 13:54:07 PDT
Created attachment 256858 [details] Patch
Brent Fulgham
Comment 5 2015-07-15 14:31:11 PDT
Comment on attachment 256858 [details] Patch r=me, based on Simon's review and the fact that this is building properly.
WebKit Commit Bot
Comment 6 2015-07-15 15:34:58 PDT
Comment on attachment 256858 [details] Patch Clearing flags on attachment: 256858 Committed r186865: <http://trac.webkit.org/changeset/186865>
Simon Fraser (smfr)
Comment 7 2020-04-14 10:11:33 PDT
Comment on attachment 256858 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=256858&action=review > Source/WebCore/rendering/RenderBox.cpp:5008 > + if (document().body() && frame().mainFrame().view() && frame().mainFrame().view()->isScrollable()) This was wrong to consult the main frame.
Note You need to log in before you can comment on or make changes to this bug.