Re: [PATCH] nconfig: add search support

[Sam Ravnborg]

On Sun, Aug 08, 2010 at 04:50:06PM +0300, nir.tzachar@xxxxxxxxx wrote:
> From: Nir Tzachar <nir.tzachar@xxxxxxxxx>
> 
> Remove the old hotkeys feature, and replace it by an interactive string
> search.
> >From nconfig help:
> 
> Searching: pressing '/' triggers interactive search mode.
>            nconfig performs a case insensitive search for the string
>            in the menu prompts (no regex support).
>            Pressing the up/down keys highlights the previous/next
>            matching item. Backspace removes one character from the
>            match string. Pressing either '/' again or ESC exits
>            search mode. All other keys behave normally.
> 
> Miscellaneous other changes (including Rundy's and Justin's input).
> 
> Signed-off-by: Nir Tzachar <nir.tzachar@xxxxxxxxx>

I like this search mode much better than the hotkey stuff.
And with the new key assignments things works almost as I would expect.

I am not sure if I prefer "space" to bind to usual
function or that it can be used to enter a whitespace.

And getting rid of the capitalization was good!

Acked-by: Sam Ravnborg <sam@xxxxxxxxxxxx>

	Sam
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/