Closed Bug 377388 Opened 17 years ago Closed 17 years ago

On startup, message list doesnt scroll to show last message in classic and wide view

Categories

(Thunderbird :: Mail Window Front End, defect)

x86
Windows XP
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: talk2sk, Assigned: mscott)

Details

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.3) Gecko/20070309 Firefox/2.0.0.3
Build Identifier: Mozilla Thunderbird version 2 beta 2 (20070116)

When I run Thunderbird it comes up with my default account's Inbox, but the list of messages is not scrolled properly to the last message. This happens only in classic and wide view, and not is vertical view. In vertical view I can see the last message at the bottom on the list, whereas in the other views it looks like the message list is still scrolling to the last message as if the list was in vertical view.

Reproducible: Always

Steps to Reproduce:
1.Run thunderbird
2.Change view layout 
3.Retry
Actual Results:  
In vertical layout the problem does not appear. When starting it in other modes the problem appears

Expected Results:  
The last message should always be scrolled to irrespective of the layout.

On analyzing the problem further I have found that when EnsureRowInThreadTreeIsVisible is called to set the last message to be visible, it doesnt set it correctly, probably because the height of the threadTree.treeBoxObject is not computed properly. The relevant code is in threadPane.js, in this code a DUMMY ACCESS to tree.treeBoxObject.height solves the problem comprehensively. Example fix is below [though this fixes the problem, it doesnt appear to be the true bug]:

FILE: threadPane.js, FUNCTION EnsureRowInThreadTreeIsVisible(index)


function EnsureRowInThreadTreeIsVisible(index)
{
  if (index < 0)
    return;

  var tree = GetThreadTree();

  //THIS IS A HACK
  var dummyCheck = tree.treeBoxObject.height;

  //THIS CAN ALSO BE USED
  //dump("TreeHeight=" + tree.treeBoxObject.height + "\n");

  tree.treeBoxObject.ensureRowIsVisible(index); 
}

I have a screen size of 1920x1200, on Windows XP SP2.
Version: unspecified → 2.0
Unable to duplicate problem in 2.0.0 Release version
Status: UNCONFIRMED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.