Bug 9144 - Nokia770 skin selector 'broken' on IE 7.0 on Windows & Safari 3.1.2 on OS X
: Nokia770 skin selector 'broken' on IE 7.0 on Windows & Safari 3.1.2 on OS X
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Skins
: 7.1
: PC Windows Vista
: P1 critical (vote)
: 7.x
Assigned To: Michael Herger
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-08-14 05:10 UTC by Darren Lloyd
Modified: 2009-07-31 10:27 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 Darren Lloyd 2008-08-14 05:10:37 UTC
Here is a really frustrating problem I am experiencing today with the Nokia770 skin:

On a Windows Vista PC with IE 7.0, I loaded the SqueezeCenter Web interface with the default skin, then scrolled through the other skins on the interface settings page. I selected each skin, clicked on Save Settings, then refreshed the browser. This worked fine for each skin until I selected the Nokia770 one.

On IE 7.0 the page does not display correctly, the page is garbled and hence the page navigation is unusable. It took a while to figure out how to get to the interface settings page, I could select a different skin but there is a problem....

The Nokia770 skin disables the scroll bars within the browser and the page is too large to fit the browser window, so you cannot sroll down to the Save Settings button to change the skin.

On a Mac running OS X 10.5.4 and Safari 3.1.2, the Web interface loads and the Nokia770 skin displays correctly. However, when I navigate to the interface settings page, select a different skin and click on Save Settings, hey presto! The page refreshes with the Nokia770 skin each time.

So I'm stuck with this skin.

SqueezeCenter Version: 7.1 - 22170 on Debian Etch.
Comment 1 Michael Herger 2008-08-14 08:12:55 UTC
http://yourserver:9000/Default/ will bring you back to the default skin (replace Default with whatever skin you want).

Nokia770 will basically only work on the Nokia tablets or in Opera or FF. If you want to use it on any platform, use Touch instead: it's basically the same skin, to be used on IE.
Comment 2 Blackketter Dean 2008-08-14 21:19:55 UTC
Maybe we should make the 770 skin bail out early on non-Nokia browsers....
Comment 3 Michael Herger 2008-08-15 04:16:22 UTC
change 22634 - fix IE7 detection to use Touch instead of Nokia skin
Comment 4 Michael Herger 2008-08-15 04:36:29 UTC
Oh my... and the Safari issue actually is a complete failure of Safari with the non-Default skins. It doesn't save any setting because the submit button's value isn't returned to SC - which uses it to trigger saving the changes.

I've heard about Safari not saving changes before, but never realised it was with the non-default skins only.
Comment 5 Michael Herger 2008-08-15 04:56:31 UTC
change 22637 - add hidden "saveSettings" form element for Safari, as it doesn't send the submit button's value when submitted using JS.

These issues should be fixed in 7.2. Thanks!
Comment 6 Spies Steven 2008-10-10 13:44:56 UTC
Verified with SqueezeCenter Version: 7.2.1 - 23502
Comment 7 James Richardson 2008-12-15 12:34:53 UTC
This bug has been fixed in the 7.3.0 release version of SqueezeCenter!

Please download the new version from http://www.slimdevices.com/su_downloads.html if you haven't already.  

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.
Comment 8 Chris Owens 2009-07-31 10:27:14 UTC
Reduce number of active targets for SC