Bug 4507 - Touch skin needs to work in IE
: Touch skin needs to work in IE
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Skins
: 6.5.1
: PC Other
: P2 normal (vote)
: ---
Assigned To: Ben Klaas
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2006-11-14 08:45 UTC by Ben Klaas
Modified: 2008-12-18 11:12 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 Ben Klaas 2006-11-14 08:45:45 UTC
Touch skin looks like garbage in IE. I was able to avoid supporting that hunk-of-junk browser in Nokia770, but it's not really justified in Touch. Hopefully this is mostly CSS to fix and it'll work.

Hope to have this functional by 7.0. I think this could back-merge to 6.5.1 as well.
Comment 1 Ben Klaas 2006-11-27 14:45:56 UTC
Touch, which inherits all but css and a couple other pages from Nokia770, doesn't render properly in IE.

I have fixed this by hacking the css and some supporting pages a great deal, and I'm satisfied now that it works. I will be checking this into 6.5 and trunk branches. I had to totally remove position:fixed attributes from all div elements and now emulate position: fixed with this hack:

http://www.divinentd.com/experiments/emulating-position-fixed.html

seems to work just fine in FF and IE both, Linux and Windows both. There are css positioning problems with Opera, but I'm pre-emptively calling that a "wontfix", as Nokia770 skin should work fine for Opera users, and I'm still using position: fixed there.

I feel these changes kind of suck, since it's a total hack, but it's a necessary evil to support the multitudes of IE users out there. Particularly PDA people, who are often restricted to IE on their device, and obviously could benefit from a skin like Touch. 

I'd really like feedback on this after people have tried out the fixes. If it doesn't work, please update this bug.
Comment 2 Ben Klaas 2006-11-27 14:50:46 UTC
Fixed in 6.5 and Trunk branches

bklaas@mediumspicy /usr/local/slimserver/HTML $ svn commit -m "Bug #4507: IE support through massive css hacking" Touch/slimserver.css Nokia770/pagefooter.html Nokia770/browsedb.html Nokia770/searchinput.html Nokia770/menu.html
Sending        Nokia770/browsedb.html
Sending        Nokia770/menu.html
Sending        Nokia770/pagefooter.html
Sending        Nokia770/searchinput.html
Sending        Touch/slimserver.css
Transmitting file data .....
Committed revision 10779.

bklaas@mediumspicy /usr/local/slim.trunk/HTML $ svn commit -m "Bug #4507: IE support through massive css hacking" Touch/slimserver.css Nokia770/pagefooter.html Nokia770/browsedb.html Nokia770/searchinput.html Nokia770/menu.html Touch/searchinput.html Touch/browsedb.html
Sending        Nokia770/browsedb.html
Sending        Nokia770/menu.html
Sending        Nokia770/pagefooter.html
Sending        Touch/browsedb.html
Sending        Touch/searchinput.html
Sending        Touch/slimserver.css
Transmitting file data ......
Committed revision 10780.
Comment 3 Chris Owens 2008-03-07 09:03:37 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.