Bug 18203

Summary: Right floats should be allowed to overflow past left border edge
Product: WebKit Reporter: Dave Hyatt <hyatt>
Component: CSSAssignee: Dave Hyatt <hyatt>
Status: RESOLVED FIXED    
Severity: Normal CC: alan
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.5   
URL: http://css-class.com/test/css/overflow/float-container-margin-overflow.htm
Attachments:
Description Flags
Remove the clamping code hyatt: review-

Description Dave Hyatt 2008-03-28 19:08:24 PDT
See:

http://css-class.com/test/css/overflow/float-container-margin-overflow.htm

In current WebKit, the right float is not allowed to have its left edge extend beyond the left border edge.  This is not correct according to the CSS2.1 spec and does not match Firefox.
Comment 1 Dave Hyatt 2008-03-28 19:09:47 PDT
Created attachment 20179 [details]
Remove the clamping code

Remove the code that clamps right floats to the left border edge.
Comment 2 Dave Hyatt 2008-03-28 19:11:26 PDT
I will land this patch with a better test case that doesn't depend on resizing the window.
Comment 3 Darin Adler 2008-03-30 16:44:24 PDT
Comment on attachment 20179 [details]
Remove the clamping code

r=me because I know you'll add a ChangeLog and test case -- but why not do that before setting the review flag next time?
Comment 4 Dave Hyatt 2008-03-30 17:07:15 PDT
Because keeping a ChangeLog in my tree for days is a giant pain when I'm doing other work.

Comment 5 Alan Gresley 2008-04-06 10:49:20 PDT
I have created a new test case without the need for scrolling. Just one element floated right. Firefox will hide the left edge of the float beyond the left edge of the viewpoint. Safari will show the full width of the float causing a horizontal scrollbar. See:

http://css-class.com/test/bug/float-right-left-edge-hidden.htm

Comment 6 Darin Adler 2008-06-08 12:27:32 PDT
Hyatt, are you planning on landing this?
Comment 7 Dave Hyatt 2008-09-03 00:11:35 PDT
Comment on attachment 20179 [details]
Remove the clamping code

Let me get a test case and changelog.
Comment 8 Dave Hyatt 2008-09-03 11:32:19 PDT
Fixed in r36061.