Bug 11431 - Problems with entering encryption with the remote.
: Problems with entering encryption with the remote.
Status: RESOLVED FIXED
Product: SB Touch
Classification: Unclassified
Component: Setup
: unspecified
: PC Windows XP
: -- normal (vote)
: MP
Assigned To: ndijulio
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-03-20 15:33 UTC by Anoop Mehta
Modified: 2009-09-08 09:31 UTC (History)
5 users (show)

See Also:
Category: ---


Attachments
Cursor color comparison (teal vs. mono selection) (65.49 KB, image/png)
2009-03-24 16:52 UTC, ndijulio
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Anoop Mehta 2009-03-20 15:33:06 UTC
When trying to enter encryption key with the Fab4 remote there a few issues. 

1. You cannot tell what letter/number you are choosing. 
2. If you try to use the logical 'cell phone text messaging' way with the key pad you get very strange behaviors.
Comment 1 James Richardson 2009-03-20 15:35:47 UTC
Which firmware version are you testing?
Comment 2 Anoop Mehta 2009-03-20 15:51:34 UTC
Sorry r4864
Comment 3 Anoop Mehta 2009-03-20 15:53:44 UTC
Sorry i did a reset to get back to setup ...then when I reconnected it upgraded to r4864 and same problem with that one..is there a certain firmware that I should test? r4875?
Comment 4 Anoop Mehta 2009-03-20 16:04:35 UTC
Same result with r4875
Comment 5 Blackketter Dean 2009-03-20 16:24:54 UTC
Remote only operation needs to work for MP. I think that the only issue here is that the selection color went from green (ok) to almost black (bad).  probably just needs to be a lighter grey until we have the full remote skin working.
Comment 6 Dan Evans 2009-03-24 12:16:14 UTC
There are 2 issues, as I see it:

1. The highlight color is black, meaning you can't see the char.

2. The order that characters are selected in is using the Jive mechanic.  That is, when I press UP it selects '9'.  When I press UP the second time for the second character it selects '8'.  This makes sense if you have a wheel for input, but makes no sense for Up/Down arrows.  Pressing UP should provide consistent input.
Comment 7 Blackketter Dean 2009-03-24 14:37:49 UTC
Noah:  this file:

/Users/dean/squeezeplay/assets/Fab4Skin/images/Text_Entry/Keyboard_Touch/tch_cursor.png

needs to be grey and not black so that the cursor is distinguished from the text being drawn on top of it.

This needs to be fixed for MP, so please prioritize appropriately
Comment 8 ndijulio 2009-03-24 16:33:03 UTC
Is the text rendered on top of the cursor (i.e. cursor becomes highlighted bkground) on remote?
Comment 9 Blackketter Dean 2009-03-24 16:38:30 UTC
yes, the text is rendered on top of the cursor.
Comment 10 ndijulio 2009-03-24 16:52:57 UTC
Created attachment 4967 [details]
Cursor color comparison (teal vs. mono selection)

We are going to fight a contrast battle the more black or white we add to the gray cursor.  In this instance the addition of color would be appropriate.  It accomplishes a higher level of readability/contrast.  By extension, all items that are selected or "highlighted" would use a teal accent color...
Comment 11 Anoop Mehta 2009-03-24 16:55:46 UTC
IMO the teal looks great!
Comment 12 ndijulio 2009-03-24 18:09:19 UTC
Asset updated to gray version in snv. 3/24/09 - 6:04pm as seen in attached comparison.  

*The cursor color treatment should be flagged as an area to review at a later date.*
Comment 13 Jim McAtee 2009-03-24 18:51:20 UTC
What's the purpose of the block cursor?  Without a way to toggle between insert/overwrite modes it doesn't make any sense.  IMO, you should always be in insert mode using the bar cursor and then use the back(space) key to erase the character to the left of the cursor and insert a new characters at the cursor.

I'm talking about exactly the way an HTML form's text input box works in most (all?) browsers.  There is no overwrite mode.

If the intention is to use the block cursor and overwrite mode when someone touches within an existing string, that doesn't work because there's no way to insert a forgotten character without some way to toggle out of overwrite mode.
Comment 14 Blackketter Dean 2009-03-24 21:26:36 UTC
Jim: if you use the remote to enter text, then there is an overwrite mode.  Try this by using your IR remote and scroll up/down when on the text entry screen.

Also, if we decide to add drag-to-select text (like on a computer gui), then the cursor becomes a highlight.
Comment 15 Richard Titmuss 2009-03-26 05:03:21 UTC
Tom, once Noah's finished this is yours to fix.
Comment 16 Blackketter Dean 2009-03-31 17:58:53 UTC
I think this is ready for you, right tom?
Comment 17 Wadzinski Tom 2009-04-01 14:39:54 UTC
(In reply to comment #16)
> I think this is ready for you, right tom?

It's unclear to me what the issue is for this bug. Can you clarify?
Comment 18 Blackketter Dean 2009-04-01 14:46:48 UTC
Based on my conversation with Richard today, I think he's working on this.
Comment 19 Jim McAtee 2009-04-01 15:54:34 UTC
(In reply to comment #14)
> Jim: if you use the remote to enter text, then there is an overwrite mode.  Try
> this by using your IR remote and scroll up/down when on the text entry screen.

I'm not sure I see this.  As with the SB2/3 remote, if you use up/down when the cursor is over a character then it cycles through characters.

I just did a factory reset and ran through the entire setup using the IR.

The biggest issue I found is that when entering text the "auto-move to next character" behavior is too fast - much faster than with previous IR interfaces - making it difficult if you pause for a split second while going through the characters on a remote button.  It also makes it difficult to use the up/down cycling when entering a special character on the '1' key and you overshoot the one you're looking for.
Comment 20 Blackketter Dean 2009-04-02 16:23:54 UTC
I think this is working great, but we still need to revert to the teal cursor.

Noah: can you check in the old asset?
Comment 21 ndijulio 2009-04-07 19:19:02 UTC
This should be resolved.  Feel free to reopen after review of current build.