Bug 6335 - Touch Skin corrupts playlist in IE7
: Touch Skin corrupts playlist in IE7
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Skins
: 7.0
: PC Windows XP
: P2 minor (vote)
: ---
Assigned To: Unassigned bug - please assign me!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2007-12-12 10:20 UTC by Steve Sheafor
Modified: 2008-12-18 11:12 UTC (History)
3 users (show)

See Also:
Category: ---


Attachments
Playlist display after first set of deletions (139.10 KB, image/jpeg)
2007-12-12 10:20 UTC, Steve Sheafor
Details
Now Playing display while another track is playing (98.67 KB, image/jpeg)
2007-12-12 10:21 UTC, Steve Sheafor
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Steve Sheafor 2007-12-12 10:20:02 UTC
I am using the Touch skin (because of bug 6333).  I added an Album to the playlist, and the display seemed fine.  However, the "Now Playing" track always seemed to show as the first one, even though the correct (different) track was actually playing.  If I refreshed the browser, the "Now Playing" track was correct, but after a few seconds the display went back to showing the first track as "Now Playing".

I then added more Albums, and when I looked at the Playlist (since it isn't displayed when adding Albums) the new tracks were gone, but all of the tracks from the original Album were still displayed.  When I refreshed the browser, all the tracks were shown, but every couple of seconds the screen changes and every other track is gone (except for the original Album, which is unchanged) until all of the new tracks have disappeared.  The Now Playing track always goes back to the first one on the first change.

The total number of tracks is more than the window limit, so the "1 2 Next" display is shown, and the total number of tracks is correct.  When the first tracks disappear, the total number of tracks changes to the number of tracks in the first Album (even though some of the other tracks are still visible), but the "1 2 Next" remains.

I tried to capture the screen with Quick Screen Capture.  If I refresh the browser and start a capture, the display remains correct (i.e. tracks aren't lost).  When I complete the capture, the screen then changes and several iterations of the "every other track" loss have occurred.  I was able to capture the screen after the first change, which will be attached.  I added all of "Surrealistic Pillow", and you can see that every other track is gone.  the total number of tracks is reported as 11, which is the number in the first Album.

If I select the "Now Playing" display, it always says "Now stopped on 1 of 11", as shown in the second attached screen capture.  Another track from another Album was playing at this time.  This display is not temporarily fixed by refreshing the browser.

I am running SC on a 64-bit XP Pro machine, and viewing it with IE7 on a client machine.  I can also look at SC in IE6 running on the same server machine as SC.  In this case, I do NOT see any of the above issues.
Comment 1 Steve Sheafor 2007-12-12 10:20:37 UTC
Created attachment 2508 [details]
Playlist display after first set of deletions
Comment 2 Steve Sheafor 2007-12-12 10:21:13 UTC
Created attachment 2509 [details]
Now Playing display while another track is playing
Comment 3 KDF 2007-12-12 12:01:53 UTC
I'm also not seeing this, but cannot test on IE7.  Please update to latest nightly builds.
cc'ing Ross (my abusee for the day) to have a look with IE7 or pass it on to someone who can :)
Comment 4 Ross Levine 2007-12-12 14:18:46 UTC
I was able to reproduce this. Refreshing also fixes it, briefly, then back to wrong data. KDF do you want some logs?
Comment 5 KDF 2007-12-12 15:20:45 UTC
no logs can help, I'm afraid.  It's IE7 doing something stupid with the same data that any other browser handles just fine.  Any error icon showing up in the lower left of the ie window?
Comment 6 Steve Sheafor 2007-12-12 16:06:33 UTC
When this happens, there are no messages in the lower left corner of IE7.
Comment 7 KDF 2007-12-12 23:15:45 UTC
change 15244 should help, at least partly.  please try it out and give us a list of what else is missing.
Comment 8 Ross Levine 2007-12-13 14:12:49 UTC
Tested for 30 minutes with 15249, I'm not seeing this problem any more. I'd say the fix helps, but not a clear fix? We should probably wait to hear Steve's feedback. 
Comment 9 KDF 2007-12-13 15:46:13 UTC
The fix may be causing the Nokia770 UI to fail, so it's not a proper fix yet.  Feedback from Steve would help to at least make sure I understood what the issue was. Unfortunately, I'm on a trip to the UK tomrorrow.  Access to 'net is questionnable for me for the next two weeks. I'll hope for feedback, but I'm going to have to revert the change tomorrow before I go. I feel that the nokia case takes priority over IE, as the nokia skin is a purpose-built ui. Touch is based on the nokia skin and only intended to have "better" IE support.

cc'ing ben as he's likely to be keen on this progress too as it links with the Nokia akin.
Comment 10 KDF 2007-12-13 16:35:35 UTC
change 15279 tries to do this in a safer way, detecting IE and using old tried and true methods if it's not IE (so should be safe for nokia770).  Will wait and see what the word is on the forum.  I'll revert all of this tomorrow if the nokia is still showing problems.
Comment 11 Steve Sheafor 2007-12-13 22:24:59 UTC
In 15249, everything seems to be working fine.  I will try the latest version tomorrow and verify.
Comment 12 Steve Sheafor 2007-12-13 22:28:58 UTC
In 15154 (before the latest fix for this), I was seeing a serious problem with Nokia770, but I have not rported it yet because I wanted to make sure I had the symptoms clear.  I will look at this when I gt the latest veriosn tomorrow.
Comment 13 Steve Sheafor 2007-12-14 07:13:35 UTC
In 15288, the problem with Touch is still gone.  Nokia770 also works fine.
Comment 14 KDF 2007-12-14 08:29:59 UTC
thanks.  marking as fixed. reopen if there is something I missed.
Comment 15 Chris Owens 2008-03-07 09:05:17 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.