[Webkit-unassigned] [Bug 17139] Add support for height property on <tbody> element

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sat May 24 23:06:54 PDT 2008


http://bugs.webkit.org/show_bug.cgi?id=17139


darin at apple.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #21071|review?                     |review-
               Flag|                            |




------- Comment #7 from darin at apple.com  2008-05-24 23:06 PDT -------
(From update of attachment 21071)
This seems wrong. It assumes that the height of the <tbody> can be evenly
divided by the number of rows. That's clearly not correct when some rows have
explicit heights themselves. For example, consider a <tbody> with a height of
100px, containing five rows where the first four rows all are 10px tall. It
doesn't make sense to have the fifth row by 20px tall. It should be 60px tall.

We'll need more than one test case for this, too. The test patch needs to
include the expected results for the test case. Does this change affect the
results of any other test cases? I'd be amazed if it didn't affect any of the
hundreds of table test cases we imported from the Mozilla project.

I want to hear Hyatt's take on this too. What is the correct behavior for table
sections?


-- 
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.



More information about the webkit-unassigned mailing list