Bug 13208 – There should be a way to break long words in tables

• Status: RESOLVED
• Resolution: FIXED
• Component: CSS

[Beth Dakin]

* SUMMARY
If you have a line of text with the word-break property set to break-all, long words do not wrap if the text is located within a table.

* STEPS TO REPRODUCE
1. See attached text file for example.

* RESULTS
The first line of text in the sample is in a span and forcibly wraps (correctly) when you resize the window. The second line of text is in a table with a width of 100% and does not wrap.

[Beth Dakin]

Created attachment 13833 [details]
Resize window to see the bug

[Beth Dakin]

<rdar://problem/4881048>

Bug 13213 sounds like it might be in the same area (assuming the area turns out to be "min intrinsic width calculation for table cells").

[Dave Hyatt]

This behavior was deliberate.  I tested in WinIE when I implemented word-wrap: break-word, and they don't compute minimum width for table cells taking word-wrap:break-word into account.  If they did, there would be all sorts of table constructs that would result in cells with one character per line.

[Dave Hyatt]

This test case works in IE7 though.  My suspicion is that there may be two kinds of min width unfortunately.  Once the containing block puts you under pressure, you shrink further.

[Dave Hyatt]

Created attachment 13881 [details]
Another test

[Dave Hyatt]

The second test shows the problem.  IE7 ignores word-wrap:break-word on it.

[Dave Hyatt]

I just noticed the first case also uses a property called "word-break."  We do not support that property, and it could be why the first case is working.

[Dave Hyatt]

Created attachment 13882 [details]
Same as previous test but with word-break: break-all added

[Dave Hyatt]

word-break does not appear to be relevant here.

[Dave Hyatt]

I am wrong about this.  word-break: break-all is key.  word-wrap is completely irrelevant.  Testing with IE7, word-break:break-all causes the cell to actually wrap under pressure.

[Dave Hyatt]

Raising to p1 to match internal bug status.

(In reply to comment #11)
> Testing with IE7, word-break:break-all causes the cell to actually
> wrap under pressure.

Is this an IE7 only thing? With IE6 I don't see any difference between attachment 13881 [details] and attachment 13882 [details].

[Dave Hyatt]

Created attachment 14085 [details]
First cut at break-all support.

Not for review yet.  Just getting a patch up to track progress.

[Dave Hyatt]

There's a typo in the tests, which is what caused all the confusion in the first place.  Marking them as obsolete to avoid confusion.

[Dave Hyatt]

Created attachment 14086 [details]
white-space:pre doesn't work with word-break:break-all (unlike word-wrap: break-word)

Don't look at me.  I didn't design this crazy shit (shakes fist at Microsoft).

[Dave Hyatt]

Created attachment 14100 [details]
Patch to implement word-break partially (we don't support keep-all).

[Beth Dakin]

Comment on attachment 14100 [details]
Patch to implement word-break partially (we don't support keep-all).

r=me!

[Dave Hyatt]

r20967.