Bug 136861 - overflow:scroll should not leave space for a scroll corner with overlay scrollbars
Summary: overflow:scroll should not leave space for a scroll corner with overlay scrol...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-09-16 11:46 PDT by Beth Dakin
Modified: 2014-09-16 13:32 PDT (History)
4 users (show)

See Also:

Attachments
Test case (711 bytes, text/html)
2014-09-16 11:47 PDT, Beth Dakin 		no flags Details
Patch (10.52 KB, patch)
2014-09-16 11:52 PDT, Beth Dakin 		sam: review+
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 Beth Dakin 2014-09-16 11:46:37 PDT 
overflow:scroll should not leave space for a scroll corner on the Mac with overlay scrollbars because it looks wrong. Historically, overflow:scroll elements were meant to have scrollbar tracks and scroll corners whether or not there was content to scroll to. Since overlay scrollbars only appear when the user is scrolling, we don't do anything to differentiate overflow:scroll from overflow:auto if there is nothing to scroll too -- EXCEPT that we leave space for a scroll corner, which just looks wrong. We should stop doing that and embrace the fact that overflow:scroll just means something slightly different in the world of overlay scrollbars.
Comment 1 Beth Dakin 2014-09-16 11:47:54 PDT 
Created attachment 238191 [details]
Test case

Here's a test case that shows the problem. It's silly that we leave space for the scroll corner in the top two <div>s. It only makes sense in the bottom <div>.
Comment 2 Beth Dakin 2014-09-16 11:52:56 PDT 
Created attachment 238192 [details]
Patch
Comment 3 Beth Dakin 2014-09-16 13:32:52 PDT 
Thanks Sam! http://trac.webkit.org/changeset/173668