DomainKey-Signature: a=rsa-sha1; c=nofws; d=maillard.im; h=from:to
	:subject:in-reply-to:references:reply-to:date:message-id
	:mime-version:content-type; q=dns; s=postfix; b=VaOtXRcYjWXbDWfk
	IZFhoBRA55M8H29p0AQXJfZmnwo+xncIBgdYNaxDfN/U5BidfemP73xEzsz1UTMp
	vIOI7WS/PXy4S1K6P9wXkBbNaaTUsgNUC4op30Rhuc56bxEIDHfuVX42DZteX5GE
	henj6d+NxBH1QsVWuoFsg0EEdyw=
From: Xavier Maillard <xavier@maillard.im>
To: Pieter Praet <pieter@praet.org>, Kan-Ru Chen <kanru@kanru.info>,
	notmuch@notmuchmail.org
Subject: Re: [PATCH 2/2] emacs: notmuch-hello: Add "n" and "p" to mod map
In-Reply-To:
 <87lizatjik.fsf@A7GMS.i-did-not-set--mail-host-address--so-tickle-me>
References: <1280195325-9066-1-git-send-email-kanru@kanru.info>
	<1280195325-9066-3-git-send-email-kanru@kanru.info>
	<87lizatjik.fsf@A7GMS.i-did-not-set--mail-host-address--so-tickle-me>
User-Agent: Notmuch/0.5-133-gc509598 (http://notmuchmail.org) Emacs/23.2.1
	(i486-slackware-linux-gnu)
Date: Mon, 18 Apr 2011 09:36:45 +0200
Message-ID: <m2vcycyprm.fsf@gnu.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Precedence: list
Reply-To: Xavier Maillard <xma@gnu.org>

Hi Pieter,

On Sat, 16 Apr 2011 15:26:11 +0200, Pieter Praet <pieter@praet.org> wrote:
> On Tue, 27 Jul 2010 09:48:45 +0800, Kan-Ru Chen <kanru@kanru.info> wrote:
> > Bind "n" to widget-forward and "p" to widget-backward to unify the
> > interface since other notmuch major mode also accept "n" and "p" to
> > navigation.
> > ---
> >  emacs/notmuch-hello.el |    2 ++
> >  1 files changed, 2 insertions(+), 0 deletions(-)
> > 
> > diff --git a/emacs/notmuch-hello.el b/emacs/notmuch-hello.el
> > index 6ef03bb..7def241 100644
> > --- a/emacs/notmuch-hello.el
> > +++ b/emacs/notmuch-hello.el
> > @@ -294,6 +294,8 @@ should be. Returns a cons cell `(tags-per-line width)'."
> >      (define-key map "=" 'notmuch-hello-update)
> >      (define-key map "G" 'notmuch-hello-poll-and-update)
> >      (define-key map (kbd "<C-tab>") 'widget-backward)
> > +    (define-key map "n" 'widget-forward)
> > +    (define-key map "p" 'widget-backward)
> >      (define-key map "m" 'notmuch-mua-mail)
> >      (define-key map "s" 'notmuch-hello-goto-search)
> >      map)
> > -- 
> > 1.7.1
> > 
> > 
> 
> 
> Sounds logical in theory, but in practice, you'll soon get stuck
> spamming "nnnnnpppnwtf" in the search field.
> 
> That, and TAB/S-TAB feels more natural.

More natural than what exactly ? Anyway, that would not hurt to add 'n'
and 'p' as another way to navigate through widgets in my opinion
(whatsmore that's how customize also works -i.e navigation done both
with TAB/S-TAB and n/p)

/Xavier