1323 - We aren't "compounding" frameset scrollbars

Status: VERIFIED FIXED

(Core :: Layout: Images, Video, and HTML Frames, defect, P1)

Description

[Angus Davis]

This URL has a frameset like this:

 ---------
|___ A____|
|  |      |
|B |      |
|  |  C   |
|  |      |
-----------

Frame A should not have any scrollbars (has scrolling=no attribute)
Frame B should have auto scrolling
Frame C should also have auto scrolling. Frame C is the right most frame.

Becuase Frame C is along the right, it's scrollbar should be to the immediate
left of the Viewer's window border, where a normal scrollbar would appear.
Instead, the frameset document is getting a scrollbar all its own, and then to
the left of *that* scrollbar is another scrollbar for frame C!

You might need to resize (enlargen) the viewer window to see this bug.

I'm using 11-9-98 Optimized bits.

Comment 1

[karnaze (gone)]

I can't fix the bug because it is crashing in javascript. Please reassign it
back to me when the crash is fixed. I've cc'ed Rick G since the parser is in the
stack as well.

Comment 2

[vidur (gone)]

The script problem is because of bug 1223, assigned to [e-mail]. Since I
don't want to lose the original bug, I'm not resolving it as a DUP. I am,
however, assigning it to Scott so that he can return it to Chris Karnaze once
the bug is fixed.

Comment 3

[leger]

Setting all current Open Critical and Major to M3

Comment 4

[Paul MacQuiddy]

per leger, assigning QA contacts to all open bugs without QA contacts according
to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser

Comment 5

[Scott Furman]

Same as #1223

Comment 6

[chris hofmann]

lets try and get this for M4.

Comment 7

[sujay]

re-assigning chrisd D as qa_contact.

Comment 8

[Mike McCabe]

Setting this bug and friends to m5.  I hope to get to it soon.

(But I'm guessing it reflects a subtle change to the semantics of javascript
variable lookup, so I don't want to make that change without a full run of the
javascript testsuite.)

Comment 9

[Mike McCabe]

Whoops, actually changing the setting...

Comment 10

[Mike McCabe]

These bugs seem to be different aspects of 1223.  Shaver has graciously taken
1223; reassigning these to him.

(thanks, Mike)

Comment 11

[chris hofmann]

moving to m6

Comment 12

[shaver-do-not-use-this-account]

1223 is fixed, so this is back to Chris.

Comment 13

[karnaze (gone)]

Fixed with recent checkins.

Comment 14

[rubydoo123]

tested and verified the fix.