361048 - Switching from Account Central to folder does not scroll to newest message

Status: RESOLVED FIXED

(SeaMonkey :: MailNews: Message Display, defect)

Description

[[e-mail]]

Please bear with me on this, it's a bit tricky to explain. The effect relies on certain fairly common settings. First you will need a folder without a selected message, or you need to disable the last selected message pref. The folder also needs to be sorted by date or order received (or thread). In these circumstances when you open the folder it should scroll to the very end of the message list.  You also need to show the message pane by default. The problem arises when you were previously viewing Account Central. In this case although it is hidden the thread tree still has the "full" height of the window. When you then switch to the aforementioned folder the thread tree is resized but it does not update internally until the next reflow. This makes it scroll based on its old size meaning that some of the newest messages are not actually visible.

Comment 1

[[e-mail]]

Attachment: Proposed patch

Comment 2

[Karsten Düsterloh]

Comment on attachment 245830 [details] [diff] [review]
Proposed patch

>Index: mailWindow.js
>===================================================================
>+    if (!threadPaneSplitter.hidden && threadPaneSplitter.getAttribute("state") != "collapsed") {
>         GetMessagePane().collapsed = false;

Since bracket style (and indentation) in this file are quite inhomogeneous, I'm in favour of using the overall prevalent style ie. no brackets at the end of line...

r=me either way.

Comment 3

[[e-mail]]

Fix checked in to the trunk.

Comment 4

[Ian Neal]

Comment on attachment 245830 [details] [diff] [review]
Proposed patch

first a=me for SM1.1

Comment 5

[Robert Kaiser]

Comment on attachment 245830 [details] [diff] [review]
Proposed patch

a=me for SeaMonkey 1.1

Comment 6

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8.1.1pre) Gecko/20061122 SeaMonkey/1.1b] (nightly) (W98SE)

V.Fixed on MOZILLA_1_8_BRANCH, between these two builds.

[Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8.1.1pre) Gecko/20061123 SeaMonkey/1.1b] (nightly) (W98SE)

*****

Would Thunderbird need the same fix ?
From a code point of view, it seems it could.
(Else, why would this work in TB and not in SM ?)

Comment 7

[[e-mail]]

(In reply to comment #6)
>Would Thunderbird need the same fix ?
Apparently it already has its own bug filed for this.

Comment 8

[Mike Cowperthwaite]

(In reply to comment #7)
> (In reply to comment #6)
> >Would Thunderbird need the same fix ?
> Apparently it already has its own bug filed for this.

Bug 321551.  I finally got around to porting this patch over, hopefully not too late for TB 2.0.