89731 - layout is messed up

Status: RESOLVED WORKSFORME

(Core :: Layout, defect)

Description

[Tom]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux 2.4.5 i686; en-US; rv:0.9.1) Gecko/20010620
BuildID:    20010620

The layout of the page is messed up.  It's fine in IE.

(I didn't see any proprietary extensions being used, but it's hard to tell. 
Maybe Mozilla should detect non-standard constructs and display a little warning
image in its status bar.  That way, users will know whether to report a bug or
complain to the web site author.)

Reproducible: Always
Steps to Reproduce:
1.go to page
2.look at page


Actual Results:  layout doesn't look right: the body of the page is way off to
the right, and two advertising images float lonely in a sea of gray on the left.

Expected Results:  A decent looking HTML page with the body text in the visible
portion of the window.

Comment 1

[Jacek Piskozub]

Confirming with 20010706 build on WindowsME. Changing Platform/OS to All/All.

I'm not sure it's not a dup - there were some similar layout bugs lately.

Comment 2

[Chris Petersen]

Confiriming issue in the July 10th build (20010710)

Comment 3

[Mats Palmgren (inactive)]

Attachment: Testcase

Comment 4

[Mats Palmgren (inactive)]

The testcase has two large HTML comments in it (thousands of characters) -
remove them and the bug disappears. Also note NOWRAP attribute in the first
cell. I am certain I have done a testcase for a bug long ago that also depended
on large comments, I'll see if I can dig that up.

(If you "View -> Page Source" you will notice that the large comments are not
 displayed properly, I'll file that separatly if not done already.)

Comment 5

[Mats Palmgren (inactive)]

The earlier bug is bug 19356, but the testcases there worked fine.

Comment 6

[Mats Palmgren (inactive)]

Reassigning to Alexandru since it involves NOWRAP.
Filed new bug 92193 for the "View Source" problem.

Comment 7

[Alexandru Savulov]

the patch that fixes 80817, will fix this one too

Comment 8

[Alexandru Savulov]

The patch for bug 80817 repairs the behaviour for the testcase but does not
solve the problem for the URL.

suspecting DUP= bug 93408 ?

Comment 9

[Alexandru Savulov]

suspecting DUP= 93363 too

Comment 10

[Mats Palmgren (inactive)]

Attachment: Testcase #2

Comment 11

[Mats Palmgren (inactive)]

The new testcase is almost the same as the first - I've added a <P> to the first
cell. I see the bug with 2001-08-09-04 on Windows 98 SE.
Please note the disturbing fact that *a comment* is affecting layout.

Comment 12

[dcone (gone)]

r=dcone.  Saw the comment code, maybee put a comment there.

Comment 13

[dcone (gone)]

wrong bug.. sorry.. 

Comment 14

[Alexandru Savulov]

Mats: please use the latest trunk/branch builds to test.

The testcases display correctly, however the page still displays bad, so I think
that the testcases do not reveal us the bug. (They did revealed another bug
before that's been fixed on trunk and 0.9.2 branch though.)

Comment 15

[Alexandru Savulov]

works for me 
trunk build: 2001092103
test with URL
(i think that Mark Attinasi fixed another a bug, and that fix, fixed this one too)