Bug 514 - Difficulty editing entered text
: Difficulty editing entered text
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Player UI
: 5.x or older
: All All
: P2 normal (vote)
: ---
Assigned To: KDF
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2004-08-25 12:19 UTC by Chris Harris
Modified: 2009-09-08 09:14 UTC (History)
0 users

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Harris 2004-08-25 12:19:57 UTC
If you are doing a search for, say, ABC and enter
ACD, scrolling backwards to change the C requires you to place the cursor under
the C and then wait a couple of seconds. If you try instantly when the cursor is
under C, you add a third letter back. If you move back so the cursor is under
the A then it changes the A rather than the C.

ACD <left>
AC (C underlined) <2> (for a B) gives
ACB

but

ACD <left>
AC (C underlined) - wait for 2 seconds - <2> (for a B) gives
AB

It's a pain having a delay to edit the text. And inconsistent as there is
seemingly no delay with the first letter.
Comment 1 KDF 2004-08-30 14:26:25 UTC
Chris, if you use the press-and-hold-PLAY to save a playlist, does that text
entry work better for you?  Search uses its own entry routines, but SavePlaylist
uses the INPUT.Text mode.  Search should eventually make use of this as well, so
feedback on that would be useful too.
Comment 2 Chris Harris 2004-08-31 12:12:46 UTC
Umm, better in that the behaviour I mentioned doesn't occur - but the playlist
entry system (which I've never used before) seems to have a whole bunch of its
own quirks: Pressing on the right arrow changes the text to capitals .. plus
arrow signs keep appearing in the middle of the words.
Comment 3 KDF 2004-08-31 12:39:56 UTC
wow, I've not seen that one before.  Pressing right should jump to the screen
confirming the save.  Do you have the latest nightly, and are using
Press-AND-Hold PLAY from teh now playing screen?  Make sure you do NOT have the
SavePlaylist plugin as that is obsoleted by this feature.  I've converted the
Search functions to use the INPUT.Text mode so tomorrow's nightly will be good
for testing the search functions for any issues that you have.
Comment 4 Chris Harris 2004-08-31 12:47:46 UTC
Yep, this is the latest nightly.

Try entering the band abba

2 <right> 2 2 <right> 2 2 <right> 2 <right> 

gives 

acBC->
Comment 5 KDF 2004-08-31 13:38:23 UTC
I end up with abba->, using the current CVS with my patch.
so, I'm not sure what is happening there. I had an old SavePlaylistPlus plugin
that used to change case on ADD. Please try out tomorrow's nightly and we'll see
what happens from there.
 
Comment 6 KDF 2004-08-31 19:03:54 UTC
looks like this is actually a problem in Common::numberLetter.  There is a 1
second timer that resets to the first letter for a given button.  Even pressing
right, the routine still compares to the last letter for the 'displayTextTimeout'.

hope to have a fix shortly.
Comment 7 KDF 2004-08-31 19:14:43 UTC
added $client->lastLetterTime(0); to the 'nextChar' function in INPUT.Text. 
This should result in the following:
2 <right> 2 2 = ab (instead of the ac that has been reported)

try this out in the Sept 1 nightly and let me know if there are any other issues.
Comment 8 Chris Harris 2004-09-01 23:02:46 UTC
Hi,

This is much better than it was, I've only found one small problem so far.

When you're moving right everything seems fine, but the timer for input isn't
reset when you move left.

If I'm trying for abba (again!), but I make a mistake forwards, when I go back
to edit it continues from the incorrect letter instead of resetting.

2 <right> 2 2 <right> 2 2 2 <right>: abc (oops, meant abb - so ...)
<left> 2 2: I'd expect abb, but instead get abA

You have to be reasonably quick to see the problem I'll admit - but all the text
messaging we do in the UK makes me pretty speedy with the remote.
Comment 9 KDF 2004-09-01 23:16:46 UTC
thanks. fixed that for sept 2. Please mark this as fixed if it works properly
for you.
Comment 10 Chris Harris 2004-09-03 00:17:04 UTC
That seems fine now, thanks.
Comment 11 Chris Owens 2006-06-16 14:40:30 UTC
There are 536 bugs in the database with targets of '---' that were fixed prior to new year 2006.  I am setting them to targets of 6.2.1 to keep them from showing up in my queries.
Comment 12 Chris Owens 2008-12-18 11:53:09 UTC
Routine bug db maintenance; removing old versions which cause confusion.  I apologize for the inconvenience.