371629 - Star header will not un-highlight

Status: RESOLVED FIXED

(Thunderbird :: Mail Window Front End, defect)

Description

[Sherman Dickman]

Once I click the star header to sort messages by their star status, that header will not un-highlight once I click on another header.  Thus, I'll have two headers highlighted, one always being the star header.

Comment 1

[Sherman Dickman]

Using the following:
version 2 beta 2 (20070223)

Comment 2

[Scott MacGregor]

this might be a bug in the pinstripe theme. I can't reproduce this on windows. 

1.5.0.9 on the mac works ok so it is a regression.

Comment 3

[Scott MacGregor]

Hey Neil, I'm seeing some weirdness with tree widgets on the 1.8 branch and the trunk. Have you seen anything like this before?

* The sortDirection attribute is not getting removed from treecol-icons when the sort order moves to another column. I do see the sortDirection attribute get removed for non icon based tree-cols. DOM Inspector for Windows and mac both show the attribute getting left behind.

* On the Mac, when a treecol-icon has the sortDirection attribute set on it, I see a sort arrow getting drawn underneath the treecol icon which shouldn't be there. Windows doesn't suffer this problem.

Comment 4

[Neil Deakin]

Which tree is this? If it's the thread pane, it has it's own implementation of handling sort indicators. (UpdateSortIndicators in threadPane.js) so there's probably a bug somewhere there.

Comment 5

[Scott MacGregor]

(In reply to comment #4)
> Which tree is this? If it's the thread pane, it has it's own implementation of
> handling sort indicators. (UpdateSortIndicators in threadPane.js) so there's
> probably a bug somewhere there.

Ah, right you are Neil. It looks like the bug is in UpdateSortIndicators  for the issue Sherman is seeing. I've got a fix for that.

The 2nd issue with the sort arrow getting drawn underneath the tree col icon, I'll spin that off into a separate bug. 

Comment 6

[Scott MacGregor]

Attachment: branch only patch

UpdateSortIndicators scared me :)

This patch is for the branch only and it properly removes the sort indicator from all columns instead of just removing it from the columns to the right of the column we are going to sort by.

I have another patch for the trunk which does a bunch of cleanup on this method to make it a bit less scarier, but I don't want to put that on the branch.

Comment 7

[[e-mail]]

This is a regression from bug 256688.

Comment 8

[[e-mail]]

I can't actually reproduce the bug as described; the bug only applies if group by sort is in effect and changing the sort order always seems to turn that off.

Comment 9

[[e-mail]]

Comment on attachment 256736 [details] [diff] [review]
branch only patch

>+  for (var i=0; i < treeColumns.length; i++)
Nit: spacing around i = 0

>+    treeColumns.item(i).removeAttribute('sortDirection');
Nit: treeColumns[i] should work.

But I'd still like to know how to reproduce this ;-)

Comment 10

[Scott MacGregor]

Ok, this is fixed on the branch with Neil's review comments. I'm leaving the bug open until I have time to finish up my patch that cleans up this routine.

Comment 11

[Robert Kaiser]

Apparently this checkin broke SeaMonkey mailnews on branch. When I pulled a current tree, thread (and msg) pane stayed empty and mouse cursor stayed in busy state in the mail window. After backing out those changes to mailnews/base/resources/content/threadPane.* it works again.

Comment 12

[Scott MacGregor]

I suck. Checking in a fix right now.

Comment 13

[Scott MacGregor]

Fix checked in sorry for the mess guys. 

Comment 14

[Marcia Knous [:marcia]]

verified fixed on the 1.8 branch using the Tbird cand build, version 2.0.0.0 (20070326). I verified using a PPC Mac running 10.3.9. I am able to switch back and forth between the star header and other headers with no issues. Adding branch keyword.

Comment 15

[Magnus Melin [:mkmelin]]

Is it a problem on trunk? I don't see it on linux.

Comment 16

[Wayne Mery (:wsmwk)]

does not fail on trunk, so closing FIXED