RESOLVED FIXED3235
CSS2: Table cell coalescing in CSS broken 
https://bugs.webkit.org/show_bug.cgi?id=3235
Summary CSS2: Table cell coalescing in CSS broken
Dave Hyatt
Reported 2005-06-01 14:56:31 PDT
<html> <head> <style> .entries { display: table-cell; } .sidebar { display: table-cell; } </style> </head> <body> <div class="nav"></div> <div class="entries">One</div> <div class="sidebar">Two</div> </body> </html> "One" and "Two" should be in the same table, so they should be in a row together. Instead they end up in separate tables all because of the presence of the extra "nav" div.
Attachments
test case as attachement (216 bytes, text/html)
2005-12-28 02:12 PST, Eric Seidel (no email) 		no flags
Details
Patch to add test (2.15 KB, text/plain)
2009-04-21 02:00 PDT, Cameron Zwarich (cpst) 		darin: review+
Details
View All Add attachment proposed patch, testcase, etc.
Dave Hyatt
Comment 1 2005-06-01 14:57:37 PDT
Apple Bug: rdar://3299883/
Eric Seidel (no email)
Comment 2 2005-12-28 02:12:04 PST
Created attachment 5331 [details] test case as attachement
Eric Seidel (no email)
Comment 3 2005-12-28 02:13:00 PST
CC'ing beth as this is in her area of expertise.
Bradley Meck
Comment 4 2008-05-31 01:06:42 PDT
After looking into the bug I cannot seem to locate the logic for testing adjacent cells. It does not occur in all the Rendering files and my next guess was the CSSParser, which also does not appear to have the logic. Also the use of setStyle only occurs once for TABLE_CELL, and that also does not appear to contain the logic. Any clues?
Rob Buis
Comment 5 2008-06-01 23:50:52 PDT
Hi Bradley, (In reply to comment #4) > After looking into the bug I cannot seem to locate the logic for testing > adjacent cells. It does not occur in all the Rendering files and my next guess > was the CSSParser, which also does not appear to have the logic. Also the use > of setStyle only occurs once for TABLE_CELL, and that also does not appear to > contain the logic. Any clues? I noticed that the render tree dumps are different if you position the non table cell div first or last. In the last case two RenderTableCell's are created, the first case just one. It may be worth debugging on what basis and where these RenderTableCell's are created, maybe by stepping through the code or adding printf statements. Cheers, Rob.
Cameron Zwarich (cpst)
Comment 6 2009-04-21 02:00:54 PDT
Created attachment 29644 [details] Patch to add test The bug is now fixed, but we should add this as a test.
Cameron Zwarich (cpst)
Comment 7 2009-04-21 08:49:21 PDT
Landed in r42711.
Note You need to log in before you can comment on or make changes to this bug.