Bug 13272 - baby and jive skins have different specs for thumbnail size, 41x41 v. 39x39px
: baby and jive skins have different specs for thumbnail size, 41x41 v. 39x39px
Status: CLOSED FIXED
Product: SB Radio
Classification: Unclassified
Component: UI Skin
: Include FW version in comment
: PC Other
: P1 major (vote)
: 7.4.0
Assigned To: Ben Klaas
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-08-06 12:33 UTC by Ben Klaas
Modified: 2009-10-05 14:36 UTC (History)
5 users (show)

See Also:
Category: Bug


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ben Klaas 2009-08-06 12:33:58 UTC
I'm not sure what to do about this, but the fact that controller and baby are specified to two different icon sizes is fairly problematic from an implementation standpoint.

issues:
Efficiencies generated by using a base skin class for QVGA skins are lessened considerably
Increases size of firmware

and from Andy via campfire:
"Andy G.	and what sucks is SC has to pre-cache all possible artwork sizes.. on my todo list to go through and update that list
if there is any way to share sizes it would be useful"

Can we consider tweaking the design to have a single icon size for the two skins?
Comment 1 Ben Klaas 2009-08-06 12:34:26 UTC
this is a P1
Comment 2 Weldon Matt 2009-08-06 12:41:30 UTC
I can't totally speak for Noah but I believe the plan was indeed to have one single icons sizes for all UIs.

Methinks this is a "bug" in the spec/reference screens.
Comment 3 ndijulio 2009-08-10 16:52:20 UTC
Ben- for now let's set the Controller skin to 41x41px.  This will result in a 1px top/bottom padding on the menu sel. bar, which is acceptable... 

The other options would be more radical revisions for both skins.  If need be, we can look into this later.  We are ultimately constrained by the leftover content height after the titlebar and status bar are factored in.

Baby - 180px (content area), 45px menu bar ht. x4

Jive - 258px (content area), 43px menu bar ht. x6
Comment 4 ndijulio 2009-08-10 16:53:37 UTC
Ben- reassigning to you.  I will update the guidelines to 41px ASAP.
Comment 5 SVN Bot 2009-08-10 22:51:52 UTC
 == Auto-comment from SVN commit #6998 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=6998 ==

Fixed Bug: 13272
Thumb size on controller from 39->41
Comment 6 James Richardson 2009-10-05 14:36:10 UTC
This bug has been marked as fixed in the 7.4.0 release version of SqueezeBox Server!
    * SqueezeCenter: 28672
    * Squeezebox 2 and 3: 130
    * Transporter: 80
    * Receiver: 65
    * Boom: 50
    * Controller: 7790
    * Radio: 7790  

Please see the Release Notes for all the details: http://wiki.slimdevices.com/index.php/Release_Notes

If you haven't already, please download and install the new version from http://www.logitechsqueezebox.com/support/download-squeezebox-server.html

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.