6.0.0-beta1
7/10/25

[#10683] Logical sorting of special folders
Summary Logical sorting of special folders
Queue IMP
Queue Version Git master
Type Enhancement
State Resolved
Priority 1. Low
Owners slusarz (at) horde (dot) org
Requester jan (at) horde (dot) org
Created 10/27/2011 (5005 days ago)
Due
Updated 11/08/2011 (4993 days ago)
Assigned
Resolved 11/01/2011 (5000 days ago)
Milestone
Patch No

History
11/08/2011 09:38:56 AM Jan Schneider Comment #27 Reply to this comment
Looks perfect now. Time for a release :)
11/08/2011 07:01:21 AM Git Commit Comment #26 Reply to this comment
Changes have been made in Git for this ticket:

Bug #10683: Sort sent-mail by abbreviated label

  1 files changed, 1 insertions(+), 1 deletions(-)
http://git.horde.org/horde-git/-/commit/7a4722c7a843259664a996262933a33b6f8820e1
11/08/2011 07:01:17 AM Git Commit Comment #25 Reply to this comment
Changes have been made in Git for this ticket:

Revert "Bug #10683: Default sent_mail_folder should be retrieved from 
identities, not prefs value"
This reverts commit 6c07b3fbb9eadcd532d52757e3fe3476853a464e.

  1 files changed, 1 insertions(+), 1 deletions(-)
http://git.horde.org/horde-git/-/commit/915b6ec74738c07a26b4ef1ac9b6b3f6a1d32a91
11/08/2011 07:00:00 AM Michael Slusarz Comment #24 Reply to this comment
Oh, and one thing I noticed: even though the "normal" sent mail 
folders are correctly sorted now, the translated folder is not. In 
my case it happens to be the first in the list, so this might be 
intended. Just checking.
We are sorting by *label*, so as far as I can tell the mailboxes are 
sorted properly.  Maybe the issue you are describing is that we are 
thereafter *viewing* by the abbreviated label.

Thus these mailboxes:

IN.Zsent
Sent

Would be viewed in this order in dimp:

Zsent
Sent

