Bug 18203 - Right floats should be allowed to overflow past left border edge
Summary: Right floats should be allowed to overflow past left border edge
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 528+ (Nightly build)
Hardware: Macintosh OS X 10.5
: P2 Normal
Assignee: Dave Hyatt
URL: http://css-class.com/test/css/overflo...
Depends on:
Reported: 2008-03-28 19:08 PDT by Dave Hyatt
Modified: 2008-09-03 11:32 PDT (History)
1 user (show)

See Also:

Remove the clamping code (585 bytes, patch)
2008-03-28 19:09 PDT, Dave Hyatt
hyatt: review-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dave Hyatt 2008-03-28 19:08:24 PDT


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:


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.