235535 - Page body appears off the screen on the right , scroll required to see

Status: RESOLVED FIXED

(Core :: Layout: Tables, defect)

Description

[Ahmet A. Akin]

User-Agent:       
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7a) Gecko/20040219 Firefox/0.8.0+

in http://www.liberal.org.tr/guncel/Diger/my_rejim.htm , Page body seems off the
screen, you have to scroll horizontally to see it. 
all the articles on the main page appears like that. looks like a standards -
bad design issue.

Reproducible: Always
Steps to Reproduce:
1.
2.
3.

Comment 1

[Patrick Quaedackers]

Confirmed (build id: 2004012808) on win2k. 
This is *NOT* bug 217369 .
I use a buildrelease with a patch for that bug.

Comment 2

[Boris Zbarsky [:bzbarsky]]

This needs a testcase

Comment 3

[Ahmet A. Akin]

when i wanted to edit the page with composer for a test sample. But it looks all
right  in the preview tab in there. strange. 

Comment 4

[Mats Palmgren (inactive)]

Attachment: Testcase

Comment 5

[Bernd]

Attachment: patch

Comment 6

[Bernd]

taking

Comment 7

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 142592 [details] [diff] [review]
patch

>Index: nsCellMap.cpp
>   if (origData) {
>     origData->Init(aCellFrame);
>+    // we are replacing a dead cell, increase the number of cells 
>+    // originating at this column

Could you add an assert right here that origData is in fact dead?

>+    nsColInfo* colInfo = aMap.GetColInfoAt(startColIndex);
>+    if (colInfo) { 

Maybe NS_WARN_IF_FALSE or NS_ASSERTION that colInfo is not null in addition to
the null check?  I assume it being null is an error condition that really
should not happen?

r+sr=bzbarsky

Comment 8

[Bernd]

fix checked in