Bug 3242

Summary: Table rows have no dimensions
Product: WebKit Reporter: Dave Hyatt <hyatt>
Component: TablesAssignee: Dave Hyatt <hyatt>
Status: RESOLVED FIXED    
Severity: Normal CC: nickshanks, pg, pshah
Priority: P2 Keywords: InRadar
Version: 412   
Hardware: All   
OS: All   
Bug Depends on: 3297    
Bug Blocks:    
Attachments:
Description Flags
Test case
none
For the curious :) test case for non-repeating backgrounds none

Description Dave Hyatt 2005-06-01 16:09:01 PDT
Table rows do not have a size in the render tree currently.  This needs to be fixed, since it is causing the 
following issues:

(1) Backgrounds don't work properly on table rows
(2) offsetWidth returns 0
(3) Anchor jumping to a row doesn't work
Comment 1 Dave Hyatt 2005-06-01 16:09:18 PDT
Apple Bug: rdar://3613374/
Comment 2 Maks Orlovich 2005-07-05 10:34:47 PDT
http://bugs.kde.org/show_bug.cgi?id=70326#c17, FYI 
 
Comment 3 Nicholas Shanks 2005-08-08 11:30:26 PDT
Possibly related to this is the bug whereby tr:hover CSS styles don't apply if the mouse is within the row 
bounds but outside any of it's cells (i.e. there is some cellspacing, or the row doesn't have a full 
complement of table cells).
Comment 4 Patrick Geiller 2005-09-05 15:14:39 PDT
Created attachment 3775 [details]
Test case

The bug also affects TBODY. TBODY has a zero offsetWidth and correct
offsetHeight, TR zero offsetWidth &  offsetHeight.
Comment 5 Dave Hyatt 2005-09-05 15:18:38 PDT
Yes, this is a major architectural flaw in tables.  It also affects background painting (try doing a non-
repeating background on a row or body).
Comment 6 Patrick Geiller 2005-09-06 14:15:22 PDT
Created attachment 3789 [details]
For the curious :) test case for non-repeating backgrounds

So the TR background-image is duplicated in all TDs. There's also something
fishy with a border on a TR, which gets semi-replicated (only right & bottom)
on TDs, but only if table has border=1.
Comment 7 Alexey Proskuryakov 2006-02-13 21:48:30 PST
*** Bug 7242 has been marked as a duplicate of this bug. ***
Comment 8 Dave Hyatt 2006-03-15 19:45:57 PST
Upon further investigation, it appears that it should not be possible to hit test a row.  Other browsers do not allow you to hit test it, so tr:hover is *not* supposed to work if you are in the space between cells.

Moreover background painting is also not done on the row.  All browsers inherit the backgrounds into the cells.  It looks like Firefox - in strict mode only - will honor the non-repeating bg, but they still don't do it correctly.

In general, our agreed strategy with tables is to match WinIE, and so the background painting is not an issue any more (nor is the hover).

Issues that are genuine, however, are the anchor bug (which is filed as 5393) and the offsetWidth/Height issue.  These are now resolved with the patch attached to 5393.
Comment 9 Alexey Proskuryakov 2006-03-26 06:38:44 PST
Marking as fixed, according to the above comment.