Bug 12835 - Text Entry - vertical bar assets not tiling correctly
: Text Entry - vertical bar assets not tiling correctly
Status: RESOLVED FIXED
Product: SB Radio
Classification: Unclassified
Component: Settings
: Include FW version in comment
: All Windows Vista
: P2 normal (vote)
: MP
Assigned To: ndijulio
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-07-13 16:17 UTC by ndijulio
Modified: 2009-09-08 09:13 UTC (History)
0 users

See Also:
Category: ---


Attachments
comparison with current textinput ref screen (20.44 KB, image/png)
2009-07-20 09:49 UTC, Ben Klaas
Details
comparison of charHeight of 25px v. 20px (16.32 KB, image/png)
2009-07-20 09:53 UTC, Ben Klaas
Details

Note You need to log in before you can comment on or make changes to this bug.
Description ndijulio 2009-07-13 16:17:04 UTC
The vertical bar appears to have two overlapping assets: 

1. Tall background asset
2. center middle asset 

Number 2 is overlapping the background creating a harsh box around the selected character.  The tiling should be seamless.  It is most noticeable on the left an right edges along the shadow.
Comment 1 Ben Klaas 2009-07-20 09:49:59 UTC
Created attachment 5489 [details]
comparison with current textinput ref screen

On the left is the current textinput ref screen, on the right is what I've done to get as close as possible to the ref screen.

What I've done:

- made the textinput horizontal bar the correct height
- tweaked the y offset of the selected char to center vertically inside the bar as
 good as currently possible.

What I haven't done:
- the spacing of the chars in the vertical bar in the ref screen doesn't work as you scroll down further in the alphabet (e.g., the g ends up bleeding into the horizontal bar when it's just above the selected character). By having that a bit larger (25px), it works better throughout.
Comment 2 Ben Klaas 2009-07-20 09:53:32 UTC
Created attachment 5490 [details]
comparison of charHeight of 25px v. 20px

25px is on the left, 20px is on the right.

25px is how I currently have it (see first attachment), 20px will achieve tighter spacing at the expense of things looking broken when chars like g show up in the vertical wheel above the selected char.

Noah, opinion?
Comment 3 Blackketter Dean 2009-07-20 10:00:36 UTC
Letters should never be jammed together like the example on the right.  If you make the text bigger the baseline spacing should also increase.

The one on the left looks ok to me.
Comment 4 ndijulio 2009-07-20 11:15:53 UTC
Agreed. Characters should never overlap.  

The ref. screen uses a 22px leading (spacing btwn letters) for 24px text ht.  Can you please post an example using the spec. spacing?
Comment 5 Ben Klaas 2009-07-20 14:23:07 UTC
r6687 changes the leading to 24px