292371 - Documents grow too wide with overflow:auto

Status: VERIFIED FIXED

(Core :: Layout, defect)

Description

[Robert Kaiser]

This is a followup to bug 273392 and a regression of bug 240176

In my Linux build from today, the testcases of 273392 stopped jumping around as
expected after the fix for bug 240176 but now all those testcases are too wide
for the screen and show a vertical scrollbar though it's not needed. The
documents have grown in width by approx. the width of a scrollbar.

This is true for those two simplified testcases:
https://bugzilla.mozilla.org/attachment.cgi?id=168031
https://bugzilla.mozilla.org/attachment.cgi?id=168032

as well as for the original page where I saw the other bug happening:
http://www.fynf.at/?vd=3

all of those use overflow:auto along with a padding: 1em; on a div containing
some other blocks.

Comment 1

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Probably fixed by the patch in bug 47710

Comment 2

[Wevah (Nate Weaver)]

You can see a rather extreme case of what appears to be this bug at
http://www.scarygoround.com/ (the document is far too wide). Getting rid of the
div with class="scroll" makes the problem go away.

Comment 3

[Wevah (Nate Weaver)]

Attachment: Reduced SGR testcase.

This might not be exactly the same bug, but it's probably similar (or at least
it regressed at the same time). Here's a reduced testcase from the above URL,
anyway.

Comment 4

[Uri Bernstein (Google)]

You guys know about bug 292370 (esp. bug 292370 comment 10), right?
Any chance this is a dupe?

Comment 5

[Robert Kaiser]

(In reply to comment #4)
> You guys know about bug 292370 (esp. bug 292370 comment 10), right?
> Any chance this is a dupe?

Could probably be related, but I don't think it's a dupe, those cases look very
different from the HTML/CSS side... even if if might go back to the same pieces
of backend code. roc might know more there...

Comment 6

[Robert Kaiser]

(In reply to comment #0)
> ... now all those testcases are too wide
> for the screen and show a vertical scrollbar ...

Sorry for that mistake, I meant a horizantal scrollbar here...

Comment 7

[Robert O'Callahan (:roc) (email my personal email if necessary)]

The testcases in comment #0 are fixed by the patch in bug 47710.

The attached testcase is fixed by the patch in bug 292370.

Comment 8

[Robert Kaiser]

Yes, this is fixed now that the patch for bug 47710 is in.

Comment 9

[Stephen Donner [:stephend] Not actively reading bugmail]

This has been fixed a while back; verified with build 2005-05-16-06, Windows XP
using Seamonkey trunk.