Bug 3790 - Sort players under settings on home page
: Sort players under settings on home page
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 6.5b1
: PC Windows XP
: P2 enhancement (vote)
: ---
Assigned To: Unassigned bug - please assign me!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2006-07-20 19:47 UTC by Jim McAtee
Modified: 2008-12-18 11:12 UTC (History)
2 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jim McAtee 2006-07-20 19:47:07 UTC
For consistency with the ordering in the drop down box of the status pane.  No doubt users with more than a couple of players would welcome this change instead of having them in listed in random order.
Comment 1 Chris Owens 2006-07-21 09:25:50 UTC
Setting the target to 'Future' since we have a strong list of 6.5 features already.  We'll probably have a meeting in the next couple weeks to decide what features are going in to the next build after 6.5.
Comment 2 Michael Herger 2007-11-19 03:12:29 UTC
The new Default skin doesn't have that menu any more. Can we close this request?
Comment 3 Jim McAtee 2007-11-19 10:37:05 UTC
It's still valid for the other skins.
Comment 4 KDF 2007-11-19 12:32:53 UTC
how about this:

Index: C:/slim/server/HTML/EN/home.html
===================================================================
--- C:/slim/server/HTML/EN/home.html	(revision 14848)
+++ C:/slim/server/HTML/EN/home.html	(working copy)
@@ -97,7 +97,7 @@
 		<li><a href="[% webroot %]settings/server/basic.html?player=[% playerURI %]">[% "SERVER_SETTINGS" | string %]</a></li>
 	[%- END %]
 
-	[%- USE Clients; players = Clients.get("name"); FOREACH client = players; %]
+	[%- USE Clients; players = Clients.get("name"); FOREACH client = players.reverse; %]
 		<li><a href="[% webroot %]settings/player/basic.html?player=[% playerURI %]&amp;playerid=[% client.id | uri %]">[% "PLAYER_SETTINGS" | string %][% IF client.name %] [% "FOR" | string %] [% client.name %][% END %]</a></li>
 	[%- END %]
 
Comment 5 KDF 2007-11-19 22:52:58 UTC
merged at change 14862.  From what I can see, this fixes things.  Please reopen if there is a problem.
Comment 6 Chris Owens 2008-03-07 09:05:11 UTC
This bug is being closed since it was resolved for a version which is now released!  Please download the new version of SqueezeCenter (formerly SlimServer) at http://www.slimdevices.com/su_downloads.html

If you are still seeing this bug, please re-open it and we will consider it for a future release.