Address book sort in reverse order with first click of column header

RESOLVED FIXED in Thunderbird 17.0

Status

MailNews Core
Address Book
--
minor
RESOLVED FIXED
8 years ago
5 years ago

People

(Reporter: Trevor Morgan, Assigned: aceman)

Tracking

({polish})

Trunk
Thunderbird 17.0
polish
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

1.47 KB, patch
mconley
: review+
Ian Neal
: review+
mconley
: ui-review+
Details | Diff | Splinter Review
(Reporter)

Description

8 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-GB; rv:1.9.1.7) Gecko/20091221 Firefox/3.5.7
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-GB; rv:1.9.1.5) Gecko/20091204 Lightning/1.0b1 Thunderbird/3.0

When clicking on a column header in the address book, the first click sorts by reverse alphabetic or high-to-low.  The second click sorts low-to-high.

Reproducible: Always

Steps to Reproduce:
Click on a column header in the address book (of an unsorted column).
Actual Results:  
The column is sorted Z..A or 9..0
A second click is needed to sort forwards

Expected Results:  
The column should be sorted 0..9 or A..Z on the first click.

The column sorting works as expected in the mail window - it's just the address book that's wrong.
Status: UNCONFIRMED → NEW
Ever confirmed: true
(Assignee)

Updated

5 years ago
Status: NEW → ASSIGNED
Component: Address Book → Address Book
Keywords: polish
Product: Thunderbird → MailNews Core
Version: unspecified → Trunk
(Assignee)

Comment 1

5 years ago
Created attachment 644760 [details] [diff] [review]
patch

This seems to do it.
Assignee: nobody → acelists
Attachment #644760 - Flags: ui-review?(mconley)
Attachment #644760 - Flags: review?(mconley)
(Assignee)

Comment 2

5 years ago
Comment on attachment 644760 [details] [diff] [review]
patch

The fix is in a /mailnews file so could affect Seamonkey.
Attachment #644760 - Flags: review?(iann_bugzilla)

Comment 3

5 years ago
Comment on attachment 644760 [details] [diff] [review]
patch

r=me
Attachment #644760 - Flags: review?(iann_bugzilla) → review+
Comment on attachment 644760 [details] [diff] [review]
patch

Review of attachment 644760 [details] [diff] [review]:
-----------------------------------------------------------------

Ah, beautiful.  Yes, this feels much better.  Thanks aceman.
Attachment #644760 - Flags: ui-review?(mconley)
Attachment #644760 - Flags: ui-review+
Attachment #644760 - Flags: review?(mconley)
Attachment #644760 - Flags: review+
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
https://hg.mozilla.org/comm-central/rev/5ab0c167661f
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 17.0
You need to log in before you can comment on or make changes to this bug.