76267 - going to next item in thread pane should autoscroll

Status: NEW

(SeaMonkey :: MailNews: Message Display, defect)

Description

[sairuh (rarely reading bugmail)]

this is a really useful feature in 4.x on linux [alas, not implemented afaik in
4.x for win32 or mac]. if i go to the next [non-visible] item in the thread
pane, 6.x/mozilla merely scrolls to display the next item --in 4.x it would
scroll half a pagefull [well, autoscroll down roughly half the height of the
thread pane].

will attach screenshots soon, but here are some steps to see this --try it out
in both 6.x/mozilla and 4.x on linux.

1. let's say i've selected the item that's visible [displayed] in the thread
view --mind you, this isn't the same as the last item in the thread pane.
2. hit N [or F] to go to the next item.

Comment 1

[sairuh (rarely reading bugmail)]

Attachment: comm 4.x [linux] - before going to next msg

Comment 2

[sairuh (rarely reading bugmail)]

Attachment: comm 4.x [linux] - after going to next msg

Comment 3

[sairuh (rarely reading bugmail)]

Attachment: 6.x - before going to next msg

Comment 4

[sairuh (rarely reading bugmail)]

Attachment: 6.x - after going to next msg

Comment 5

[Håkan Waara]

Seth, I will look into this.  Outliner has support for autoscroll, I think.

Comment 6

[Håkan Waara]

It should only autoscroll if one is navigating with the keys, right? Not when
clicking?

Comment 7

[sairuh (rarely reading bugmail)]

yes, in 4.x autoscroll occurs when using the keyboard --but not when clicking
the scrollbar in the thread pane [if that's what you're referring to].

Comment 8

[Mike Gratton]

Håkan, I'll have a look at this, if you don't mind.

Comment 9

[Håkan Waara]

be my guest!

Comment 10

[Mike Gratton]

Okay, I know how to fix this. I'm just awaiting an IP waiver from my employers
for any Moz code that I write. Once I have that, it'll take a few minutes to put
together.

Comment 11

[Mike Gratton]

Attachment: initial scroll-to-middle patch

Comment 12

[Mike Gratton]

The patch depends on bug 97110, but you can still test it by editing the patch
thusly:

+  var lastVisible= outlinerQueryInterface.getLastVisibleRow() -2;

(bug 97110 just outputs a last visible row count that's too big by 2).

Comment 13

[Wayne Mery (:wsmwk)]

still wanted?

Comment 14

[Karsten Düsterloh]

> still wanted?

I'd say yes, but I bet the patch is slightly bitrotted after almost 6 years. ;-)

Comment 15

[Mike Gratton]

Someone else is going to have a look at this, I really don't have the time/impetus to do so. :(