240148 - <table height="100%"></table> takes up space (gap on sonique.lycos.com)

Status: RESOLVED DUPLICATE of bug 241161

(Core :: Layout, defect)

Description

[Hussam Al-Tayeb]

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

When I open http://sonique.lycos.com , there is a gap at the top of the page.
This bug did not exist in mozilla 1.6

Reproducible: Always
Steps to Reproduce:
1.Open http://sonique.lycos.com
2.Notice the big gap at the top of the page


Actual Results:  
there is a big gap at the top of the page.

Expected Results:  
there shouldn't be a big gap at the top of the page

Comment 1

[Hussam Al-Tayeb]

See http://validator.w3.org/check?uri=http://sonique.lycos.com
it gives a fatal error 
Fatal Error: No DOCTYPE specified!
but somehow it used to render correctly in firefox0.8 (1.6)

Comment 2

[Hussam Al-Tayeb]

Can somebody confirm this ?

Comment 3

[Bernard Alleysson]

confirming firefox 20040409 trunk

Comment 4

[Frank Wein [:mcsmurf]]

Where's the minimal testcase people? Without a minimal testcase a layout bug
can't be fixed (and shouldn't be set to NEW in most cases).

Comment 5

[Hussam Al-Tayeb]

Attachment: simplified testcase

simplified testcase attached

Comment 6

[Boris Zbarsky [:bzbarsky]]

Hussam, thank you for the testcase.

Bernard, please read
http://groups.google.com/groups?dq=&hl=en&lr=&ie=UTF-8&oe=UTF-8&th=ac50fff3f060ead7

Bernd, didn't we recently make empty tables like that actually display, whereas
they used not to?  Should we just evangelize this site, or should we make not
displaying them a quirk?

Comment 7

[Hussam Al-Tayeb]

Ok, I got some input about this from a couple of people.
what's happening here is that after opening the table, there are some script
elements. Firefox0.8 closes the table right after those script elements whereas
firefox0.8 (1.6 ) did not. Hence, the rest of the page is not being included in
the table because the table is now closed.

Comment 8

[Boris Zbarsky [:bzbarsky]]

> Firefox0.8 closes the table right after those script elements whereas
> firefox0.8 (1.6 ) did not. 

Er... you're talking about the same browser version there?

Looks like this table-closing issue is what we need a testcase for?

Comment 9

[Hussam Al-Tayeb]

(In reply to comment #8)
sorry I meant 0.8+ (1.7 ) and firefox 0.8 (1.6)

Comment 10

[Boris Zbarsky [:bzbarsky]]

Ah, ok.  Note that I doubt that our parsing changed here (which is what comment
7 claims to have happened; that's why I'd love to see a testcase showing that
comment to be correct).

Comment 11

[Hussam Al-Tayeb]

(In reply to comment #8)
After thinking about this, the table closing issue seems to be another bug, so I
think the testcase attached is valid.
If you view the testcase in firebird0.8 (1.6), the empty table won't shove the
the rest of the page down.

Comment 12

[Boris Zbarsky [:bzbarsky]]

Right.  Per standards it SHOULD shove it all the way down.  The only question is
whether we want to violate standards in this case, and if so how.

Comment 13

[Patrick Quaedackers]

This was also an issue on Mozilla 1.6!!

Comment 14

[Jesse Ruderman]

This bug also creates a large gap on http://www.girlpaladin.com/introverts.html.
 (I made a testcase and it was essentially the same as the one here.)

Comment 15

[Bernd]

fixed by checkin for bug 241161

*** This bug has been marked as a duplicate of 241161 ***