63457 - Text overflows table cell if image in another cell specifies align=

Status: VERIFIED FIXED

(Core :: Layout: Tables, defect)

Description

[Akkana Peck]

If a table row has two cells, the first containing text and the second
containing an image with align=right, the text overflows its cell boundaries and
spills over into the image.  Remove the align=right, and this stops happening.

Unfortunately, putting an image into a table cell without an align= causes it to
display with a "-" character in NS4, so one has to design a page either to look
bad in NS4 or be unreadable in gecko.

Comment 1

[Akkana Peck]

Attachment: Small sample page illustrating the problem

Comment 2

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Sounds like the floating image isn't making the table cell expand to hold it. 
I'd argue that it probably should expand, although according to the current CSS
spec, it's totally unclear.  It's an interesting case, though.

->HTMLTables, although I guess it could be a block layout issue.

Comment 3

[Hixie (not reading bugmail)]

dbaron: iirc at the oslo face to face it was decided that floats inside cells do
grow their parent cells (or parent floats, or parent absolutely positioned
shrink wrapping blocks).

Comment 4

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

It seems to me, if anything was decided, it was the opposite, but the minutes
aren't very clear:
http://lists.w3.org/Archives/Member/w3c-css-wg/2000JulSep/0119.html

Comment 5

[Hixie (not reading bugmail)]

Ho hum. It is unclear indeed. Maybe we should bring this up again...

Comment 6

[Bernd]

The patch attached to bug 61772 fixes the problem. Not doing this by some
strange w3c decision will break bug 61772 and 13553 which has regressed. The
interesting thing from bug 13553 is that this growing was already
implemented!!!. Nav4 and IE5 also do this. If browser independency is a goal one
should allow the growing. Or shouldn't we take the freedom to innovate and make
the testcase working? In other word's if we dont allow the growing IE5 will work
like web developers expect it, and mozilla will break the pages. My
recommendation would be to fix this bug and make a mark in the source, to be
able to disable it, when it is really necessary.

Comment 7

[Akkana Peck]

Thanks for the pointer!  Indeed, that patch does fix the problem, or at least
makes it much better (sometimes pages still lay out wrong the first time, but on
reload they get better).  Looks like this is probably a dup of 13553 (to which
61772 was DUPed) but I'll let the owner decide.  Cc'ing buster, who owns 13553.

Comment 8

[Bernd]

buster checked in the fix

Comment 9

[Amarendra Hanumanula]

QA contact update

Comment 10

[Amarendra Hanumanula]

  The text is not spilling on to the image anymore. verified on 
Build ID: 2001052213..
OS: Linux

Marking verified