Closed Bug 18650 Opened 20 years ago Closed 18 years ago

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

Categories

(Core :: Layout, defect, P3)

defect

Tracking

()

RESOLVED WORKSFORME
Future

People

(Reporter: mahjongg, Assigned: alexsavulov)

References

()

Details

(Keywords: testcase)

Attachments

(4 files, 4 obsolete files)

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
Summary: ad turns op in scrollable box
Summary: ad turns op in scrollable box → ad turns up in scrollable box
Attached file testcase 1, enter here (obsolete) —
Assignee: leger → troy
Component: Browser-General → Layout
Whiteboard: [TESTCASE]
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.
Assignee: troy → pollmann
Looks to be an IFRAME issue
Status: NEW → ASSIGNED
OS: Windows 95 → All
Hardware: PC → All
Summary: ad turns up in scrollable box → <BR> causes scrollbars in iframe, but doesn't use up space
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?
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.
Target Milestone: M14
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.
Resetting QA contact from leger.
Whiteboard: [TESTCASE] → [TESTCASE] (py8ieh:considering for {css1} radar)
Target Milestone: M14 → M17
Noncritical. Moving to M17.
Bulk moving [testcase] code to new testcase keyword. Sorry for the spam!
Keywords: testcase
The testcases now refer to GIFs that 404. We need to remake the testcases.
Whiteboard: [TESTCASE] (py8ieh:considering for {css1} radar) → [TESTCASE]
Rescheduling
Target Milestone: M17 → M21
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.
Target Milestone: M21 → Future
Bulk reassigning form bugs to Alex
Assignee: pollmann → alexsavulov
Status: ASSIGNED → NEW
Attachment #2819 - Attachment is obsolete: true
Attachment #2820 - Attachment is obsolete: true
Attachment #2823 - Attachment is obsolete: true
Attachment #2896 - Attachment is obsolete: true
Attached file Testcase 3
WORKSFORME, build 2002-02-22-03 on Windows 98 SE.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → WORKSFORME
Whiteboard: [TESTCASE]
You need to log in before you can comment on or make changes to this bug.