Summary: | CSS Scroll Snap does not work if scrollbar is hidden | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Rômulo Fernandes <romulo.fernandes> | ||||||
Component: | CSS | Assignee: | Wenson Hsieh <wenson_hsieh> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | bdakin, bfulgham, commit-queue, simon.fraser, sriramkrish85, webkit-bug-importer, wenson_hsieh | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Mac | ||||||||
OS: | OS X 10.11 | ||||||||
Attachments: |
|
Description
Rômulo Fernandes
2016-08-01 21:15:52 PDT
is this being worked on? Huh, looks like <http://trac.webkit.org/changeset/209087> also fixed this issue. I'll add a LayoutTest covering this soon. (In reply to comment #2) > Huh, looks like <http://trac.webkit.org/changeset/209087> also fixed this > issue. > I'll add a LayoutTest covering this soon. ^ I meant to write: <https://trac.webkit.org/changeset/209070> (In reply to comment #3) > (In reply to comment #2) > > Huh, looks like <http://trac.webkit.org/changeset/209087> also fixed this > > issue. > > I'll add a LayoutTest covering this soon. > > ^ I meant to write: <https://trac.webkit.org/changeset/209070> This is actually wrong. I tested this out in mainframe scroll snapping and saw that it worked when hiding scrollbars, but I tried again with overflow scrolling and found that it is still broken when hiding scrollbars. I figured it out after some investigation, and I'll have a patch up momentarily. Created attachment 297668 [details]
Patch
Created attachment 297672 [details]
Patch for landing
Comment on attachment 297672 [details] Patch for landing Clearing flags on attachment: 297672 Committed r210117: <http://trac.webkit.org/changeset/210117> |