389468 - Text in iGoogle's news Div is clipped off

Status: RESOLVED FIXED

(Core :: Layout, defect)

Description

[Tony Chung [:tchung]]

Attachment: Minefield attachment

The chinese text located in the left hand column (news section) is slightly clipped off.   This only appears on the minefield builds, and not the nightlies.  Screenshots from both Firefox 2.0.0.6 and minefield nightly is attached.

Comment 1

[Tony Chung [:tchung]]

Attachment: Bon Echo Attachment

Comment 2

[philippe (part-time)]

Is that a recent regression on trunk ? If so,can you try to track down when this changed ?
I suspect it comes from one of the New TextFrame landings over the past few weeks (see bug 367177).

One thing I noticed on the url: if I first zoom out, and then zoom up, then text flows differently (more correctly) than when I first loaded the page: there is one character less per line.

Comment 3

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Could use a reduced testcase...

Comment 4

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

I can reproduce this on WinXP too.

Comment 5

[Daniel Holbert [:dholbert]]

This isn't specific to the Chinese version of iGoogle.

I currently see the problem in the standard English version of iGoogle, in all news tabs except for the default "Top Stories" tab.  (i.e. I see text-clipping in the World, U.S., Business, and Sci/Tech news tabs.)  These tabs look fine in Firefox 2.

Attaching screenshots in a sec.

--> Removing "Chinese" from summary and intl from keywords, as it seems this bug isn't related to those.

Comment 6

[Daniel Holbert [:dholbert]]

Attachment: Minefield attachment, english

Comment 7

[Daniel Holbert [:dholbert]]

Attachment: Bon Echo Attachment, english

Comment 8

[Daniel Holbert [:dholbert]]

This may be related to bug 391431, for which I've got a reduced testcase posted.
(Both bugs with Google News stuff, both bugs with text overflowing its bounds.)

Comment 9

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Attachment: reduced testcase

Here's a reduced testcase... bug 393096 ought to help here, I'm not sure why it doesn't.

Comment 10

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Attachment: improved testcase

This testcase is fully automated and will be easy to convert into a reftest. For any reasonable font, the text should fit inside the table, but it doesn't. If you comment out the temporary setting of the table width to 4em, the problem disappears.

Comment 11

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Should be fixed by bug 389666.

Comment 12

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Fixed by checkin for bug 389666. I also check in a reftest based on attachment #285055 [details].