94787 - Alphabetical Order for Profiles

Status: RESOLVED WONTFIX

(Toolkit :: Startup and Profile System, enhancement)

Description

[Robert Accettura [:raccettura]]

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

[Greg K.]

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.

Comment 2

[Robert Accettura [:raccettura]]

Is there a timeframe for completion of this?  Maybe 9.5?  I don't think it is
that much work is it?

Comment 3

[Robert Accettura [:raccettura]]

Status?

Comment 4

[Robert Accettura [:raccettura]]

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.

Comment 5

[Robert Accettura [:raccettura]]

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.

Comment 6

[Robert Accettura [:raccettura]]

Taking, I have a patch

Comment 7

[Robert Accettura [:raccettura]]

Attachment: Patch v1

My first attempt at writing a patch.

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

Comment 8

[Robert Accettura [:raccettura]]

Attachment: Patch v1.0001

Deleted a return for no good reason, contributors.

Comment 9

[Robert Accettura [:raccettura]]

Attachment: Patch v1.0002

I suck, I'm used to tabbing.

Comment 10

[Robert Accettura [:raccettura]]

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

Suggesions for sr?

Comment 11

[:deanis74]

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.

Comment 12

[Robert Accettura [:raccettura]]

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.

Comment 13

[David E. Ross]

See bug 193755.  

Comment 14

[Serge Gautherie (:sgautherie)]

Robert,
Are you still working on this ?