Bug 3782 - Artwork issues with Handheld skin
: Artwork issues with Handheld skin
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 6.5b1
: PC Windows XP
: P2 minor (vote)
: ---
Assigned To: Chris Owens
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2006-07-19 17:02 UTC by Philip Meyer
Modified: 2008-09-15 14:39 UTC (History)
3 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Philip Meyer 2006-07-19 17:02:58 UTC
When viewing a single album, I see a thumbnail (100x100) followed by the full-size
version of the artwork.

Pocket PC - Dell Axim X50.  Windows Mobile 2003SE.
I have also tried on FireFox 2.0 beta 1 (WinXP Pro), and get the same problem.

The view mode toggle button has no effect.  This works in Firefox though.

In Firefox, when viewing thumbnails of albums, there is no spacing between the thumbnails.
I can't say what this looks like on the PDA, due to the view mode button not working ;)

Also, artwork that doesn't scale to 100x100 (eg. 100x88px) should perhaps be centred?
Comment 1 KDF 2006-07-19 17:38:04 UTC
find me an example in handheld where spacing ever existed between artwork.  this is for a handheld, after all.
Comment 2 KDF 2006-07-19 18:10:23 UTC
does the Dell Axim support javascript? if not, toggle won't work.
any javascript console messages if it does?

rest should be done in change 8524
Comment 3 KDF 2006-07-19 18:39:10 UTC
change 8530 replicates a recent change from EN that might help fix the toggle problem, depending on the actual cause.
please update and see if that helps.
Comment 4 Michael Herger 2006-07-20 01:24:09 UTC
Javascript was broken: as Handheld doesn't use frames, there's not parent object. That's the ugly side of having to port from skin to skin...

Change 8532 should fix this - sorry, too late for today's build.
Comment 5 KDF 2006-07-20 12:50:13 UTC
change 8539 merges the scripts.  should work without having to keep porting in the future.
Comment 6 Michael Herger 2006-07-20 13:08:31 UTC
Thanks. Working perfectly fine for me (and my Zaurus' Opera).
Comment 7 Jim McAtee 2006-07-21 18:52:49 UTC
I think change 8539 has caused bug 3799 in the Default skin.
Comment 8 KDF 2006-07-21 19:27:03 UTC
taken care of, thanks.