Bug 13625 - Too much padding in title on NP, users get to see less as a result
: Too much padding in title on NP, users get to see less as a result
Status: CLOSED FIXED
Product: SqueezePlay
Classification: Unclassified
Component: UI Skin
: unspecified
: PC Other
: P1 normal (vote)
: 7.4.0
Assigned To: Ben Klaas
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-08-24 08:50 UTC by Wadzinski Tom
Modified: 2009-10-05 14:33 UTC (History)
3 users (show)

See Also:
Category: ---


Attachments
SS (26.91 KB, image/png)
2009-08-24 08:50 UTC, Wadzinski Tom
Details
Current Guideline for NP (13.34 KB, image/png)
2009-08-28 17:19 UTC, ndijulio
Details
Ref artwork - composite of SS + Guideline (52.46 KB, image/png)
2009-08-28 17:27 UTC, ndijulio
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Wadzinski Tom 2009-08-24 08:50:29 UTC
Created attachment 5673 [details]
SS

The padding for the NP track and title in at least the landscape skin seem
overly large.


See as a comparison the attachment which shows the Mac red close icon closer to
the left than our text. 

Having so much padding gives less on-screen text, on an already small screen.
Comment 1 ndijulio 2009-08-24 11:09:15 UTC
*Baby*
10px padding - 245px for track title - 8px padding - 47px for X of Y (fixed width) - 10px padding
-reducing the 10px will impact the L/R viewing angle

*Controller*
8px padding - 169px for track title - 8px padding - 47px for X of Y (fixed width) - 8px padding
-note that elapsed time would appear below text and not replace X of Y for this skin
-if possible for internet radio stations we would not truncate the the top track title text (this can be an enhancement for a later release).


By using a 47px fixed width we get *999/1... *and *1:59:59* using a FS Medium *14px* font size.  Any further info. would need to scale down.  In general:

   FS Medium fonts = 12px ht is the MIN
   FS Bold fonts = 10px ht is the MIN

For small playlist, etc. can we switch displaying the X of Y from 1/2 to *01/02*?  This will also help even out the padding for MAX track titles and MIN X of Y.  I think that this combined with the fixed width will help significantly.  For now I do not want to increase the X of Y font beyond 14px MAX.  Any larger and the template hierarchy gets thrown off.

Please let me know if you need anything further...
Comment 2 ndijulio 2009-08-28 16:42:31 UTC
Tom- sorry I misread your commit.  You are proposing reducing the padding away from the edge of the screen no?

Couple of comments:

1. This is very doable and would indeed add a 2-3 characters of viewable info. to the titlebar.

2. The trade-off is the viewing angle left-right away from center would be decreased.

The reason for the extra padding is due to the depth the screen is in-set inside the plastic lens.
Comment 3 ndijulio 2009-08-28 17:19:19 UTC
Created attachment 5747 [details]
Current Guideline for NP
Comment 4 ndijulio 2009-08-28 17:27:23 UTC
Created attachment 5748 [details]
Ref artwork - composite of SS + Guideline

By overlaying the SS and the Guideline, the SS (code) is off by a good 4-6px.  The correct padding is 10px from the edge of the screen.  Need to reduce the padding in the code.  This would be go for the Play icon is the status bar as well.  The left edge should be in-line with the text.
Comment 5 ndijulio 2009-08-28 17:29:17 UTC
Ben - reassigning to you.  Not sure who's this is to fix...
Comment 6 SVN Bot 2009-08-28 19:12:41 UTC
 == Auto-comment from SVN commit #7316 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7316 ==

Fixed Bug: 13625
Description: fix left/right padding to 10px instead of 14px (whoops, my bad)
Comment 7 Wadzinski Tom 2009-08-29 09:14:24 UTC
Thanks guys.
Comment 8 James Richardson 2009-10-05 14:33:58 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.