[Tickets #9684] Re: Changing sort direction doesn't work
bugs at horde.org
bugs at horde.org
Tue Mar 22 10:49:09 UTC 2011
DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.
Ticket URL: http://bugs.horde.org/ticket/9684
------------------------------------------------------------------------------
Ticket | 9684
Updated By | Jan Schneider <jan at horde.org>
Summary | Changing sort direction doesn't work
Queue | IMP
Version | Git master
Type | Bug
State | Assigned
Priority | 2. Medium
Milestone | 5.0
Patch |
Owners | Michael Slusarz
------------------------------------------------------------------------------
Jan Schneider <jan at horde.org> (2011-03-22 10:49) wrote:
>> We always used to support
>> sorting in search results, and we must still support it. We don't
>> have to support sorting across all mailboxes, agreed, though this
>> would be nice long-term too. But at least within in the results per
>> mailbox, sorting must be possible.
>
> Well I misspoke - we were sorting search results (other than by
> thread). However, this is totally broken behavior because it
> completely breaks search display in every interface BUT imp. e.g.
> in dimp, sorting appears to be totally random.
>
> This is NOT going to work. I don't care what we have done in the
> past, but this isn't going to fly. We either need to to abandon ALL
> sorting in the search views or implement client side sorting.
I agree, without the mailbox headers, sorting in other views than IMP
looks random. I don't want to drop sorting of search results either,
so client side sorting seems to be the only way out.
> And in imp, we need to get rid of the mailbox headers separating the
> various search mailbox results. This is entirely inconsistent with
> the way we display search results in the other views. Mailboxes
> should be irrelevant in search results - the messages are all that
> matters.
True, but at the moment, without client-side sorting, this is the only
way to indicate how the sorting works. I assume that client sorting is
a big task, so maybe we can add similar indicators for the other views
too?
Short-term I'm fine with how it works now, even if it's kinda broken,
because this how it works in IMP 4/DIMP.
More information about the bugs
mailing list