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

Dave Hyatt
Reported 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
Attachments
Test case (1.04 KB, text/html)
2005-09-05 15:14 PDT, Patrick Geiller 		no flags
Details
For the curious :) test case for non-repeating backgrounds (423 bytes, text/html)
2005-09-06 14:15 PDT, Patrick Geiller 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Dave Hyatt
Comment 1 2005-06-01 16:09:18 PDT
Apple Bug: rdar://3613374/
Maks Orlovich
Comment 2 2005-07-05 10:34:47 PDT
http://bugs.kde.org/show_bug.cgi?id=70326#c17, FYI
Nicholas Shanks
Comment 3 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).
Patrick Geiller
Comment 4 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.
Dave Hyatt
Comment 5 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).
Patrick Geiller
Comment 6 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.
Alexey Proskuryakov
Comment 7 2006-02-13 21:48:30 PST
*** Bug 7242 has been marked as a duplicate of this bug. ***
Dave Hyatt
Comment 8 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.
Alexey Proskuryakov
Comment 9 2006-03-26 06:38:44 PST
Marking as fixed, according to the above comment.
Note You need to log in before you can comment on or make changes to this bug.