Bug 11430 - cosmetic: keyboard tweaks
: cosmetic: keyboard tweaks
Status: RESOLVED WONTFIX
Product: SB Touch
Classification: Unclassified
Component: UI Input
: unspecified
: PC Other
: P3 normal (vote)
: 7.5.0
Assigned To: Ben Klaas
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-03-20 14:46 UTC by Ben Klaas
Modified: 2010-02-18 13:01 UTC (History)
3 users (show)

See Also:
Category: ---


Attachments
Keyboard skinning for rounded corners (12.67 KB, patch)
2009-03-20 14:46 UTC, Ben Klaas
Details | Diff
current IP keyboard (78.11 KB, image/png)
2009-03-23 21:56 UTC, Ben Klaas
Details
current lowercase qwerty keyboard (67.03 KB, image/png)
2009-03-23 21:57 UTC, Ben Klaas
Details
current numeric keyboard (67.58 KB, image/png)
2009-03-23 21:57 UTC, Ben Klaas
Details
keyboard with light colored wallpaper. highlights edge issues (20.35 KB, image/png)
2009-03-23 21:59 UTC, Ben Klaas
Details
Top edge test artwork (transparent box) (225 bytes, image/png)
2009-03-24 10:29 UTC, ndijulio
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ben Klaas 2009-03-20 14:46:27 UTC
Created attachment 4955 [details]
Keyboard skinning for rounded corners

this ain't workin' yet, so attaching patch so as not to lose work in progress
Comment 1 Ben Klaas 2009-03-23 21:56:43 UTC
Created attachment 4961 [details]
current IP keyboard
Comment 2 Ben Klaas 2009-03-23 21:57:10 UTC
Created attachment 4962 [details]
current lowercase qwerty keyboard
Comment 3 Ben Klaas 2009-03-23 21:57:34 UTC
Created attachment 4963 [details]
current numeric keyboard
Comment 4 Ben Klaas 2009-03-23 21:59:40 UTC
Created attachment 4964 [details]
keyboard with light colored wallpaper. highlights edge issues

There are still some rendering problems at the edges, where it appears darker. Noah asserts this may be due to two assets with transparency layering on top of each other. That doesn't appear how the skin is configured, however.
Comment 5 Ben Klaas 2009-03-23 22:01:57 UTC
r4916 takes us most of the way there for the keyboard, including pressed assets.

still need:
confirm with Noah that pressed behavior is as intended with each key
figure out problem with "dark" edge assets
extend titlebar to go beyond and below textinput (will require some fakery to do this, as the textinput will not actually be part of the titlebar)
Comment 6 ndijulio 2009-03-24 10:29:22 UTC
Created attachment 4965 [details]
Top edge test artwork (transparent box)
Comment 7 Ben Klaas 2009-03-25 09:17:56 UTC
Changing summary to use this bug as the catch-all for remaining issues with keyboard window layout.

Remaining issues:

1. pressed tile assets for keys along top and bottom rows incorrect (except for top left and bottom left corners). Problem resides with these two assets:

Buttons/keybrd_n_button_press_t.png
Buttons/keybrd_s_button_press_b.png

working this issue with Noah

2. vertical spacing between keyboard <--> textinput and keyboard <--> bottom of window needs to be the same

3. need to create visual effect that titlebar + textinput are all part of the titlebar.
Comment 8 ndijulio 2009-03-25 11:35:27 UTC
Additional cosmetic issues to resolve:

1. Cursor height should be 26px high and NOT extend into text entry window border

2. The top buttons (back, ?), text entry field, and keyboard should share the same right/left edge alignment.  For example the left vertical edge of the back
button, left edge.  Current build r4940 text entry window and keyboard aligned, but not aligned to titlebar buttons (left/right). 

3. Keyboard key font sizes are not correct
main keys:
QWERTY/1234.../@#$ - FS Bold 24px

bottom row keys:
(any text grouping whether numeric or alpha)
123-&/ABC, done, search, .com - FS Bold 20px

(any individual characters/symbols)
@, ., _, etc. - FS Bold 24px

4. "Done" button should have added emphasis, propose quick fix of inc. contrast RGB 231,231,231.

5. Delete button appears misaligned top/bottom.  Could move up ~1px.

6. Shift icon should move down ~1px to appear centered top/bottom

7. There is no way to get to addition symbols.  Need to add jump button to lower, left corner (above ABC).

Enhancements:

8. Cursor should blink on/off every sec.  Currently artwork is static.

9. Press and hold on delete button should clear all.

10. Back button in titlebar currently goes back one character at a time.  Back button should always be back to previous menu.
Comment 9 Ben Klaas 2009-03-25 13:53:04 UTC
r4948 has smaller font sizes for the "special" keys, space bar, done, and keyboard switcher buttons.

I also changed the font color for the done button to 231, 231, 231, while the others are 220, 220, 220. To my eyes, it's imperceptible, so there may be additional work there.
Comment 10 Ben Klaas 2009-03-30 14:50:57 UTC
issues #1 and #3 from comment#7 are fixed, r5023
Comment 11 Ben Klaas 2009-04-08 16:01:27 UTC
Looking this over, I think we're good for MP.

Remaining cosmetic tweaks to Keyboard:

1. pressed assets are still not right, but they are definitely good enough for MP
2. done button should have a spinny when input is submitted but hasn't pushed to a new window (separate bug open for that)
3. cosmetic tweaks to textinput cursor
Comment 12 Ben Klaas 2009-07-16 09:09:12 UTC
changing several bugs at once-- target milestone on these do not apply to the CXR/CAT milestone
Comment 13 Ben Klaas 2009-09-30 09:53:23 UTC
There may not be anything to do on this bug any more, but keeping open for audit and retargetting for 7.5
Comment 14 ndijulio 2009-09-30 10:41:15 UTC
Ben, Does this include everything from comment #8?

11. I am still seeing funky alignment along the top row and corners when pressed.  

12. There is also a drop shadow or nested text for "done."  The net effect is that it appears blurry.  Need to remove the drop shadow.

13. The titlebar bkgrd and bkgrd asset for the text input window are different alphas (darker).  If possible, should reuse asset from titlebar to prevent future discrepancies.
Comment 15 Ben Klaas 2009-09-30 12:43:55 UTC
Okay, work to do then. Sorry, it's been a while since I've considered this...

estimating 4 hours because the keyboard is esp. tricky for layout issues
Comment 16 SVN Bot 2009-10-29 08:56:57 UTC
 == Auto-comment from SVN commit #7972 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=7972 ==

Bug: 11430
Description: unblur the done button on the keyboard
Comment 17 Ben Klaas 2009-10-29 10:09:37 UTC
What's left in this bug is not a P1. The most glaring needs-fixing is some 1px aberrations on pressed state for some of the keyboard keys. But, seeing as these aberrations are typically under a finger...moving to P3
Comment 18 Ben Klaas 2010-02-18 13:01:18 UTC
This is going to be a WONTFIX