Sort order changed if you change thread-mode

VERIFIED FIXED in mozilla0.9.4

Status

SeaMonkey
MailNews: Message Display
--
major
VERIFIED FIXED
17 years ago
13 years ago

People

(Reporter: Håkan Waara, Assigned: Håkan Waara)

Tracking

Trunk
mozilla0.9.4
x86
All

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

17 years ago
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?).
(Assignee)

Comment 1

17 years ago
I have a patch for this and another evil threadpane bug, bug 79588.

CCing folks for review.
Assignee: sspitzer → hwaara
Keywords: patch
(Assignee)

Comment 2

17 years ago
Created attachment 47110 [details] [diff] [review]
fix -- un-hardcode sortorder and threadmode where it's not needed.
(Assignee)

Updated

17 years ago
Target Milestone: --- → mozilla0.9.4

Comment 3

17 years ago
r=doron on patch, nice catch
looks harmless enough. 
sr=ben@netscape.com

Comment 5

17 years ago
a=tor on behalf of drivers
(Assignee)

Comment 6

17 years ago
fix checked in.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Updated

17 years ago
QA Contact: esther → laurel

Comment 7

17 years ago
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.
(Assignee)

Comment 8

17 years ago
Cristoph, do you have a testcase to reproduce this bug in new builds?

Comment 9

17 years ago
Nevermind - my mistake

Comment 10

16 years ago
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?

Comment 11

16 years ago
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.
Status: RESOLVED → VERIFIED
(Assignee)

Comment 12

16 years ago
Laurel, feel free to CC me on that bug if you find one.
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.