[Webkit-unassigned] [Bug 52185] Cell heights are disproportional when a row-spanning cell contains a block element that determines the height of the rows

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=52185


Julien Chaffraix <jchaffraix at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jchaffraix at webkit.org,
                   |                            |robert at webkit.org




--- Comment #2 from Julien Chaffraix <jchaffraix at webkit.org>  2012-02-17 11:05:21 PST ---
For the record, I spend some time investigating this bug and here is the analysis. It's not a trivial bug to fix as our current rowspan algorithm is too simple. We just add the extra rowspan height to the last row of a rowspan which matches other browsers in some cases but not all cases.

To properly implement that, I think we need to do something akin to FF and implement a 2 pass-algorithm: one without the rowspans and then we patch the logical heights to account for them.

See nsTableRowGroupFrame::CalculateRowHeights and more importantly http://mxr.mozilla.org/mozilla-central/source/layout/tables/nsTableRowGroupFrame.cpp#682 about how FF handles rowspans.

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