Re: [Abook-devel] Trivial patch: Synonym for search-next, 'n'

[Jostein B. <jb...@br...>]

On 17.12.10,19:28, Hagen Fuchs wrote:
> Dear abook-devels,
> 
> I never quite understood why searching for the next matching item in
> abook required me to use the backslash character rather than 'n'ext.
> Apart from being defined like that by many ncurses-based interfaces, 'n'
> has the advantage of being a single-key character in all latin* keyboard
> configurations - the backslash sometimes involves an awkward
> AltGr-combination (for example, in de).
> 
> As the 'n' key is not defined by abook at the moment, I propose to make
> it a synonym for '\\'.  The corresponding one-line patch for 0.6.0pre2
> is attached.
> 
> Best Regards,
>   Hagen Fuchs
> 

> diff -rN -u old-abook-0.6.0pre2/ui.c new-abook-0.6.0pre2/ui.c
> --- old-abook-0.6.0pre2/ui.c	2010-12-17 19:17:32.468497621 +0100
> +++ new-abook-0.6.0pre2/ui.c	2010-12-17 19:17:32.472497572 +0100
> @@ -494,6 +494,7 @@
>  			case 'F': sort_by_field(NULL);	break;
>  
>  			case '/': ui_find(0);		break;
> +			case 'n':  // TODO Not in use, is it?
>  			case '\\': ui_find(1);		break;
>  
>  			case ' ': if(list_get_curitem() >= 0) {
> 

This looks like a useful patch. Will test it. Thanks!


Jostein