Closed Bug 202644 Opened 18 years ago Closed 18 years ago

Column sort behavior should parallel mailnews

Categories

(Other Applications :: ChatZilla, defect)

x86
All
defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mrmazda, Assigned: rginda)

Details

Attachments

(2 files)

0.8.26 Linux & OS/2

In mailnews, clicking a headers column heading causes a sort on that column.
Clicking the same column header again turns the triangle upside down, reversing
the sort order.

In CZ, sort ascending and sort descending are accompanied by a strange third
click option that removes the triangle from all columns. When the nick column is
selected for "non" sort, the first item listed is the alphabetical lowest
(closest to a), the last item listed is the alphabetical highest (closest to
zzzzzz), and all items in between are reverse alpha sorted.

The third, "non" sort option has a perplexing purpose and should either be
eliminated or made random.
Attached patch patch v1Splinter Review
Attachment #121162 - Flags: review?(rginda)
The natural order of the channel list is the order the server sent them to us,
which, I believe, is the join order.  I don't know if that order is interesting
enough to include in the sort options or not.
Attachment 121064 [details] shows the "natural" "un" sorted order looks like a broken
reverse alpha sort, a repeatable order that users simply aren't going to find a
use for or understand. I doubt that joins are always a* followed by perfect
reverse alpha except for z* last like the ss shows.
I now see that join order is the apparent sort order when selecting another
channel while the triangle is missing. But, as soon as you start clicking the
nick header to change the sort order, you do not again get the join order when
the triangle disappears. Instead you get what the screenshot shows and comment 4
describes. Join order might be useful to some, but what user would understand
why/that join order is only possible when a channel tab is freshly selected?
Either make join order redisplay when cycling through the sort options, or
eliminate it.
Comment on attachment 121162 [details] [diff] [review]
patch v1

will be included in latest xpi
Attachment #121162 - Flags: review?(rginda) → review+
marking fixed, 0.8.34 checked in.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Product: Core → Other Applications
You need to log in before you can comment on or make changes to this bug.