58653 - Bad table reflow (different layout each reload)

Status: VERIFIED WORKSFORME

(Core :: Layout: Tables, defect, P3)

Description

[[e-mail]]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux 2.4.0-test9 i686; en-US; m18) Gecko/20001031
BuildID:    2000103106

Mozilla fails to render the table in the section "Galleri" correctly at the
first load. Other browsers (NS 4.75, IE 5.5 renders it correctly).

Reproducible: Always
Steps to Reproduce:
1.Load http://iesbild.com/ - Works fine
2.Click "Galleri" - thumbnails rendered incorrectly
3.Click the flower (or wherever, just load a new page in the site)
4.Click "Galleri" again - the table layout is correct

Actual Results:
On 1, -
On 2, Table layout is INCORRECT
On 3, -
on 4, Table layout is CORRECT

Expected Results:  Mozilla should have rendered the table layout in "Galleri"
correctly on the first try.

Comment 1

[Luis Villa]

This is even worse than it looks at first glance, I think. Hitting reload, I've
been able to force Moz to recreate the table randomly- it is different from the
previous reload something like 50% of the time. Pictures get merged, and it
appears that order is changed at least some of the time. Don't know if this is a
JS or table problem, but it is pretty nasty. #mozillazine confirms it on Linux
and but no problems with win32. 

Comment 2

[[e-mail]]

As timeless on IRC/#Mozilla requested, here are screenshots:

Netscape 4.75: http://findus.dhs.org/~odd/mozbugs/iesbild.com_netscape475.png
Mozilla 2000103106 1st try:
http://findus.dhs.org/~odd/mozbugs/iesbild.com_mozilla2000103106_first.png
Mozilla 2000103106 2nd try:
http://findus.dhs.org/~odd/mozbugs/iesbild.com_mozilla2000103106_first.png

Comment 3

[[e-mail]]

Erratum:

Mozilla 2000103106 2nd try:
http://findus.dhs.org/~odd/mozbugs/iesbild.com_mozilla2000103106_second.png

Comment 4

[Heikki Toivonen (remove -bugzilla when emailing directly)]

I am seeing this on NT branch build 2000-10-31-14 as well. OS -> All.

Over to HTMLTables.

Comment 5

[Keyser Sose]

WORKSFORME
Platform: PC
OS: Windows 98
Mozilla Build: 2001012205
Marking as such.

Comment 6

[[e-mail]]

Did NOT work in the Jan 22 Linux build I tried.

Hrmph.

Comment 7

[[e-mail]]

Can someone PLEASE show me a screenshot where this renders correctly THE FIRST
TIME?!

Comment 8

[Boris Zbarsky [:bzbarsky]]

Reopening.  I tried this with the 2001-02-12-08 Linux build and I see the exact
problem described.

Comment 9

[[e-mail]]

Well, hm. I've been told that it's my faulty, non-standards HTML/CSS that is to
blame?

Now, I'm not good at all at coding standards-compliant HTML/CSS, so perhaps
someone could take the code and correct it and see if that may help the
rendering... though I doubt that'd help... the table is pretty much correct, no?

Comment 10

[Boris Zbarsky [:bzbarsky]]

OK.  I have an HTML4.01-compliant version of this page.  I still see the bug
with this page and the 2001-02-14-08 build.  Going to attach the page here.

Changes I've made from the original:

1) Removed the external stylesheet (to simplify things)
2) added a <base> tag so that it will find the images correctly.
3) Removed the style from <a> from the inline stylesheet (again, to simplify things)
4) Added inline style to put borders around <td> elements (blue for outer table,
red for inner tables) so we can see how they are being laid out.
5) Added an inline style to put borders around the inner <table> elements (green
border)

Here is what I see:  the <td> elements of the inner tables are overflowing their
tables.  Looks like the page is not correctly reflowed as the images are loaded.
 Setting width and height on each image explicitly may fix the problems we are
seeing here, but Mozilla should still get this right.

Comment 11

[Boris Zbarsky [:bzbarsky]]

Attachment: HTML4.01-compliant version of page

Comment 12

[[e-mail]]

All praise Boris Zbarsky! :)

Comment 13

[Boris Zbarsky [:bzbarsky]]

Going to attach a fairly minimal testcase.  To reproduce with it, just load it
and hit the reload button a few times....  Maybe shift-reload if just reload
does not do it for you, but I have found that unnecessary.

Comment 14

[Boris Zbarsky [:bzbarsky]]

Attachment: Testcase. green border for table, red for <td>

Comment 15

[[e-mail]]

After the first load, when it is layed out incorrectly, I have to mark the URL
location bar and press enter for it to render correctly -- shift reload doesn't
do any difference on this machine (build 2001021409).

Comment 16

[Fabian Guisset]

I can confirm using the last testcase on 0.8 branch build, win95. The red border
extends further to the right than the green border.

Comment 17

[Samuel Sieb]

For more fun (at least on Linux), try dragging another window over the right end
of the table, either slowly or quickly for interesting redrawing effects.

Comment 18

[Mats Palmgren (inactive)]

Bug only occurs for Linux - adding keyword pp, OS -> Linux.
Bug occurs in build 2001-02-26-11 on Linux/Debian
Bug does NOT occur in build 2001-02-26-05 on Windows NT4 sp6
Bug does NOT occur in build 2001-02-23-11 on Windows 98 SE

Comment 19

[Fabian Guisset]

I retried this on build 2001022605 on windows 95. When first opening the
testcase, it displays correctly. Then everytime I reload I get a new layout,
images cut in half, borders extending to the right, etc.
Updating summary

Comment 20

[[e-mail]]

Buhuu buhuu.

Not to spam, but I really want this fixed (if it in fact is a problem).

Comment 21

[karnaze (gone)]

chrisd, can you see if this is still a problem on Linux. I can't reproduce it on 
WinNT.

Comment 22

[Amarendra Hanumanula]

QA contact update

Comment 23

[Samuel Sieb]

Still a problem on Linux, latest CVS build.

Comment 24

[Boris Zbarsky [:bzbarsky]]

works correctly for me now, linux build 2001-03-23-08

Comment 25

[[e-mail]]

It seems fixed here too.

Marking as such, okay?

Comment 26

[karnaze (gone)]

Reopening to change the status.

Comment 27

[karnaze (gone)]

A bug is usually marked fixed when there is a patch attached or it can be 
explained by some other patch (and it is not a dup). Marking worksforme.

Comment 28

[Amarendra Hanumanula]

 Verified works for me..