Bug 13214

Summary: Right hand float is placed at the bottom of document
Product: WebKit Reporter: Oliver Hunt <oliver>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: CLOSED INVALID    
Severity: Normal CC: mitz
Priority: P2 Keywords: HasReduction
Version: 523.x (Safari 3)   
Hardware: Mac   
OS: OS X 10.4   
URL: http://stuff.co.nz/4008837a10133.html
Attachments:
Description Flags
Webarchive of referenced page
none
Reduction
none
Apparent root cause
none
Regression test darin: review+

Oliver Hunt
Reported 2007-03-28 04:19:58 PDT
The right hand column of the referenced url is placed at the bototm of the article, rather than beside it. Firefox renders correctly
Attachments
Webarchive of referenced page (296.02 KB, application/octet-stream)
2007-03-28 04:20 PDT, Oliver Hunt
no flags
Reduction (348 bytes, text/html)
2007-03-28 05:55 PDT, mitz
no flags
Apparent root cause (419 bytes, text/html)
2007-03-28 13:54 PDT, mitz
no flags
Regression test (48.60 KB, patch)
2007-03-28 14:54 PDT, mitz
darin: review+
Oliver Hunt
Comment 1 2007-03-28 04:20:44 PDT
Created attachment 13841 [details] Webarchive of referenced page
mitz
Comment 2 2007-03-28 05:55:55 PDT
Created attachment 13842 [details] Reduction
Oliver Hunt
Comment 3 2007-03-28 13:08:06 PDT
Hmmm, the reduction doesn't seem to trigger the layout bustage i see in the webarchive :-/
mitz
Comment 4 2007-03-28 13:41:26 PDT
(In reply to comment #3) > Hmmm, the reduction doesn't seem to trigger the layout bustage i see in the > webarchive :-/ > Not seeing red?
Oliver Hunt
Comment 5 2007-03-28 13:54:33 PDT
Oh, oops Firefox != Safari :D Must ensure that i'm using the right browser when checking tests :D
mitz
Comment 6 2007-03-28 13:54:54 PDT
Created attachment 13848 [details] Apparent root cause
mitz
Comment 7 2007-03-28 14:24:18 PDT
Hyatt (and I!) think that WebKit's behavior is correct, according to <http://www.w3.org/TR/2006/WD-CSS21-20061106/visudet.html#q8>. Firefox 3 also matches WebKit. So this bug is INVALID. Leaving it open until a layout test documenting this behavior is landed.
mitz
Comment 8 2007-03-28 14:54:10 PDT
Created attachment 13850 [details] Regression test Just add a layout test documenting the current correct behavior.
Darin Adler
Comment 9 2007-03-28 14:57:10 PDT
Comment on attachment 13850 [details] Regression test r=me
Alexey Proskuryakov
Comment 10 2007-03-31 03:36:39 PDT
Committed revision 20638, closing as INVALID.
Note You need to log in before you can comment on or make changes to this bug.