62556 - Scrolling an overflow:scroll page causes reflow

Status: RESOLVED WORKSFORME

(Core :: XUL, defect)

Description

[Ilya Konstantinov]

This is the minimal testcase I could minimize it to, so it would still exhibit
the problem.

Testcase contains:
A right-aligned table cell containing a box of fixed dimensions with text, which
would have a scroll (CSS overflow:scroll) if the content doesn't fit the box' size.

Result:
On my system, clicking the scroll bar causes enlarging of the table cell's
width, while the box dimensions remain the same. The table cell expands every
time the scroll bar is used, and apparently expands endlessly.

Comment 1

[Ilya Konstantinov]

Attachment: testcase

Comment 2

[Asa Dotzler [:asa]]

I was unable to see any changes to the testcase clicking the scrollbar with
,inux or win32 builds 121508/10.  what build did you test this with?  over to
Style System for triage.

Comment 3

[Ilya Konstantinov]

Using build 2000121521 and still the same problem. Also, using standard X Type1
fonts for Western (ISO-8859-8), the character set of this testcase, though I
have TrueType fonts via FreeType for other character sets.

Comment 4

[Keyser Sose]

Verified
Platform: PC
OS: Linux 2.2.16
Mozilla Build: 2000121908 M18 Trunk Build

You have to click on the vertical scrollbar. For Me double click did it. Watch it expand horizontally. Marking NEW.

Comment 5

[HJ]

Attachment: Try this one

Comment 6

[HJ]

Attachment: Try this one

Comment 7

[HJ]

I took a short dive into this problem.
If you are on top or the bottom, and you keep clicking, then there's an overflow
on the page. But the output of this example stays the same, only the screen
width is growing (watch your browser's window scrollbar grow)! And the output
for the classic and modern themes are not the same! So this needs to be checked
and fixed.

How did I attach two of them?? Sorry for that!

Comment 8

[Hixie (not reading bugmail)]

Netscape's standard compliance QA team reorganised itself once again, so taking 
remaining non-tables style bugs. Sorry about the spam. I tried to get this done 
directly at the database level, but apparently that is "not easy because of the 
shadow db", "plus it screws up the audit trail", so no can do...

Comment 9

[Pierre Saslawsky]

My apologies for having neglected this bug for so long...

To reproduce this bug, load the attached testcase at
http://bugzilla.mozilla.org/showattachment.cgi?attach_id=21143 and repeatedly 
click on any arrows from the scrollbars.  After 8 clicks, the browser window 
displays a horizontal scrollbar. The number of clicks doesn't change with the 
font size used within the text field.

Reassigned to evaughan/XPToolkit

Comment 10

[Peter Trudelle]

->karnaze per evaughan

Comment 11

[Peter Trudelle]

*** Bug 63513 has been marked as a duplicate of this bug. ***

Comment 12

[Boris Zbarsky [:bzbarsky]]

ccing evaughan, since karnaze reassinged bug 63513 to him/her

Comment 13

[Boris Zbarsky [:bzbarsky]]

nominating, OS ==> all, since people have seen this on all Linux/MacOS/Win98

Comment 14

[Boris Zbarsky [:bzbarsky]]

*** Bug 83135 has been marked as a duplicate of this bug. ***

Comment 15

[Ryan Frishberg]

Hey guys,

I found something similar, but it shows a more serious problem.  When I put two 
div elements with style="overflow:scroll" and filled them with text and put 
them in a table, it's screwed up.

#1.  If you click the vertical scroll bar of the second element, it changes the 
second element's width.

#2.  If you click the vertical scroll bar of the first element, it changes the 
second element's width.

Comment 16

[Ryan Frishberg]

Attachment: overflow:scroll in table

Comment 17

[Moiz Dohadwala]

Check out this url. this page very nicely articulates the issue.

http://www.xs4all.nl/~ppk/css2tests/overflow.html. 

credits to the original author.

Comment 18

[Moiz Dohadwala]

Forgot to mention this. It happens on Win 2K Mozila 0.9.3 Build ID: 2001080110

Comment 19

[Ilya Konstantinov]

Still happens on 0.9.4 (Linux).

Comment 20

[Boris Zbarsky [:bzbarsky]]

I just checked out a build with the fix to bug 73235.  Every single testcase in
this bug is working correctly for me in that build.  No problems whatsoever.

Comment 21

[Chris Waterson]

Yeah, I agree. This is a dup.

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

Comment 22

[Tom Mraz]

I'm sorry but the http://www.xs4all.nl/~ppk/css2tests/overflow.html
isn't fixed. 
Or rather it's fixed only partially - the columns with overflow:auto/scroll
elements get wider when you click on the arrows but only for a while. Then the
widening stops.
(I don't know if the bug 73235 shouldn't be reopened rather then this one.)

Updating the URL to test.

Comment 23

[Chris Waterson]

Taking.

Comment 24

[Boris Zbarsky [:bzbarsky]]

Tom, what platform are you on?  That testcase worksforme with linux build
2001-09-30-06...

Comment 25

[kasper]

It works for me with 2001100108, RedHat 7.1

Comment 26

[Chris Waterson]

The test cases appear to work fine, but the ``overflow: scroll'' case from the
original URL (http://www.xs4all.nl/~ppk/css2tests/overflow.html) is causing
problems.

Comment 27

[kasper]

Sorry, still doesn't work with 2001100108. It's better now, but the test at
http://www.xs4all.nl/~ppk/css2tests/overflow.html is still causing problems.
http://www.denlilleskole.dk/index.php?page=12 did not work before, it's perfect now.

Comment 28

[Chris Waterson]

rods, this is more <select> fun. I think what's happening here is that the
border/padding for the ScrollFrame is being added twice, then removed only once,
for each incremental reflow.

Comment 29

[Boris Zbarsky [:bzbarsky]]

*** Bug 104665 has been marked as a duplicate of this bug. ***

Comment 30

[[not reading bugmail]]

note: http://www.xs4all.nl/~ppk/css2tests/overflow.html - scrollbars are working
properly in 2001102203 for me on W2K.  Could have been another checkin that
fixed or made this work in this build.

-dennis

Comment 31

[Tom Mraz]

Are you sure? You have to have the browser window at least 900 pixels wide to
see the unwanted effect.

Comment 32

[[not reading bugmail]]

yup, I see it now after closing the sidebar it doesn't work.. so that is part of
the culprit of this page.. size of viewport or page width.. has a real effect on
this one.

-cuz

Comment 33

[rods (gone)]

Moving to Future

Comment 34

[Boris Zbarsky [:bzbarsky]]

This is likely causing bug 107902 on slate.msn.com

Comment 35

[[not reading bugmail]]

see some possible related plague mouse bugs (mouseover issues actually) in style
system such as fixed bug 69398 and fixed Layout bug 71979: adding content and
reflow/moving stuff; to possibly get some insight to the problem here.

Comment 36

[Thomas Hoppe]

Mhhh, this bug http://www.xs4all.nl/~ppk/css2tests/
still isn't fixed people and you want to release 1.0 soon... no this couldn't be
true!!

Comment 37

[[not reading bugmail]]

Tom,

no one has mentioned what the real problem is here yet that is causing a reflow
from mouse & scrollbar clicking linked with updating overflow, and browser
window viewsize, its kinda needs investigating first.

Comment 38

[[not reading bugmail]]

pause, well except waterson's comment #28 on <select>.  sorry for spam.

Comment 39

[Tom Mraz]

I think it was fixed a long time ago.
At least I couldn't replicate the bad behaviour on the testcases and the url.