[Webkit-unassigned] [Bug 76948] min-width is not implemented on <table> for table-layout: fixed

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Feb 13 00:43:51 PST 2012


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





--- Comment #7 from Antaryami Pandia <antaryami.pandia at motorola.com>  2012-02-13 00:43:51 PST ---
(In reply to comment #5)
Yes the code AutoTableLayout::computePreferredLogicalWidths is required. Actually what I was saying was "AutoTableLayout::computePreferredLogicalWidths" already sets the minwidth and maxwidth, so what's purpose of code added in "RenderTable::computeLogicalWidth". Does this code solves any specific issue?

>>It does seem to work in most cases, but I think some cases would fail for fixed table layout without additional code in >>FixedTableLayout::computePreferredLogicalWidths. I'm not exactly sure what these cases are. This is related to your observation below.

Actually I was also thinking that can we put code similar to "AutoTableLayout" in "FixedTableLayout".

And if the code in "RenderTable::computeLogicalWidth" does not have any specific use case can we remove this. It's not because I don't like the code but think it suppresses some unknown like it fixes the "table-layout: fixed" case without any code being added to "FixedTableLayout.cpp". Here I am assuming that the purpose of the code in RenderTable.cpp is not to fix the "table-layout: fixed" case. Please correct me if I am wrong.

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



More information about the webkit-unassigned mailing list