Alphabetical Order for Profiles

RESOLVED WONTFIX

Status

()

--
enhancement
RESOLVED WONTFIX
17 years ago
6 years ago

People

(Reporter: raccettura, Assigned: raccettura)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 obsolete attachments)

(Assignee)

Description

17 years ago
Could profiles maybe be listed in alphabetical order rather than the order in
which they were created?  That would be great for when you have a lot of
profiles... and considering how Netscape Comm. and most other programs list
information (including the component portion of buzilla) alphabetical order just
makes more sence and feels better.

Comment 1

17 years ago
This is probably a legitimate RFE, and there's no obvious duplicate in this component 
already.

In truth, in the Profile Manager dialog, the profile names are topped by a column header, 
"Available Profiles," which appears similar to that used in list views under Classic in Mail/
News, for example. Such column headers usually are clickable to sort by their type. So, if 
we don't provide this "clickable" sort functionality, the default sort order of these profiles 
should be by that described in the header, and in this case that is profile name.

So, to state it succinctly, the profiles should be listed in alphabetical order, and the list 
should resort when a change is made to the list.

Marking enhancement, though, as the current functionality does work.
Severity: trivial → enhancement
Status: UNCONFIRMED → NEW
Ever confirmed: true
(Assignee)

Comment 2

17 years ago
Is there a timeframe for completion of this?  Maybe 9.5?  I don't think it is
that much work is it?
Status: NEW → ASSIGNED
Target Milestone: --- → Future
(Assignee)

Comment 3

16 years ago
Status?
(Assignee)

Comment 4

16 years ago
Anyone?  I guess no chance on 1.3a... but perhaps for 1.3b?

Would be nice.


With lots of profiles, would look neater if it were all in alphabetical order.
(Assignee)

Comment 5

15 years ago
Can we hvae this added to the list of bugs for Firebird 1.0?

I think it would be a good way to polish it up (as small as it is).  With large
lists, it can be a bit annoying.
(Assignee)

Comment 6

15 years ago
Taking, I have a patch
Assignee: bugs → robert
Status: ASSIGNED → NEW
(Assignee)

Comment 7

15 years ago
Created attachment 138833 [details] [diff] [review]
Patch v1

My first attempt at writing a patch.

Please don't hurt me.  I'm a nice guy.	Really!  :-)
(Assignee)

Comment 8

15 years ago
Created attachment 138836 [details] [diff] [review]
Patch v1.0001

Deleted a return for no good reason, contributors.
(Assignee)

Updated

15 years ago
Attachment #138833 - Attachment is obsolete: true
(Assignee)

Comment 9

15 years ago
Created attachment 138838 [details] [diff] [review]
Patch v1.0002

I suck, I'm used to tabbing.
Attachment #138836 - Attachment is obsolete: true
(Assignee)

Comment 10

15 years ago
Comment on attachment 138838 [details] [diff] [review]
Patch v1.0002

Suggesions for sr?
Attachment #138838 - Flags: review?(neil.parkwaycc.co.uk)
(Assignee)

Updated

15 years ago
Status: NEW → ASSIGNED
(Assignee)

Updated

15 years ago
Attachment #138838 - Flags: review?(neil.parkwaycc.co.uk) → review?(dean_tessman)

Comment 11

15 years ago
Comment on attachment 138838 [details] [diff] [review]
Patch v1.0002

> function loadElements()
> {
>   try {
>+    var ProfileArray = new Array()
>     var profileRoot = Registry.getKey(Registry.Common, "Profiles");
>     var regEnum = Registry.enumerateSubtrees( profileRoot );
> 
>@@ -170,15 +172,18 @@
>         break;
> 
>       var migrated = Registry.getString( node.key, "migrated" );
>-
>-      AddItem( "profiles", new Profile( node.name, migrated ) );
>+      ProfileArray[ProfileArray.length] = new Array(node.name,migrated);

Why do you create two Arrays?  Also, you need a space between the comma and
"migrated" to be consistent with the surrounding code.
Attachment #138838 - Flags: review?(dean_tessman) → review-
(Assignee)

Comment 12

15 years ago
Comment on attachment 138838 [details] [diff] [review]
Patch v1.0002

This bug still exists ;-)  me kind of forgot. 

Going to update sometime... I could do better than this.
Attachment #138838 - Attachment is obsolete: true
Product: Browser → Seamonkey

Comment 13

13 years ago
See bug 193755.  
Robert,
Are you still working on this ?

Updated

9 years ago
Component: Startup & Profiles → Startup and Profile System
Product: SeaMonkey → Toolkit
QA Contact: agracebush → startup
Target Milestone: Future → ---

Updated

6 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.