Steps to reproduce: 1. Display a newsgroup as Messages > All 2. Change to Messages > Threads With Unread The sort order is switched, from ascending to descending (and vice versa?).
I have a patch for this and another evil threadpane bug, bug 79588. CCing folks for review.
Created attachment 47110 [details] [diff] [review] fix -- un-hardcode sortorder and threadmode where it's not needed.
r=doron on patch, nice catch
looks harmless enough. firstname.lastname@example.org
a=tor on behalf of drivers
fix checked in.
This bug can still be seen with build 2001082803. I did see this on bonsai the other day as checked in, but then it disappeared the same day.
Cristoph, do you have a testcase to reproduce this bug in new builds?
Nevermind - my mistake
Using aug29 commercial trunk build: If I leave the View|All as default threaded/ascending then switch to View|Threads with Unread, the view stays as threaded/ascending. If I click the thread column to set the View|All to threaded/descending, when I switch to View|Threads with Unread we reset to ascending. So, we do change/reset to ascending when changing the view. I'm not sure we save ascending/descending in a lot of cases, so I'm a bit confused about this particular fix -- we have an open bug about not saving on switching sorts (bug 80936). Wouldn't this (save across Views) just be an extension of that bug?
OK, I think this is OK to mark verified. In the cases I've tried since, I've used the Sort By menu to change Ascending/Descending and then I have no problem when switching views (threaded). So we may have an issue with the thread column not changing necessary sort state in code somewhere...? I'll play with that more and log a separate report.
Laurel, feel free to CC me on that bug if you find one.