Bug 9531

Summary: Pages that need only a horizontal scroll bar get a vertical one as well
Product: WebKit Reporter: mitz
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: ddkilzer
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
URL: data:text/html,<div%20style="width:5000px"></div>
Attachments:
Description Flags
Allow body to shrink when the scrollbar is added
none
The "binaries" part of the patch
none
Allow body to resize when the view's height changes hyatt: review+

mitz
Reported 2006-06-21 09:12:57 PDT
The URL opens with a horizontal scroll bar as well as a vertical scroll bar that lets you scroll 15px down. It happens because the body element does not resize as it should when the horizontal scroll bar eats into the view. I am going to post a patch.
Attachments
Allow body to shrink when the scrollbar is added (75.70 KB, patch)
2006-06-21 09:27 PDT, mitz 		no flags
DetailsFormatted DiffDiff
The "binaries" part of the patch (655.56 KB, application/octet-stream)
2006-06-21 09:30 PDT, mitz 		no flags
Details
Allow body to resize when the view's height changes (75.71 KB, patch)
2006-06-21 11:40 PDT, mitz 		hyatt: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
mitz
Comment 1 2006-06-21 09:27:39 PDT
Created attachment 8945 [details] Allow body to shrink when the scrollbar is added The "binaries" part of the patch is in a separate attachment.
mitz
Comment 2 2006-06-21 09:30:05 PDT
Created attachment 8946 [details] The "binaries" part of the patch
David Kilzer (:ddkilzer)
Comment 3 2006-06-21 10:18:34 PDT
SUPER SWEET!! Does this fix Bug 8909 as well?
mitz
Comment 4 2006-06-21 11:40:21 PDT
Created attachment 8950 [details] Allow body to resize when the view's height changes Changed the > to != in the height comparison. No change to the binary part.
mitz
Comment 5 2006-06-21 11:41:10 PDT
(In reply to comment #3) > Does this fix Bug 8909 as well? No :-( 
Dave Hyatt
Comment 6 2006-06-22 15:03:39 PDT
Comment on attachment 8950 [details] Allow body to resize when the view's height changes r=me
David Kilzer (:ddkilzer)
Comment 7 2006-06-24 08:05:58 PDT
Committed revision 15011.
Note You need to log in before you can comment on or make changes to this bug.