Bug 1917 - Ability to customize the number/type of 'Now Playing' options to cycle through.
: Ability to customize the number/type of 'Now Playing' options to cycle through.
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Player UI
: unspecified
: All All
: P2 enhancement (vote)
: ---
Assigned To: Chris Owens
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2005-08-02 18:36 UTC by Daryle Tilroe
Modified: 2009-09-08 09:20 UTC (History)
3 users (show)

See Also:
Category: ---


Attachments
Patch to add hidden lock display mode client pref (1.46 KB, patch)
2005-09-24 03:59 UTC, Max Spicer
Details | Diff
choose now playing list options, radio for current (7.88 KB, patch)
2005-11-21 21:53 UTC, KDF
Details | Diff
skip properly through chosen options (8.66 KB, patch)
2005-11-22 21:09 UTC, KDF
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Daryle Tilroe 2005-08-02 18:36:20 UTC
Over time and with the SB2 there are now many 'Now Playing' options to chose and
to have to cycle through on the remote.  IMHO it would be nice to customize the
number and type available.  I would suggest moving to a dialog very much like
the one above it, 'Track Format', in 'Player Settings'; where you can choose one
or more of the options in whatever order you like.  I classifed this as "Player
UI" even though there would be a change in the web interface since the main
purpose is to improve the usability on the player/remote end.
Comment 1 Blackketter Dean 2005-08-02 21:06:55 UTC
good suggestion for a new setting.
Comment 2 KDF 2005-08-03 10:01:09 UTC
so, how about changing the current UI to use the radio button list like the one
for font settings?  Radio button selects the current one, and the user can then
select from the pulldown for any that s/he wants to be actively available?
Comment 3 KDF 2005-08-03 10:40:05 UTC
oh yeah...can assign to me if that is desireable.  won't be for 6.1.2, though. 
I'd also need a list of the defaults that we want available after install.
Comment 4 Daryle Tilroe 2005-08-03 19:40:19 UTC
> so, how about changing the current UI to use the radio button list
> like the one for font settings?  Radio button selects the current one,
> and the user can then select from the pulldown for any that s/he wants
> to be actively available?

As far as I can tell the 'Font' selection is the same radio button list as
the 'Track Format' so that is what I am talking about.  I actually don't know
how useful the radio button for current is since it is so easy to change but
it's probably fine.  Easy to cut 'n paste :).

> I'd also need a list of the defaults that we want available after install.

You might just want it default to all available or just one.  What also
might be useful is a "Lock" setting if it is simple to do while you are
at it.  In other words a way to temporarily lock it without deleting all
but one option.  Actually that might be a bit to complicated... so
probably just forget it.
Comment 5 KDF 2005-08-03 21:33:40 UTC
oh yeah, sorry. missed that part. whether the radio buttons are useful or not
will have to be seen.  However, keeping them means we reuse a template. 
creating new ones is a much larger task.
Comment 6 Max Spicer 2005-09-24 03:58:26 UTC
I find that I am forever changing the now playing display mode by accident then
having to cycle through them all just to get back to the one I had before.  I
realised that I never actually want to change the mode from the one that I use
all the time so have added a per-client pref to disable changing the display
mode at all.  This lets you choose your display mode then lock it.  It's a
temporary thing, so I haven't added any sort of interface to the pref apart from
editing the prefs file (although a "lock display mode" setting in the web ui
wouldn't be hard).  However, I believe it's a useful intermediary fix.  I've
attached the simple patch, if anyone's interested.
Comment 7 Max Spicer 2005-09-24 03:59:32 UTC
Created attachment 848 [details]
Patch to add hidden lock display mode client pref
Comment 8 KDF 2005-11-21 21:53:01 UTC
Created attachment 1044 [details]
choose now playing list options, radio for current

similar to titleformats, choose the current item, and the list of options from a global pulldown.  defaults for sb2 and other players.
Comment 9 KDF 2005-11-22 21:09:02 UTC
Created attachment 1045 [details]
skip properly through chosen options

the previous patch had the web ui, but the player toggle wasn't going through the selected list.  This now jumps through the currently selection options array, and properly deletes blank options from the web pref.
Comment 10 Max Spicer 2005-11-24 06:18:11 UTC
I think the latest patch is definitely the way this should be going.  I think the description text could be better though.  "The player can display information on the progress of a song as it's playing. Choose the information you wish to be displayed."  On an SB2, this does nothing to inform the user about the visualiser modes.  On all players, it doesn't tell you that you can cycle through these modes  by pressing Now Playing.  It's also unclear that you can disable some modes, although this may be a fault of the prefs system in general.

For starters, how about "The player can display information on the progress of a song as it's playing. You can select the information that you wish to be displayed by default, and can enable or disable the modes that you wish to be made available. You can cycle through these modes by pressing Now Playing on the remote repeatedly.".

Is it possible to change description text based on the current player?  I couldn't tell if $client was available initSetupConfig().
Comment 11 KDF 2006-01-06 11:17:38 UTC
initSetupConfig is a bad place to do that sort of thing, since it is global.  However, settings to have an preEval option, where you can look at the player model.  Look at player settings for audio, and you can see how this kind of thing is done for the digital settings.

Comment 12 KDF 2006-01-08 13:40:42 UTC
added patch 1045 to 6.5 at change 5571. additions and changes to it are welcome.  
Comment 13 Chris Owens 2006-08-16 13:54:47 UTC
So this has been done?  Shall I mark it fixed then?
Comment 14 KDF 2006-08-28 17:00:01 UTC
Functionally, it's done, but there is the issue of wording as expressed in comment 10.  This probably needs to be addressed by Dean
Comment 15 Chris Owens 2006-08-29 10:14:38 UTC
Dean, how do you feel about Max's proposed text change?
Comment 16 KDF 2007-10-30 15:22:13 UTC
looks like this is already done in change 10014 (and change 10015 ;)
please reopen if there are any issues.
Comment 17 Chris Owens 2008-03-07 09:03:41 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.