Sorting in descending order in the thread pane does not work in today's build (Build ID 1999120815). Steps to reproduce: 1. Launch Seamonkey 2. Start Mail (Tasks | Mail) 3. Select Inbox for email account (and supply password when prompted) 4. After headers have loaded, sort by date (by pressing on "Date" column header). Then, try to reverse the sort order (by pressing on the "Date" column header again) to get descending sort by date. Expected result: Message headers sort by date in descending order. Actual result: Message headers remain sorted by date in ascending order.
*** Bug 21256 has been marked as a duplicate of this bug. ***
Reassign to putterman, cc rjc
I'm reassigning to rjc and cc'ing hyatt. When David added the column chooser it broke this part of sorting. BTW, the column chooser is very cool. I was just playing around with it. In order for the sorting code to set the new sort direction for the column it does this in a loop that iterates over the children of the tree looking for columns which used to be children of the tree. Now that there's a column chooser, the columns are no longer children of the tree but instead of a treecolgroup.
The sort code should be smart enough to look for the columns either as children of the tree OR as children of a treecolgroup that is a child of the tree.
Linux Redhat (1999-12-09-10 M12) Sort by Date, Sender, Subject, Status and Size in decending order also do not work. Have not tested Win_nt and Mac yet.
It won't work on any platform or for any thread pane column.
This is a big regression. A lot of folks read their mail in reverse dated order (cause natural order, and reverse order or not yet supported). Anyway, without this, I need to sit through the slow delete process, and then select a new message (assuming I use forward date sorted, and then struggle to read my messages backwards). This makes reading email REALLY tedious for me. I marked the original bug as dogfood, but when it was dupped, they dropped the dogfood tag. Please consider for dogfood. Thankks, Jim
Just for you, Jar. :^) Scott (Putterman), can you try the proposed fix that I've attached to this bug and see if sorting then starts working again for Mail/News? [I'm working at home today, and don't have SERA access on my Mac, otherwise I'd test it for Mail/News myself.]
looks good. I was able to sort the date both ascending and descending. thanks for the fix.
chofmann and/or jar, will you approve this for checkin into M12?
ok, checkin. are there other places that use the code being fixed or is it localized?
There are lots of places which use trees and sort on columns, however I suspect that at the moment only mail/news uses <treecolgroup> tags at the moment, since Hyatt just added them. With these changes, I've tested sorting in other areas (bookmarks, searching) and things work. I'm about to go grab some dinner, so I'll check in the fix later this evening when I get back home.
moving on to the m12 radar
Fix checked in.
Linux Redhat 6.0 (1999-12-13-08 M12) Win_nt (1999-12-13-09 M12) Mac (1999-12-13-08 M12) This problem has been fixed.