Closed
Bug 17476
Opened 25 years ago
Closed 23 years ago
Move/Copy menus should sort
Categories
(MailNews Core :: Backend, defect, P2)
MailNews Core
Backend
Tracking
(Not tracked)
VERIFIED
DUPLICATE
of bug 112301
mozilla0.9.4
People
(Reporter: fenella, Assigned: naving)
References
Details
(Keywords: polish, Whiteboard: [nsbeta1+ 2/13])
Attachments
(5 files)
11.12 KB,
patch
|
Details | Diff | Splinter Review | |
1.33 KB,
patch
|
Details | Diff | Splinter Review | |
12.35 KB,
patch
|
Details | Diff | Splinter Review | |
11.89 KB,
patch
|
Details | Diff | Splinter Review | |
11.98 KB,
patch
|
Details | Diff | Splinter Review |
Linux (1999-10-27-15 M11) Win32 (1999-10-27-09 M11) Summary: When doing a Move/Copy message, Drop-down list's sort order in server should match sort order of the folder pane. Steps: 1. Launch Messenger, select an account (qatest01) 2. From the Inbox, select a message 3. Select Message|Move Message (or Copy Message) to qatest01. Actual result: Notice that the sort order of the drop-down list from the server does not match with those in the folder pane. Expected result: I expect it to be sorted the same order like the folder pane, meaning that non-special folders are listed below special folders and that they are sorted like the Subject field in the thread pane. folder pane folder's order ( win32) --------------------------------- INBOX Drafts Sent Trash 123folder 234folder afolder attachfolder bugfolder imagefolder replyfolder testfolder xyz folder's order in the Move/Copy message drop-down list ---------------------------------------------------------- 123folder 234folder afolder bugfolder Drafts INBOX Replyfolder Sent testfolder Trash xyz
Updated•25 years ago
|
Assignee: phil → putterman
Target Milestone: M13
Comment 1•25 years ago
|
||
Reassign to putterman for M13, cc alecf
Updated•25 years ago
|
Summary: Drop-down list's sort order in server should match account folder list → Move/Copy menus should sort
Comment 2•25 years ago
|
||
changing summary. cc'ing rjc and hyatt. I think we had this discussion on a newsgroup once, but is there anyway to apply a sort to a menu?
Comment 3•25 years ago
|
||
don't we just change the arc that's being followed?
Comment 4•25 years ago
|
||
Alec, can you explain what you mean. The problem as I understood it, is that the RDF Sort Service only sorts trees. At least it did the last I was involved in the discussion. We probably either want it to sort content in general, or we'll need another way to do our sorting.
Comment 5•25 years ago
|
||
oh...I guess I was thinking that it just fiddled with the template, changing the arc that was followed, not the tree itself.
Comment 6•25 years ago
|
||
I'll try and code up what you need to make this work. Give me a few days to massage the XULSortService.
Comment 7•25 years ago
|
||
You can now sort any content by slapping a "sortResource" and "sortDirection" on the root node of the generated content. [Yes, if you need a second level sort, you can also use a "sortResource2" attribute]. Note: you can also specify a "template" attribute on the root node so that the template can be defined OUTSIDE of the generated content subtree. You might want to do this with <html:select> for example so that default select will work as expected, for example.
Updated•25 years ago
|
Status: NEW → ASSIGNED
Comment 8•25 years ago
|
||
Great. Thanks for getting this in.
Updated•25 years ago
|
Target Milestone: M13 → M15
Comment 9•25 years ago
|
||
Wouldn't hold beta for this, but I'd love to have it especially since some of the work is apparently done.
Comment 10•25 years ago
|
||
*** Bug 26558 has been marked as a duplicate of this bug. ***
Comment 11•25 years ago
|
||
*** Bug 26827 has been marked as a duplicate of this bug. ***
Comment 12•25 years ago
|
||
*** Bug 23854 has been marked as a duplicate of this bug. ***
Comment 13•24 years ago
|
||
Mass moving to M16 to get these off the M15 radar. Please let me know if this is really an M15 stopper.
Target Milestone: M15 → M16
Updated•24 years ago
|
Target Milestone: M16 → M17
Comment 14•24 years ago
|
||
Mail Review hopes this is post-beta2, easy, polish work. Marking M18.
Target Milestone: M17 → M18
Comment 15•24 years ago
|
||
marking nsbeta3 polish for the moment.
Comment 17•24 years ago
|
||
second pass: - per mail triage
Whiteboard: [nsbeta3+] → [nsbeta3-][cut 8/28]
Comment 19•24 years ago
|
||
*** Bug 57926 has been marked as a duplicate of this bug. ***
Comment 20•24 years ago
|
||
*** Bug 58484 has been marked as a duplicate of this bug. ***
Comment 21•24 years ago
|
||
*** Bug 59308 has been marked as a duplicate of this bug. ***
Comment 22•24 years ago
|
||
nominating for mail3. Based on all of the dups I get I think this is a highly requested bug to fix.
Keywords: mail3
Comment 23•24 years ago
|
||
reassigning to sspitzer
Assignee: putterman → sspitzer
Status: ASSIGNED → NEW
Comment 24•24 years ago
|
||
*** Bug 60484 has been marked as a duplicate of this bug. ***
Comment 25•24 years ago
|
||
Please remember that this is not only for foldernames, but also accounts.
Comment 26•24 years ago
|
||
this should include all of our menu lists including new folder dialog and copies and folders menu lists. marking nsbeta1+ and moving to mozilla0.9 milestone.
Comment 31•24 years ago
|
||
marking nsbeta1- and moving to future milestone.
Comment 32•24 years ago
|
||
*** Bug 69602 has been marked as a duplicate of this bug. ***
Comment 33•24 years ago
|
||
*** Bug 70703 has been marked as a duplicate of this bug. ***
Comment 34•23 years ago
|
||
*** Bug 74364 has been marked as a duplicate of this bug. ***
Comment 35•23 years ago
|
||
My suggestion is to use the sort order of the folders pane (standard folders first, *then* user created folders). The folders should be sorted alphabetically within their group (standard folders, user created folders), *regardles* of whether they contain subfolders or not. Target Milestone: Mozilla0.91? Status: assigned ???
Comment 36•23 years ago
|
||
*** Bug 77768 has been marked as a duplicate of this bug. ***
Comment 37•23 years ago
|
||
*** Bug 81101 has been marked as a duplicate of this bug. ***
Comment 38•23 years ago
|
||
*** Bug 81633 has been marked as a duplicate of this bug. ***
Comment 39•23 years ago
|
||
13 dups. Marking mostfreq and 4xp as this is a Netscape 4.* parity issue. I would mark it mostannoy as well if there were such a keyword :-(
Comment 40•23 years ago
|
||
Can really 4xp bugs be targetted to Future. I believe mozilla1.0 is more appropriate. Bug 68725 is on similar problem with Get Message dropdown sorting
Keywords: mozilla1.0
Comment 41•23 years ago
|
||
*** Bug 89969 has been marked as a duplicate of this bug. ***
Comment 42•23 years ago
|
||
*** Bug 90180 has been marked as a duplicate of this bug. ***
Comment 43•23 years ago
|
||
Comment 44•23 years ago
|
||
*** Bug 91944 has been marked as a duplicate of this bug. ***
Comment 45•23 years ago
|
||
Can someone update this bug to add the patch, review and/or ui keywords?
Updated•23 years ago
|
Comment 46•23 years ago
|
||
I'll do the review today.
Comment 47•23 years ago
|
||
r=chuang. I applied the patch and tested on Move/Copy msg menu, File button, New Folder, Message Filter and search dialog. All folder lists appeared the same order as in folder pane.
Comment 48•23 years ago
|
||
sr=mscott. Although this might break when the outliner branch lands for the folder pane.
Comment 49•23 years ago
|
||
this looks good. (and I think I've got dups of this in my bug list.) this shouldn't cause a conflicts with the folder outliner branch. (see bug #73865) setting target to 0.9.4
Target Milestone: Future → mozilla0.9.4
Comment 50•23 years ago
|
||
there might be two other places to check. the picker in messengercompose.xul and the picker in SearchDialog.xul
Comment 51•23 years ago
|
||
Picker in Search dialog is fine. I'll attach the patch for the compose window menu Options|File Sent Message.
Comment 52•23 years ago
|
||
Comment 53•23 years ago
|
||
The file button in Search dialog is not covered. I'll attach a final patch including the file button in search dialog.
Comment 54•23 years ago
|
||
Comment 55•23 years ago
|
||
why the change from <menu> to <menulist> in the patch?
Comment 56•23 years ago
|
||
I'm not sure about that one, neil@parkwaycc.co.uk did the fix for mailWindowOverlay.xul and msgFolderPickerOverlay.xul. I can find out what happen if using menu.
Comment 57•23 years ago
|
||
Comment 58•23 years ago
|
||
Using <menu> works fine. The new patch doesn't use <menulist>.
Comment 59•23 years ago
|
||
Forgot to mention, those are the pickers in Account Setting|Copies and Folders.
Comment 60•23 years ago
|
||
r/sr=sspitzer on http://bugzilla.mozilla.org/showattachment.cgi?attach_id=44474
Comment 61•23 years ago
|
||
*** Bug 94595 has been marked as a duplicate of this bug. ***
Comment 62•23 years ago
|
||
*** Bug 94071 has been marked as a duplicate of this bug. ***
Comment 63•23 years ago
|
||
So when is this fix going in? I only ask because 93061 and 93011 have gone in. P.S. menu to menulist is covered by another bug which should go in anyway.
Comment 64•23 years ago
|
||
Candice, can you try to get this patch checked into the tree soon? If you can't get to it then we should push this off to .9.5 per the mailnews triage meeting today. Feel free to do so if you aren't going to have time to check this in.
Comment 65•23 years ago
|
||
Comment 66•23 years ago
|
||
Fix checked in. The patch just posted is the same patch combined first three patches using <menulist>.
Reporter | ||
Comment 67•23 years ago
|
||
I am reopening this bug because the sort order in the folder pane is Upper case letters sort followed by small letters sort INBOX Trash Business Confirm Drafts Mail-to-read Sent Template bugstatus fen_to_read fenella imapfolder mail issue mail _news sarah skin But in the MOVE/COPY sort, it is sorting by alphabetical orders with small letters preceding upper case characters (except for INBOX and Trash). Eg INBOX Trash bugstatus Business Confirm Drafts fen_to_read fenella imapfolder mail issue mtg Mail-to-read mail_news sarah sentskin Templates
Comment 68•23 years ago
|
||
that's another bug. http://bugzilla.mozilla.org/show_bug.cgi?id=95358 (fix in hand for that one.)
Status: REOPENED → RESOLVED
Closed: 23 years ago → 23 years ago
Resolution: --- → FIXED
Comment 70•23 years ago
|
||
I am testing this with build 2001090908. The sorting looks OK, but there is a problem: If while focused on a message header I click on the right button I do see an alphabetically sorted list of folders. If then I move the mouse down the list , if I position the mouse over a folder with subfolders, sometime the subfolder list is shown , some other times it is not shown. The behaviour is not consistent.
Comment 71•23 years ago
|
||
This worked for me as of 0.9.6 but in 0.9.7 it is broken again. Specifically I'm trying to do a move to on linux and the sort order appears to be random. In 0.9.6 it was alphabetical.
Comment 72•23 years ago
|
||
It's *not* sorted alphabetically on win98, build 2002-01-03 also. This needs to be reopened. I also really like the idea that it should be sorted as in the folder pane (special folders first). I would further suggest to put a horizontal line between the special and the user-created folders - I think that would help visually demonstrate why there is no straight alphabetical sorting.
Comment 73•23 years ago
|
||
Also now broken again on 0.9.7 Solaris 8 SPARC.
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
Comment 74•23 years ago
|
||
naving, this wasn't broken by the sortKey code, was it?
Assignee: chuang → naving
Status: REOPENED → NEW
Updated•23 years ago
|
QA Contact: fenella → olgam
Assignee | ||
Comment 75•23 years ago
|
||
This bug is really old. I didn't know such a bug existed. However I have a fix to make it sort just as they appear in folder pane. *** This bug has been marked as a duplicate of 112301 ***
Status: NEW → RESOLVED
Closed: 23 years ago → 23 years ago
Resolution: --- → DUPLICATE
Updated•20 years ago
|
Product: MailNews → Core
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•