For some reason, overflow inside a table is clipped. For example, the text in the following table: <table> <tr> <td> <div style="width:80px;"> Truncated?&nbsp;It&nbsp;should&nbsp;not&nbsp;be! </div> </td> </tr> </table> ...is truncated after the "It". This should not happen, and in fact it doesn't, if you take the table out. See the following test case: http://www.bath.ac.uk/%7Epy8ieh/internet/projects/mozilla/tableoverflow.html

See also bug 5999.

Kipp, can you take a look. nsTableCellFrame::Reflow makes a call to ReflowChild on its area frame and it is returning 80px as its desired size.

[Kipp is on sabatical.]

The problem is that the table cell frame painting code is ignoring the css "overflow" property. In particular, what the cell code does is *always* sets a clip so that things in the cell can't paint outside. The fix is to do what the block code does: Test the value of the mOverflow property and if its value is "NS_STYLE_OVERFLOW_HIDDEN" then and only then set the clip. I could make the change, but I'd probably break some nasty navigator compatibility thing. So I'm reassigning it to karnaze to answer the compatability question so that he can either nod or frown....I'll even be glad to make the fix if he wants...

how the browser reads tables and CSS together needs to be fixed

Kipp, I would appreciate it if you could fix this. Pushing/poping the rendering context state and setting the clip rect was done to make collapsing row/row groups work.

Reassigning to Kipp, based on earlier comments.

Moving to M10.

s/with/without/ in that previous comment. Urk.

Fixed. The table cell rendering code now honors the css overflow property.

I'm going to accept the bug and back out Kipp's changes which fixed the table cell problem, because it caused a problem with collapsing rows. This won't be the case until I do a checkin today or tomorrow.

Spoke to karnaze during bug triage today. Moving to M11. Not an M10 blocker.

Moving to M13.

Troy reinstated Kipp's changes without breaking collapsing rows.

Using 11/16 build, verified fixed. Text flows outside the cell.

Migrating from {css1} to css1 keyword. The {css1}, {css2}, {css3} and {css-moz} radars should now be considered deprecated in favour of keywords. I am *really* sorry about the spam...