Bug 2102 - when changing skins, buttons do not reflect current playing state
: when changing skins, buttons do not reflect current playing state
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Skins
: 6.2.0
: PC Windows (legacy)
: P2 normal (vote)
: ---
Assigned To: Blackketter Dean
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2005-09-08 10:24 UTC by Joel
Modified: 2008-09-15 14:37 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Joel 2005-09-08 10:24:35 UTC
With slimserver: 6.2b1 - 4150 - Linux
Browsing with: Opera 7.54u2
Normal Skin: Handheld

Whenever I change to a different skin I find that the up/down/delete "Buttons" 
are incorrect in the playlist. 
I must change the playing state(stop, next, prev) to get the correct buttons 
displayed. (also the overall HTML formating can be different)

Example:
Skin: Handheld
Play, then stop a playlist.
Change to WebPad Skin.
**Button Images are not displayed.(html OK? but image not served) **

Press Play, now I get plain characters buttons "^", "v", etc.
Press Stop and change back to Handheld Skin.
Go to playlist display.
** Buttons are still characters ^,v and Title, artist, album is different**

Press Play and display the playlist.
Now all is good.
Comment 1 Michael Herger 2005-09-08 13:04:27 UTC
When changing skins there's a link at the top of the page which says "Click here 
to refresh and see the new skin." - you should try this one. After a skin change 
slimserver won't reload all the html. You have to do this manually, either by 
clicking that link or by hitting F5/reload in your browser. 
Comment 2 Joel 2005-09-08 17:22:43 UTC
I Know.
Been There.
Done That.
The T-Shirt Doesn't Hide the Bug ;-)

Only thing that will cause the HTML and/or the images to be served corectly is a 
change of State (Next/Prev/etc)

as a side note: the playlist display is not auto refreshed for the Handheld skin
(The Now Playing is, but the current song is not indicated on the playlist page 
for this skin, so no need to update)
Comment 3 Michael Herger 2005-09-08 21:02:23 UTC
Can you confirm the problem with another browser? I'm using Opera as my standard 
browser on all platforms I have (Windows 2000/XP, Mandrake, Ubuntu, Zaurus :-)). 
But except for the Zaurus I'm on 8.x. I have seen too aggressive caching with 
opera before, but can't say whether this is the problem here.

And you're correct: the playlist wouldn't update automatically. Do you often 
leave it on that page? I don't like it, as with the one panel design there's 
only very limited information available.
Comment 4 Joel 2005-09-08 21:50:38 UTC
Well I tried with Opera 8.02
It has the exact same behaviour as 7.54
Note: I Always leave Java disabled.

Cacheing can not be the reason for generating incorrect html in the Webpad skin

The a snipet of the html from right pane after switching to the webpad skin
--------------------------------------------------------------------------
	<!-- Element status_list.html -->
	<tr >
		<td ><a name=0>
			<a href="status.html?p0=playlist&p1=jump&p2=0&player=192.168.1.4">1.</
a>&nbsp;
			<a href="songinfo.html?item=4403&player=192.168.1.4">Sorceress (Return to 
Forever)</a>
		</td>

		<td width="13" align="center">
			<a href="status.html?p0=playlist&p1=jump&p2=0&player=192.168.1.4">
			<img src="html/images/b_play.gif" width="13" height="13" border="0" 
align="center" alt="Play"></a>
		</td>
		<td>&nbsp;</td>

		<td width="13" align="center">
			<a href="status.html?p0=playlist&p1=move&p2=0&p3=%2D1&player=192.168.1.
4&start=0">
			<img src="html/images/b_up.gif" border="0" width="13" height="13" 
align="center" alt="Move up"></a>
		</td>
		<td>&nbsp;</td>

		<td width="13" align="center">
			<a href="status.html?p0=playlist&p1=move&p2=0&p3=%2B1&player=192.168.1.
4&start=0">
			<img src="html/images/b_down.gif" border="0" width="13" height="13" 
align="center" alt="Move down"></a>
		</td>
		<td>&nbsp;</td>

		<td width="13" align="center">
			<a href="status.html?p0=playlist&p1=delete&p2=0&player=192.168.1.4&start=0">
			<img src="html/images/b_delete.gif" border="0" width="13" height="13" 
align="center" alt="Remove"></a>
		</td>
	</tr>
<!-- /Element status_list.html -->
--------------------------------------------------------------------------
Note: That I have two panes but the images are not present (server does not send 
images as this should be the following)

Webpad skin Page after start/stop same "status line" as above
--------------------------------------------------------------------------
	<tr>
   <td class="odd" bgcolor="#d0d0d0" >
       	  			 <a href="status.html?p0=playlist&p1=jump&p2=0&player=192.168.1.
4">Sorceress (Return to Forever)</a>
 		 		
		<br />&nbsp;&nbsp;&nbsp;&nbsp;	
<i><a href="browseid3.html?genre=*&artist=&album=The%20Best%20Of%20Return%20to%
20Forever&player=192.168.1.4" target="browser">The Best Of Return to Forever</
a></i><a href="status.html?p0=playlist&amp;p1=deletealbum&amp;p2=&amp;p3=&amp;
p4=The%20Best%20Of%20Return%20to%20Forever&amp;player=192.168.1.4&amp;
start=0">[x]</a> 
   </td>
   <td class="odd" bgcolor="#d0d0d0" >
      <table>
         <tr>
            <td>
                <a href="status.html?p0=playlist&amp;p1=move&amp;p2=0&amp;p3=%
2D1&amp;player=192.168.1.4&amp;start=0" target="status">&nbsp;^&nbsp;</a>
            </td>
            <td>
                <a href="status.html?p0=playlist&amp;p1=move&amp;p2=0&amp;p3=%
2B1&amp;player=192.168.1.4&amp;start=0" target="status">&nbsp;v&nbsp;</a>
            </td>
            <td>
               <a href="status.html?p0=playlist&amp;p1=delete&amp;p2=0&amp;
player=192.168.1.4&amp;start=0" target="status">&nbsp;X&nbsp;</a>
            </td>
         </tr>
      </table>
   </td>
</tr>
--------------------------------------------------------------------------

I've noticed that most skins NEVER pass w3c validation(http://validator.w3.org/)
Not that that has anything to do with this problem.


Comment 5 Joel 2005-09-08 21:56:03 UTC
Grit my teeth and dust off IE.

BARF!! 
Same thing with IE 6.02800.1106
Comment 6 Michael Herger 2005-09-08 23:06:57 UTC
Ok, I was finally able to reproduce. And in fact it's neither an Opera nor a 
Handheld issue, but rather server side caching. Fixed in change 4217. (if 
somebody could close the ticket...)
Comment 7 KDF 2005-09-08 23:10:38 UTC
done
Comment 8 Chris Owens 2006-06-16 14:41:27 UTC
There are 536 bugs in the database with targets of '---' that were fixed prior to new year 2006.  I am setting them to targets of 6.2.1 to keep them from showing up in my queries.