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




Layout: Tables
17 years ago
17 years ago


(Reporter: Akkana Peck, Assigned: karnaze (gone))



Dependency tree / graph

Firefox Tracking Flags

(Not tracked)


(Whiteboard: WG: do floats inside cells grow the cell (yes please))


(1 attachment)



17 years ago
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

17 years ago
Created attachment 21097 [details]
Small sample page illustrating the problem
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.
Assignee: clayton → karnaze
Component: Layout → HTMLTables
QA Contact: petersen → chrisd
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).
Keywords: css2
It seems to me, if anything was decided, it was the opposite, but the minutes
aren't very clear:
Ho hum. It is unclear indeed. Maybe we should bring this up again...
Whiteboard: WG: do floats inside cells grow the cell (yes please)


17 years ago
Blocks: 62091

Comment 6

17 years ago
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.


17 years ago
Depends on: 13553

Comment 7

17 years ago
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

17 years ago
buster checked in the fix
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 9

17 years ago
QA contact update
QA Contact: chrisd → amar

Comment 10

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

Marking verified 
You need to log in before you can comment on or make changes to this bug.