Bug 2046 - Random Mix plugin UI issues.
: Random Mix plugin UI issues.
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Plugin
: 6.2.0
: Macintosh All
: P2 normal (vote)
: ---
Assigned To: Dan Sully
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2005-08-31 09:35 UTC by Blackketter Dean
Modified: 2008-09-15 14:36 UTC (History)
2 users (show)

See Also:
Category: ---


Attachments
per request (975 bytes, patch)
2005-09-01 12:18 UTC, KDF
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Blackketter Dean 2005-08-31 09:35:36 UTC
Couple of things need to be cleaned up in the Random Mix plugin UI.

Player side:  Pressing right to enter Random Mix plugin should take you to a screen that says:

----------------
Random Mix (Press PLAY to start)
Random Songs
----------------
then there should be two more choices to scroll through:

Random Artist
Random Albums


Server side:  Random Mix link on home page should take you to Random Mix page that has this:

----------------------------------------------------------------

Random Mix

Choose a random mix below:

Random songs from your whole library                               Play
Random artist from your whole library                                Play
Random albums from your whole library                             Play


------------------------------------
Also, the current playlist name (as displayed in the top right section of the playlist scrolling area on 
the web UI needs to show "Random Mix" as the playlist.

-----------------------------------
Finally the setup needs to be reworded (and the option to not add new songs should be removed, it's 
always on).

-----------------------------------
Random Mix

The Random Mix plugin let's SlimServer create a random mix of songs from your entire library.  When 
creating a random mix of songs, you can specify how many tracks should stay on your Now Playing 
playlist.

Number of songs in a random mix:  [ 10 ]

(Change)

-----------------------------------------------
Comment 1 Dan Sully 2005-08-31 17:51:59 UTC
Fixed in subversion change 4141.
Comment 2 Blackketter Dean 2005-09-01 08:36:02 UTC
Much better.

Couple of open issues:

The "Random Mix" playlist name on the right side has (Modified) after it, which probably shouldn't be 
there.

The links on the Random Mix page don't reload the right side, so you don't see the effect until the page 
is refreshed, possibly 30 seconds later.

One more request:  make the "Random Mix" playlist name be "Random Song Mix", "Random Artist Mix" 
or "Random Album Mix" as appropriate.
Comment 3 KDF 2005-09-01 12:18:22 UTC
Created attachment 788 [details]
per request

changes existing strings to 'random xxx mix', so this shows as playlist title,
as well as the player UI items in the list. I felt it was less redundancy this
way, but it could also be done by cat-ing a MIX string to the old setup.

mark the playlist as un-modified after the addtracks.

not sure how we want to refresh the status_header.  Maybe a simply onload=
javascript in the randommix_list templates. I haven't done those yet, but could
if that sounds right
Comment 4 KDF 2005-09-05 23:50:13 UTC
patch 788 committed to trunk at change 4177 since many of the other patches
today are based on having these UI tweaks in place.
Comment 5 KDF 2005-09-06 17:45:53 UTC
change 4183 should clear up the issues. reopen if there are any more.