18650 - <BR> causes scrollbars in iframe, but doesn't use up space

Status: RESOLVED WORKSFORME

(Core :: Layout, defect, P3)

Description

[mahjongg]

ad turns up in scrollable box.
at the website www.tweakers.net you will find that the main page has an ad in
the upper right hand corner that turns up in a scrollable box.
This is certainly not intended as the crollbars overlap a big part of the image.
This site uses a lot of frames and javascript so it is hard to see for me what
is going on by looking at the source, but i guess there is something funny going
on with a mismatch of the actual size of the gif and the HTML coded size.
Internet explorer renders the page correctly.
I used the nightly build of seamonkey with buid ID: 1999111016

Comment 1

[mahjongg]

Attachment: gif showing ad in scrollbox

Comment 2

[Mats Palmgren (inactive)]

Attachment: testcase 1, content for first IFRAME

Comment 3

[Mats Palmgren (inactive)]

Attachment: testcase 1, content for second IFRAME

Comment 4

[Mats Palmgren (inactive)]

Attachment: testcase 1, enter here

Comment 5

[Mats Palmgren (inactive)]

This may look like bug 14827, but I don't think it is. The bug here is that
the trailing <BR> in the first IFRAME is causing vertical space (which triggers
bug 14827).
Bug occurs in build 1999111016 on Windows NT4 sp5.
Bug does not occur with IE4 on the same machine.

Comment 6

[troy]

Looks to be an IFRAME issue

Comment 7

[Eric Pollmann]

To separate this from bug 14827, let's remove the scrolling=no attribute from
the testcase.  I'll attach a modified test case after this update.

Actually, IE does add extra whitespace after the image for the <BR>.  Try the
testcase I'm going to add.  You'll see scrollbars in both iframes.  Scroll down
in both, you'll see an extra line of height in the top iframe.  This seems to
indicate that IE people felt that <BR> should add extra vertical whitespace.  We
don't seem to have made up our mind:

1) In the first frame, we add scrollbars, but scrolling down reveals that no
   extra space was added to the height.

2) If the height of the first frame in increased by 15 pixels to 75, no extra
   scrollbars will be added.

The real problem seems to be that we assume the <BR> will require extra vertical
space when determining if scrollbars are needed, but them we don't actually
create extra vertical space for it.  We should decide to either follow IE's lead
and both reserve and use the whitespace, or go our own way and not do either.

Nisheeth, do you have any comments on this?

Comment 8

[Eric Pollmann]

Attachment: Testcase 2 same as 1 but with scrolling=no removed

Comment 9

[Nisheeth Ranjan]

I would say that BR's should behave the same way inside an IFRAME as they do in
a top level document.  If we create vertical space for BR's in a top level
document (I think we do), we should do the same for BR's in an IFRAME.  And I
agree that we should either reserve and use the whitespace or do neither.

Comment 10

[Eric Pollmann]

I'll try to figure out where the height calculations are going wrong here and
why we aren't actually using the space.  Marking M14 as a really rough
guestimate.

Comment 11

[leger]

Resetting QA contact from leger.

Comment 12

[ekrock's old account (dead)]

Noncritical. Moving to M17.

Comment 13

[ekrock's old account (dead)]

Bulk moving [testcase] code to new testcase keyword. Sorry for the spam!

Comment 14

[Hixie (not reading bugmail)]

The testcases now refer to GIFs that 404. We need to remake the testcases.

Comment 15

[Eric Pollmann]

Rescheduling

Comment 16

[Eric Pollmann]

This bug has been marked "future" because the original netscape engineer working 
on this is over-burdened. If you feel this is an error, that you or another 
known resource will be working on this bug,or if it blocks your work in some way 
-- please attach your concern to the bug for reconsideration.

Comment 17

[Kevin McCluskey (gone)]

Bulk reassigning form bugs to Alex

Comment 18

[Mats Palmgren (inactive)]

Attachment: Testcase 3, content for first IFRAME (image+br)

Comment 19

[Mats Palmgren (inactive)]

Attachment: Testcase 3, content for second IFRAME (image)

Comment 20

[Mats Palmgren (inactive)]

Attachment: Testcase 3

Comment 21

[Mats Palmgren (inactive)]

WORKSFORME, build 2002-02-22-03 on Windows 98 SE.