Bug 13659 - Cosmetic alignment - song progress text + bar is too low/too close to bottom edge
: Cosmetic alignment - song progress text + bar is too low/too close to bottom ...
Status: RESOLVED WONTFIX
Product: SB Controller
Classification: Unclassified
Component: Now Playing
: unspecified
: PC Windows Vista
: P5 minor (vote)
: 7.5.x
Assigned To: Ben Klaas
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-08-25 10:35 UTC by ndijulio
Modified: 2011-01-14 09:13 UTC (History)
2 users (show)

See Also:
Category: Bug


Attachments
Ref artwork - alignment for progress text/bar (19.85 KB, image/png)
2009-08-25 10:35 UTC, ndijulio
Details
Ref artwork - NP MAX (55.35 KB, image/png)
2009-09-11 17:48 UTC, ndijulio
Details
Ref image- taken from squeezeplay of current skin (51.58 KB, image/png)
2009-09-14 12:07 UTC, ndijulio
Details
Ref image- composite of design and current skin (60.77 KB, image/png)
2009-09-14 12:17 UTC, ndijulio
Details

Note You need to log in before you can comment on or make changes to this bug.
Description ndijulio 2009-08-25 10:35:10 UTC
Created attachment 5686 [details]
Ref artwork - alignment for progress text/bar

Please see guidelines for proper placement.  Looking at it by eye, the entire
set needs to be pushed up 1-2px. Text should nominally be centered in negative
space btwn artist - album text and bottom edge of titlebar.

Attaching guidelines for reference.
Comment 1 Ben Klaas 2009-08-26 07:50:22 UTC
this is an administrative shuffle on priority fields to help make better judgment on the top end of the priority list. P4->P5, P3->P4, and P2->P3.
Comment 2 ndijulio 2009-09-11 17:48:48 UTC
Created attachment 5835 [details]
Ref artwork - NP MAX

The text spacing and alignment is really off for the Portrait screen.

Ben-Can you attached a screen shot of the current build that includes a song progress bar?
Comment 3 ndijulio 2009-09-11 17:53:05 UTC
We should also consider revising the padding on the left and right margins from 10px to 8px for the Portrait skin.
Comment 4 ndijulio 2009-09-14 12:07:09 UTC
Created attachment 5840 [details]
Ref image- taken from squeezeplay of current skin
Comment 5 ndijulio 2009-09-14 12:17:14 UTC
Created attachment 5841 [details]
Ref image- composite of design and current skin

Red lines = correct placement to designed layout

Yellow lines = actual placement in skin code

Lines are set to baseline of text strings.  FS font (letters a, c, o, etc.) extended below the baseline.
Comment 6 Pat Ransil 2009-10-23 05:09:57 UTC
Administrative move of 7.5 bugs. All P2, P3, P4 being downgraded one level. Will then split P1s.
Comment 7 Chris Owens 2010-03-08 11:24:09 UTC
Moving lower-priority bugs to next target
Comment 8 Ben Klaas 2011-01-14 09:13:55 UTC
I've spent a lot of time trying to make this right, but there are skinning challenges that made it basically unfixable, at least at the Lua level. It's as good as it's going to get.