Maybe sorting by abbrev_label is the more appropriate thing to do...
11/03/2011 06:15:41 PM Jan Schneider Comment #23 Reply to this comment
This is not exactly the same issue, but might be related: the 
sent-mail drop down in the compose window is a bit "confused" at the 
moment. It doesn't display the translated labels, but it puts the 
special folders where they would be *if* they were translated.
11/03/2011 05:49:33 PM Jan Schneider Comment #22 Reply to this comment
Oh, and one thing I noticed: even though the "normal" sent mail 
folders are correctly sorted now, the translated folder is not. In my 
case it happens to be the first in the list, so this might be 
intended. Just checking.
11/03/2011 05:46:36 PM Jan Schneider Comment #21 Reply to this comment
I did notice one issue in the label code - we were not loading the 
default identity's sent-mail mailbox but rather the sent-mail 
mailbox defined by prefs.php by default.  So this has been changed; 
not sure if this fixes things (doesn't change any behavior for me).
I think this behavior makes more sense though than using the default 
identity's sent-mail folder for translation. Rationale: by default all 
identities use the sent mail folder from the prefs. If an identity has 
a different sent mail folder, this one was explicitly 
picked/created/renamed by the user. Chances are high that he used a 
certain name to distinct it from the default sent mail folder. That's 
why it should not use the default translated label.
11/02/2011 11:31:24 PM Michael Slusarz Comment #20 Reply to this comment
For me the only brokenness is the sent mail folder labels not being 
sorted alphabetically.
Fixed.
11/02/2011 11:30:57 PM Git Commit Comment #19 Reply to this comment
Changes have been made in Git for this ticket:

Bug #10683: Sort multiple special mailboxes in a single category by 
their labels

  1 files changed, 10 insertions(+), 3 deletions(-)
http://git.horde.org/horde-git/-/commit/1c79c9836183734b729e5ce0d75ed1910e4cad84
11/02/2011 10:25:25 PM Jan Schneider Comment #18 Reply to this comment
Depends what you consider broken. For me the only brokenness is the 
sent mail folder labels not being sorted alphabetically.
11/02/2011 08:55:18 PM Git Commit Comment #17 Reply to this comment
Changes have been made in Git for this ticket:

Bug #10683: Default sent_mail_folder should be retrieved from 
identities, not prefs value

  1 files changed, 1 insertions(+), 1 deletions(-)
http://git.horde.org/horde-git/-/commit/6c07b3fbb9eadcd532d52757e3fe3476853a464e
11/02/2011 08:55:16 PM Michael Slusarz Comment #16 Reply to this comment
But this is also a first-level mailbox.  So it seems the issue is 
that all mailboxes underneath a special mailbox are labeled as a 
special mailbox.  Whatever I think about the sent-mailbox labeling, 
this is simply flat-out wrong and needs to be fixed.
And now I am even more confused.  Because I can't reproduce.  I only 
ever see one 'Sent' mailbox - all others are displayed with their 
display names.

I did notice one issue in the label code - we were not loading the 
default identity's sent-mail mailbox but rather the sent-mail mailbox 
defined by prefs.php by default.  So this has been changed; not sure 
if this fixes things (doesn't change any behavior for me).
11/02/2011 07:39:40 PM Michael Slusarz Comment #15 Reply to this comment
It may be caused by the fact that they are all second-level folders. 
But again: this is the desired behavior, and I do *not* want seven 
"Sent" folders in my special folders section.
Now I am just confused.  Apparently, somewhere in the last few months, 
my identities switch to saving all sent messages to a single mailbox.   
I changed my Horde identity to its sent-mail mailbox... and it 
correctly displays the name of the mailbox, not 'Sent'.

But this is also a first-level mailbox.  So it seems the issue is that 
all mailboxes underneath a special mailbox are labeled as a special 
mailbox.  Whatever I think about the sent-mailbox labeling, this is 
simply flat-out wrong and needs to be fixed.
11/01/2011 11:28:36 PM Jan Schneider Comment #14 Reply to this comment

[Show Quoted Text - 9 lines]
No thanks, that's awful usability.
Displaying the mailbox name in the special folders would be 
inconsistent UI (your sent-mail mailbox no longer displays as Sent 
simply because you've defined more than 1?  that is not intuitive).
It not less intuitive than having a bunch (7 in my case) of sent mail 
folders all with the same name.
I could see adding a "label" option to the sent-mail setting in an 
identity.  But by default, the mailboxes should be labeled Sent (it 
could be argued that their should be a single virtual Sent mailbox).
FWIW the sent mail folder that matches the default sent mail folder 
name (Sent) is still displayed translated correctly. And I never found 
it confusing that the non-default sent mail folders are displayed by 
their real names. I find this pretty logical actually.

[Show Quoted Text - 9 lines]
It may be caused by the fact that they are all second-level folders. 
But again: this is the desired behavior, and I do *not* want seven 
"Sent" folders in my special folders section.
11/01/2011 11:08:20 PM Michael Slusarz Comment #13 Reply to this comment
Now I am confused. There are individual sent-mail folders for each
identity and of course they are not all displayed as "Sent". :)
Why not?  They are all sent-mail folders.
Because how do you want to distinguish them if they all have the same label?
You have to know the mailbox name (which you can get by hovering over 
the Sent mail).

Displaying the mailbox name in the special folders would be 
inconsistent UI (your sent-mail mailbox no longer displays as Sent 
simply because you've defined more than 1?  that is not intuitive).

I could see adding a "label" option to the sent-mail setting in an 
identity.  But by default, the mailboxes should be labeled Sent (it 
could be argued that their should be a single virtual Sent mailbox).
Plus, this is the way they have always been displayed.
No, it never worked (for me) that way at any time. And I doubt for 
anyone else because different sent-mail folders would be unusable 
otherwise (see above).
This is the way it's always worked for me.  I have 2 different sent 
mailboxes (Sent and sent-mail) - they both have always displayed as 
Sent.
11/01/2011 10:56:12 PM Jan Schneider Comment #12 Reply to this comment
Now I am confused. There are individual sent-mail folders for each
identity and of course they are not all displayed as "Sent". :)
Why not?  They are all sent-mail folders.
Because how do you want to distinguish them if they all have the same label?
Plus, this is the way they have always been displayed.
No, it never worked (for me) that way at any time. And I doubt for 
anyone else because different sent-mail folders would be unusable 
otherwise (see above).
11/01/2011 10:13:04 PM Michael Slusarz Comment #11 Reply to this comment
Now I am confused. There are individual sent-mail folders for each 
identity and of course they are not all displayed as "Sent". :)
Why not?  They are all sent-mail folders.

Plus, this is the way they have always been displayed.
11/01/2011 10:02:27 PM Jan Schneider Comment #10 Reply to this comment
Much better, but the sent-mail folders are now sorted by their value,
not by their label.
Confused by this - how would we sort by label?  All sent-mail 
mailboxes should be displayed with "Sent" (or its translated 
equivalent) - sorting won't change that order.
Now I am confused. There are individual sent-mail folders for each 
identity and of course they are not all displayed as "Sent". :)
11/01/2011 06:35:02 PM Michael Slusarz Comment #9 Reply to this comment
Much better, but the sent-mail folders are now sorted by their 
value, not by their label.
Confused by this - how would we sort by label?  All sent-mail 
mailboxes should be displayed with "Sent" (or its translated 
equivalent) - sorting won't change that order.
11/01/2011 09:08:04 AM Jan Schneider Comment #8 Reply to this comment
Much better, but the sent-mail folders are now sorted by their value, 
not by their label.
11/01/2011 02:04:07 AM Michael Slusarz Assigned to Michael Slusarz
State ⇒ Resolved
 
