Bug 5454

Summary: Table at dinodream.com is too tall in Safari
Product: WebKit Reporter: Dave Hyatt <hyatt>
Component: TablesAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: ddkilzer, ian, jonathanjohnsson, mitz, sam
Priority: P2 Keywords: HasReduction
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
URL: http://www.dinodream.com/
Bug Depends on:    
Bug Blocks: 9610    
Attachments:
Description Flags
Test case none

Dave Hyatt
Reported 2005-10-21 13:42:23 PDT
The table at www.dinodream.com is too tall in Safari. Works in Opera, Firefox and WinIE.
Attachments
Test case (3.62 KB, text/html)
2006-02-11 05:18 PST, jonathanjohnsson
no flags
Eric Seidel (no email)
Comment 1 2005-12-27 15:34:55 PST
It's the "Taze" table it seems. Needs a reduced test case though.
jonathanjohnsson
Comment 2 2006-02-11 05:18:02 PST
Created attachment 6408 [details] Test case The test case consists mainly of a table inside a div inside a table. Safari seems to have trouble calculating the height of the td containing the div. The div has "overflow: auto", and the table it contains has align="left". Removing the latter property makes Safari render correctly. The text below the div moves depending on the amount of the text inside the div.
Joost de Valk (AlthA)
Comment 3 2006-02-11 07:35:08 PST
Though this testcase might need some work, it accurately shows the problem it seems, so changing keyword.
David Kilzer (:ddkilzer)
Comment 4 2006-12-23 07:06:54 PST
*** Bug 11944 has been marked as a duplicate of this bug. ***
David Kilzer (:ddkilzer)
Comment 5 2006-12-23 07:07:45 PST
(In reply to comment #4) > *** Bug 11944 has been marked as a duplicate of this bug. *** This is a Quriksmode bug.
David Kilzer (:ddkilzer)
Comment 6 2007-01-28 20:16:03 PST
This has been fixed as of a locally-built debug build of WebKit r19207 with Safari 2.0.4 (419.3) on Mac OS X 10.4.8 (8L127). Probably fixed by Mitz's recent table layout fixes.
mitz
Comment 7 2007-01-28 22:27:17 PST
I think Hyatt fixed this one.
Note You need to log in before you can comment on or make changes to this bug.