43459 - Trees can't always scroll to the bottom

Status: VERIFIED FIXED

(Core :: XUL, defect, P3)

Description

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

I'm very surprised not to see a bug on this.  Since I can't find one, I'm filing
what may be a duplicate.  But I think it's important...

DESCRIPTION:  It is often impossible to scroll to the bottom of a tree.
Sometimes resizing the window up and down a bit will make it possible, but
sometimes it won't.

STEPS TO REPRODUCE:
 * load ftp://ftp.mozilla.org/pub/mozilla/nightly/
 * scroll to the bottom

ACTUAL RESULTS:
 * the newest builds are not visible

EXPECTED RESULTS:
 * see the newest builds

DOES NOT WORK CORRECTLY ON:
* Win98 mozilla 2000-06-21-08-M17

ADDITIONAL INFORMATION:

I am nominating this bug for *dogfood* because it prevents me from using the
browser for:

 * downloading a new build of the browser (as described above)

 * using the browser for reading mail, since I can't see the newest messages,
which are the ones I most want to see

Comment 1

[Jay Patel [:jay]]

Putting on [dogfood+] radar. 

Comment 2

[Mike Pinkerton (not reading bugmail)]

tested on mac and win2k on 6/26 builds, this works just fine for me. the entire 
tree is visible. note the bug was reported with a 6/21 build, so this may have 
been fixed by more grid bug fixes that landed later in the week.

marking WFM, pushing back on dogfood+ characterization in case it's reopened.

Comment 3

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Reopening and putting back [dogfood+].  I can reproduce this pretty reliably 
(but not 100%) in my Win98 2000-06-26-09-M17 release build.  If you want to see 
it, come down to my cube (right at the bottom of the stairs near the bridge of 
soda cans) tomorrow.  I'm pretty sure I see it in my Linux debug build too (my 
Win98 debug build is somewhat out of date).

Comment 4

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Really putting back the [dogfood+]

Comment 5

[John Morrison]

I do see this, on a hit-and-miss (~30%) basis, on win95 with today's 
comm. opt. build, when I drag the thumb to the bottom.

However, I cannot reproduce it on today's mac or linux builds (all tested 
using ftp://ftp.mozilla.org/pub/mozilla/nightly/). 

I can workaround on windows by scrolling back to the top, and then clicking
below the thumb to move to the bottom of the tree. 

Removing [dogfood+] as I think Mike's point was that "if reopened, I want 
another assessment by PDT as to whether this is dogfood".

Comment 6

[Jay Patel [:jay]]

Putting on [dogfood+] radar. PDT says dogfood+

Comment 7

[scalkins]

This affects aim buddy list as well in commercial.
AIm tets case is :
-start browser
-Tasks|Instant Messenger
-Add some buddy groups and buddies (enough so that you need to scrol down)
-if you try and scroll down you can't completely do it

Comment 8

[scalkins]

BTW, I also see this in AIM on  Linux build  2000-06-27-20M17 and Mac build 
2000-06-27-11 M17. setting platforms to All. In most cases you can't scroll down 
to the last buddy in the list using the test case for AIM.

Comment 9

[scalkins]

cc'ed vishy, scalkins, benc

Comment 10

[Viswanath Ramachandran]

Seems to happen in MailNews as well, cannot scroll to the bottom of the list of 
messages in the thread pane. Also the scroll bar occassionally disappears in the 
thread pane - reappears if you resize - do I need to file a seperate bug on 
that? Thanks, Vishy

Comment 11

[John Morrison]

No. The disappearing scrollbar is bug #43467

Comment 12

[Mike Pinkerton (not reading bugmail)]

we're hoping that the fix for 43169 might fix this one too. we're having a hard 
time reproducing it to fix it, although we've all seen it enough to know it's for 
real.

hyatt and I both contest that since (with a bit of wrangling) it is always 
possible to get to the row you want to see, that this isn't dogfood, perse, but 
pdt seems to think it is.

due to the fact that we're having a hard time reproducing to fix, if the patch 
for 43160 doesn't fix it, it probably won't get fixed until at least 7/11 when 
hyatt returns from vacation.

Comment 13

[Mike Pinkerton (not reading bugmail)]

hyatt is stumped, the fix for our other dogfood bug didn't help at all. bryner is 
helping on it, but...

Comment 14

[David Hyatt]

I should say also that with the specified URL (the FTP site), I was unable to 
reproduce the bug.  I can reproduce it in mail, but not reliably.

Comment 15

[djk]

I just put entered a comment to bug 39710, which looks similar to this one.

Comment 16

[Stephen P. Morse]

Bug 35560 also looks similar to this one and might be related.  Not sure if 
it's a dup or not.  That bug is 100% reproducible.

Comment 17

[Brian Ryner (not reading)]

I've been looking at this with evaughan... I think I have an idea what's going
on.  It's possible for the tree's preferred size to be too small, which causes
the scroll frame to only scroll part way down.  The fix is rather involved but I
have some progress.

Comment 18

[Brian Ryner (not reading)]

Checked in fix.

Comment 19

[Viswanath Ramachandran]

Thanks bryner! Fixes an AIM bug in bugscape that depends on this as well. 
Vishy

Comment 20

[John Morrison]

I can still see this on occasion for bookmark manager, ftp, and mail 

  1) only on win32 (not on mac or linux) 

  2) only by grabbing the thumb, dragging up, and then sharply snapping down
     to the bottom of the tree.

  3) I can always recover by going back to the top of the tree, and then  
     dragging the thumb at a normal speed to the bottom of the tree again

Reopening, but removing [dogfood+] -- I am not prevented from using the product
to do these tasks, only slightly inconvenienced. 

Nominating nsbeta2, as it would be good to finalize this fix for PR2, but 
I would concede that this is not a 'pull off the wire' bug. (IMHO. If others
(perhaps mailnews|aim people) have a stronger opinion, they should add their 
argument here). 

Comment 21

[scalkins]

It's much better in Aim, but it did happen to me once on Winnt build yesterday 
for Aim,...In my case trying to scroll up. Resizing the Aim window corrected it 
though, where before that did not have any effect.

Comment 22

[Jay Patel [:jay]]

Putting on [nsbeta2-][dogfood-] radar. Not critical to beta2. There's a 
workaround right?

Comment 23

[David Hyatt]

I nailed this. (hyatt says with false bravado...)

Comment 24

[John Morrison]

I, um, er, can still reproduce this, with the same caveats noted 7/12 (only
on windows, mailnews|ftp|bookmarks, only with a scrollup then sharp scrolldown, 
can always recover and get to the bottom). Sorry (maybe general perf work will 
cure this somewhat).

Comment 25

[Peter Trudelle]

We have too many larger fish to fry, moving to future

Comment 26

[Brian Ryner (not reading)]

I think hyatt's changes for fast tree scrolling fixed the last remnants of this
bug.

Comment 27

[Brian Ryner (not reading)]

i haven't seen this in a long time.  marking verified.