72589 - View->sort->by->status does not sort correctly

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Message Display, defect)

Description

[fenella]

Linux (2001-03-19-11 mtrunk)
Win32 (2001-03-19-12 mtrunk)
Mac (2001-03-19-12 mtrunk)
1. Select a folder
2. Click on the Status column, it shows arrow up
Actual result: Status column does not sort correctly
Expected result: Read threads should be grouped at the bottom of the column
3. Click on the Status column again, it shows arrow down
Actual result: Status column does not sort correctly
Expected result: Read threads should be grouped at the top of the column

This occurs on all platforms.

Comment 1

[fenella]

Add perf to the whiteboard.

Comment 2

[(not reading, please use [e-mail] instead)]

accepting.

I'm currently sorting by StatusOffset!  (zoinks!)

I've just brought over the code from 4.x to get the status flags, and create a
sort value for them.

patch coming soon.

Comment 3

[(not reading, please use [e-mail] instead)]

Attachment: here's the fix, stolen from 4.x

Comment 4

[(not reading, please use [e-mail] instead)]

whoops, new patch on the way.

Comment 5

[(not reading, please use [e-mail] instead)]

Attachment: patch, v2.

Comment 6

[(not reading, please use [e-mail] instead)]

Attachment: patch, v3 (again, without my fix for #72592)

Comment 7

[(not reading, please use [e-mail] instead)]

fixed.

Comment 8

[fenella]

Linux (2001-03-23-05 mtrunk)
Win32 (2001-03-21-06 mtrunk)
Mac (2001-03-23-08 mtrunk)
Problem is gone.

Comment 9

[(not reading, please use [e-mail] instead)]

*** Bug 59599 has been marked as a duplicate of this bug. ***