Bug 4089 - background update for current song
: background update for current song
Status: RESOLVED INVALID
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 7.0
: PC Windows (legacy)
: P2 enhancement (vote)
: 7.x
Assigned To: Chris Owens
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2006-09-11 16:36 UTC by KDF
Modified: 2009-07-31 10:13 UTC (History)
0 users

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description KDF 2006-09-11 16:36:44 UTC
Aimple background requests can be used to get updated playlist information.  Redrawing the entire playlist may get complex, but only updating the css for the current track is rather simple and would avoid the need to a page refresh after every song.  This means we can drop the $client->currentPlaylistChangeTime(time()); calls from Player::Source and only do playlist renders when the actual list changes. 

I currently have this in trunk for the fishbone skin, along with some extra complexity to bypass the standard render after each song.  This enhancement would simplify that code a bit and the ajax can trigger a refresh of the page when there is a new list to render.

propose this as something for post 6.5 of course.
Comment 1 KDF 2007-11-07 20:40:30 UTC
punting this until the skin work can settle down a bit.  might have to move to 'future' unless it actually becomes irrelevant.
Comment 2 KDF 2008-03-10 22:37:19 UTC
pushing off to .1 as it's not a bug fix
Comment 3 KDF 2008-05-13 15:12:55 UTC
this is more or less invalid with the skinning having changed so much.  new skins are better based on the new js libraries rather than hacking away at the old html.
Comment 4 Chris Owens 2009-07-31 10:13:56 UTC
Reduce number of active targets for SC