[PATCH v2] emacs: call notmuch-show instead of notmuch-search in buttonised id: links

Pieter Praet pieter at praet.org
Thu Jan 12 09:20:12 PST 2012


On Thu, 22 Dec 2011 20:59:12 -0800, Jameson Graef Rollins <jrollins at finestructure.net> wrote:
> Since message-ids necessarily match just a single message, there's no
> reason to do a search for the id before viewing the actual message;
> the search just becomes an extra screen to click through.  Clicking on
> an id: links now just jumps straight to the message itself.
> ---
> This just fixes the subject of the patch, which had a typo.  Sorry.
> 
>  emacs/notmuch-show.el |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el
> index 98db8f2..623d521 100644
> --- a/emacs/notmuch-show.el
> +++ b/emacs/notmuch-show.el
> @@ -803,7 +803,7 @@ a corresponding notmuch search."
>        (remove-overlays (match-beginning 0) (match-end 0) 'goto-address t)
>        (make-text-button (match-beginning 0) (match-end 0)
>  			'action `(lambda (arg)
> -				   (notmuch-search ,(match-string-no-properties 0)))
> +				   (notmuch-show ,(match-string-no-properties 0)))
>  			'follow-link t
>  			'help-echo "Mouse-1, RET: search for this message"
>  			'face goto-address-mail-face))))
> -- 
> 1.7.7.3

Anti-RSI FTW!

However...  If no message with that id: exists, `notmuch-show'
will drop us to a blank screen.

See id:"87lisjzrsc.fsf at kepler.schwinge.homeip.net" for some mock
'id:' links which demonstrate this nicely.

Patches follow.


Peace

-- 
Pieter


More information about the notmuch mailing list