Bug 1996 - Enable MusicMagic seeding directly from the "now playing" list.
: Enable MusicMagic seeding directly from the "now playing" list.
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Plugin
: unspecified
: PC Windows XP
: P2 enhancement (vote)
: Future
Assigned To: Blackketter Dean
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2005-08-19 09:01 UTC by yann oehl
Modified: 2008-12-15 13:07 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments
web interface, mixer links in current playlist (2.28 KB, patch)
2005-08-19 13:42 UTC, KDF
Details | Diff
trackinfo patch (1.14 KB, patch)
2005-08-19 13:52 UTC, KDF
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description yann oehl 2005-08-19 09:01:51 UTC
Allow songs and albums to trigger mixes directly from the "Now Playing" list. 
Currently, an MMM user must hunt through the track's info/album link/ and 
eventually find the song listed in order to seed. Cutting out all of these 
additional steps would be a huge MMM usability improvement.

See the thead at:
http://forums.slimdevices.com/showthread.php?p=48827#post48827
Comment 1 KDF 2005-08-19 12:07:21 UTC
seems possible to do in web interface.

cannot be done from player UI, as play_hold function is already taken for "save
playlist" from the now playing section.  possible option might be to enable
mixes from trackInfo (which means one extra push right from a track)
Comment 2 yann oehl 2005-08-19 12:37:34 UTC
Enabling mixes from trackInfo would definetely work for me! 

TrackInfo is a click away from both albums and songs, right? That would be 
=lovely=. The important bit for me (more than just the # of "clicks away"), is 
that the "now playing" song (and the other songs in the current playlist) are 
essentially "remembered" for me to seed from... which is =much more= 
intuitive/friendly than making the user remember which track he/she was just 
listening to, then dig down into the albums browse view to find it several 
clicks away.

y.
Comment 3 KDF 2005-08-19 13:42:43 UTC
Created attachment 745 [details]
web interface, mixer links in current playlist

fishbone, en, and default skins.  other skins are easy.  requires some tweaks
to the mixer plugin html templates that I've not included (to specify target
frames)

of course, other option (if clutter in teh now playing isn't desired) is to
simply use the album link to get the song list on the left. Not hard to
remember with the track still sitting on the right.

player UI doesn't look easy.  two ways to go that I can see.  TITLE: can
provide a link to mix on the track, etc.  This needs a mapping in IR.  Other
option is to simply scroll to album, press right to get track list.  Support
would have to be added to enter the album at the currently selected track from
the playlist. Might take a bit of doing, but at least its relying on the
existing browsedb modes.
Comment 4 KDF 2005-08-19 13:52:23 UTC
Created attachment 746 [details]
trackinfo patch

this patch sends trackinfo through to album/artist/genre with the track
pre-selected.  You can then press-and-hold play from there.  Given the
simplicity of this one, my personal slant is to go wtih this patch, and do
nothing with the web interface.  The links through to the browser are provided
and the browser (web and playerUI) can do all of the mixer functions.

If there is still a difficulty in navigating this way for web UI, please feel
free to explain the specifics of the issue
Comment 5 yann oehl 2005-08-19 14:23:44 UTC
My goodness, That was fast!
I have no usability issues with the Web UI as is... only with the Player UI, 
which your patch goes a long way towards resolving. Thanks KDF!

Q: Do I apply the patch directly myself, or will it be included an upcoming 
nightly?
Comment 6 KDF 2005-08-19 14:28:21 UTC
If you are using slimserver.pl, then you can apply using "patch -p0 <
patchfile.diff", or manually.

If you use slim.exe then you have to wait until it is built into a nightly. 
Having a patch here is only the first step into a nightly.  It will be up to
Dean as to when or which UI changes should be added.
Comment 7 KDF 2005-08-21 21:17:44 UTC
Trackinfo change committed to 6.2 at change 4026.  If that is all that is wanted
for this bug, let me know and it can be closed off.
Comment 8 yann oehl 2005-08-22 09:11:21 UTC
You can close it as far as I'm concerned. Great work KDF! 
Comment 9 KDF 2005-08-22 09:17:52 UTC
Done deal, then :)
Comment 10 James Richardson 2008-12-15 13:07:00 UTC
This bug appears to have been fixed in the latest release!

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.

Make sure to include the version number of the software you are seeing the error with.