Hi list,
I've fixed a small editing bug: For menu string edit items, when in insert mode, its impossible to delete the last character of the string. Steps to reproduce:
- Go into some edit field: [A]BCDEF - Switch to insert mode : []ABCDEF - Move into the string : ABC[]DEF - Press yellow key : ABC[]EF - Press yellow key : ABC[]F - Press yellow key : ABC[]F - nothing happens.
This doesn't happen if you move the cursor to the end once, as this will insert a blind space: ABCDEF[]_
The attached patch fixes this in one possible way: It deletes the last char, but keeps the cursor position by inserting a whitespace:
ABC[]F -> ABC[]_
Without the whitespace, the cursor would have to move in an ugly way:
ABC[]F -> AB[]C -> A[]B -> []A -> []_
The implementation requires another quirk: To do the delete-towards-left at the end of the string, just as the overwrite mode does, the delete key behaves differently if it deletes a final whitespace: Instead of deleting the whitespace, the second last char is deleted:
ABC[]F -> ABC[]_ -> AB[]_ -> A[]_ -> []_
Cheers,
Udo
--- vdr-1.4.1-2-orig/menuitems.c 2006-07-24 00:24:25.176897944 +0200 +++ vdr-1.4.1/menuitems.c 2006-07-24 01:19:52.383818056 +0200 @@ -395,6 +399,16 @@ if (strlen(value) > 1) { if (!insert || pos < int(strlen(value)) - 1) memmove(value + pos, value + pos + 1, strlen(value) - pos); + else if (insert && pos == int(strlen(value)) - 1) { + // in insert mode, deleting the last char replaces it with whitespace to keep cursor pos + if (value[pos] != ' ' || pos < 1) + value[pos] = ' '; + else { + // unless the last char is already a whitespace + value[pos-1] = ' '; + value[pos] = 0; + } + } // reduce position, if we removed the last character if (pos == int(strlen(value))) pos--;