11/01/2011 02:03:51 AM Git Commit Comment #7 Reply to this comment
Changes have been made in Git for this ticket:

Bug #10683: Logical sorting of special mailboxes

  3 files changed, 10 insertions(+), 15 deletions(-)
http://git.horde.org/horde-git/-/commit/78b9f80ca9f345774ef00b34dcf08f18fc3d23d3
10/28/2011 12:56:22 PM rsalmon (at) mbpgroup (dot) com Comment #6 Reply to this comment
I don't mind about the *exact* order, this was just the order that 
made most sense to me intuitively.
Although nobody ever complain over here (myself included), I agree with Jan.
Over here in French, special folders appear in this order :
Inbox
Drafts
Trash
Sent
Spam

10/27/2011 05:28:00 PM Jan Schneider Comment #5 Reply to this comment
I don't mind about the *exact* order, this was just the order that 
made most sense to me intuitively.
10/27/2011 05:15:04 PM Michael Slusarz Comment #4 Reply to this comment
Also, for the record, this has never previously worked the way you 
request since Spam used to appear before Sent (again, at least in 
English).
10/27/2011 05:06:10 PM Jan Schneider Comment #3 Reply to this comment

[Show Quoted Text - 11 lines]
Lucky you :)
10/27/2011 04:52:26 PM Michael Slusarz Comment #2 Reply to this comment
I think this worked already earlier. It makes more sense to me to 
sort special folders in the dynamic view not alphabetically (at 
least not outside "groups" of folders), but by importance, i.e.
Inbox
Drafts
Sent (all sent-mail folders alphabetically sorted)
Spam
Trash
For the record... this *is* the way it is sorted currently (at least 
in English :) ).
10/27/2011 04:31:54 PM Jan Schneider Comment #1
Priority ⇒ 1. Low
State ⇒ New
Patch ⇒ No
Milestone ⇒
Summary ⇒ Logical sorting of special folders
Type ⇒ Enhancement
Queue ⇒ IMP
Reply to this comment
I think this worked already earlier. It makes more sense to me to sort 
special folders in the dynamic view not alphabetically (at least not 
outside "groups" of folders), but by importance, i.e.
Inbox
Drafts
Sent (all sent-mail folders alphabetically sorted)
Spam
Trash

Saved Queries