From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934004Ab0HESOB (ORCPT ); Thu, 5 Aug 2010 14:14:01 -0400 Received: from mail-fx0-f46.google.com ([209.85.161.46]:48989 "EHLO mail-fx0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933995Ab0HESN4 convert rfc822-to-8bit (ORCPT ); Thu, 5 Aug 2010 14:13:56 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; b=NI7YrIe4j5W8xXCRl1uvqS08yiVXqBPI9uHfAVgpl+rMCG1/RbavcWgDbf00kK64L1 sUjoldB2usspJYRfQriyeTisVnTNlcDG9B2AB8ZzWTWnxjeYEtCmCjN2aefbCGOK0ptz ygmgDsOU9Ok5C1N0hDGBMjB4SJ6J5mJNfxBHI= MIME-Version: 1.0 In-Reply-To: <20100805100408.5b724047.randy.dunlap@oracle.com> References: <1280838492-9942-1-git-send-email-nir.tzachar@gmail.com> <20100805100408.5b724047.randy.dunlap@oracle.com> Date: Thu, 5 Aug 2010 21:13:55 +0300 Message-ID: Subject: Re: [PATCH] nconfig: add search support From: Nir Tzachar To: Randy Dunlap Cc: mmarek@suse.cz, linux-kbuild@vger.kernel.org, linux-kernel@vger.kernel.org, sam@ravnborg.org Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Aug 5, 2010 at 8:04 PM, Randy Dunlap wrote: > On Tue,  3 Aug 2010 15:28:12 +0300 nir.tzachar@gmail.com wrote: > >> From: Nir Tzachar >> >> Remove the old hotkeys feature, and replace by a regular string search. >> Behaviour of search is as advised by Sam. >> >> From nconfig help: >> >> Searching: pressing '/' triggers search mode. nconfig performs a >>            regular string compare, case insensitive, starting at > > I would say: simple string compare > "regular" has no meaning (at least for me) there. Regular means strcasecmp... >>            the beginning of each menu line.\n" >>            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 Justin's input). > > Yes, the bottom menu and the typos are fixed.  Thanks. > > However, I'm not finding this terribly useful, mostly due to it starting > comparison at the beginning of each menu line. > > E.g., I'd like to be able to find "emulations" in the top-level menu here: >        Executable file formats / Emulations  ---> > or I'd like to be able to find "crc" menu items in any sub-menu. > Maybe I should just stick to config symbol searches.  I don't think it's all > that likely that people will know how each menu line text begins. > > We can replace strcasecmp with strcasestr. I agree it would be more useful. > As for the search UI, I'd rather that it be presented like the symbol search, > in a box, instead of just a single line at the top of the screen. But then it is not interactive. I was aiming for something similar to vim's search, where the search is matched as you type and the only free terminal real-estate to display the match string was at the top of the screen. I think such a minimal design is better than a cumbersome text box which displays the search results afterwards (as is symbol search), as the search is only intended for the currently displayed menu and the user would usually just want to save the extra typing of navigating to a specific menu item. Cheers. >> Signed-off-by: Nir Tzachar >> --- >>  scripts/kconfig/nconf.c     |  350 +++++++++++++++++++++++++------------------ >>  scripts/kconfig/nconf.gui.c |   20 ++-- >>  scripts/kconfig/nconf.h     |    3 +- >>  3 files changed, 213 insertions(+), 160 deletions(-) > > > --- > ~Randy > *** Remember to use Documentation/SubmitChecklist when testing your code *** >