Return-Path: <pieter@praet.org>
X-Original-To: notmuch@notmuchmail.org
Delivered-To: notmuch@notmuchmail.org
Received: from localhost (localhost [127.0.0.1])
	by olra.theworths.org (Postfix) with ESMTP id 71591431FD0
	for <notmuch@notmuchmail.org>; Thu,  7 Jul 2011 00:04:35 -0700 (PDT)
X-Virus-Scanned: Debian amavisd-new at olra.theworths.org
X-Spam-Flag: NO
X-Spam-Score: -0.7
X-Spam-Level: 
X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5
	tests=[RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled
Received: from olra.theworths.org ([127.0.0.1])
	by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)
	with ESMTP id wHoEaaMQzB-R for <notmuch@notmuchmail.org>;
	Thu,  7 Jul 2011 00:04:34 -0700 (PDT)
Received: from mail-ww0-f45.google.com (mail-ww0-f45.google.com
 [74.125.82.45])	(using TLSv1 with cipher RC4-SHA (128/128 bits))	(No client
 certificate requested)	by olra.theworths.org (Postfix) with ESMTPS id
 325E5431FB6	for <notmuch@notmuchmail.org>; Thu,  7 Jul 2011 00:04:34 -0700
 (PDT)
Received: by wwj40 with SMTP id 40so505371wwj.2
	for <notmuch@notmuchmail.org>; Thu, 07 Jul 2011 00:04:32 -0700 (PDT)
Received: by 10.227.174.79 with SMTP id s15mr368546wbz.68.1310022272426;
	Thu, 07 Jul 2011 00:04:32 -0700 (PDT)
Received: from localhost ([109.131.160.107])
	by mx.google.com with ESMTPS id fy9sm1587025wbb.25.2011.07.07.00.04.30
	(version=TLSv1/SSLv3 cipher=OTHER);
	Thu, 07 Jul 2011 00:04:31 -0700 (PDT)
From: Pieter Praet <pieter@praet.org>
To: Daniel Schoepe <daniel.schoepe@googlemail.com>, notmuch@notmuchmail.org
Subject: Re: [PATCH v2] emacs: Turn id:"<message-id>" elements into buttons
	for notmuch searches
In-Reply-To: <1309961881-28333-2-git-send-email-daniel.schoepe@googlemail.com>
References: <1309896760-378-1-git-send-email-daniel.schoepe@googlemail.com>
	<1309961881-28333-1-git-send-email-daniel.schoepe@googlemail.com>
	<1309961881-28333-2-git-send-email-daniel.schoepe@googlemail.com>
User-Agent: Notmuch/0.5-331-g04cf689 (http://notmuchmail.org) Emacs/23.1.50.1
	(x86_64-pc-linux-gnu)
Date: Thu, 07 Jul 2011 09:04:28 +0200
Message-ID: <87pqlmwozn.fsf@praet.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
X-BeenThere: notmuch@notmuchmail.org
X-Mailman-Version: 2.1.13
Precedence: list
List-Id: "Use and development of the notmuch mail system."
	<notmuch.notmuchmail.org>
List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,
	<mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>
List-Archive: <http://notmuchmail.org/pipermail/notmuch>
List-Post: <mailto:notmuch@notmuchmail.org>
List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>
List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,
	<mailto:notmuch-request@notmuchmail.org?subject=subscribe>
X-List-Received-Date: Thu, 07 Jul 2011 07:04:35 -0000

On Wed,  6 Jul 2011 16:18:01 +0200, Daniel Schoepe <daniel.schoepe@googlemail.com> wrote:
> This fixes the minor annoyance that message ids were parsed as mail
> addresses by goto-address-mode in notmuch-show buffers.
> ---
>  emacs/notmuch-show.el |   24 +++++++++++++++++++++---
>  1 files changed, 21 insertions(+), 3 deletions(-)
> 
> diff --git a/emacs/notmuch-show.el b/emacs/notmuch-show.el
> index f96743b..f924d17 100644
> --- a/emacs/notmuch-show.el
> +++ b/emacs/notmuch-show.el
> @@ -27,6 +27,7 @@
>  (require 'mm-decode)
>  (require 'mailcap)
>  (require 'icalendar)
> +(require 'goto-addr)
>  
>  (require 'notmuch-lib)
>  (require 'notmuch-query)
> @@ -778,6 +779,24 @@ current buffer, if possible."
>  (defvar notmuch-show-parent-buffer nil)
>  (make-variable-buffer-local 'notmuch-show-parent-buffer)
>  
> +(defun notmuch-show-buttonise-links (start end)
> +  "Buttonise URLs and mail addresses between START and END.
> +
> +This also turns id:\"<message id>\"-parts into buttons for
> +a corresponding notmuch search."
> +  (goto-address-fontify-region start end)
> +  (save-excursion
> +    (goto-char start)
> +    (while (re-search-forward "id:\"[^ ]+\"" end t)
> +      ;; remove the overlay created by goto-address-mode
> +      (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)))
> +			'follow-link t
> +			'help-echo "Mouse-1, RET: search for this message"
> +			'face goto-address-mail-face))))
> +
>  ;;;###autoload
>  (defun notmuch-show (thread-id &optional parent-buffer query-context buffer-name crypto-switch)
>    "Run \"notmuch show\" with the given thread ID and display results.
> @@ -822,9 +841,8 @@ function is used. "
>  	  (notmuch-show-insert-forest
>  	   (notmuch-query-get-threads basic-args))))
>  
> -      ;; Enable buttonisation of URLs and email addresses in the
> -      ;; buffer.
> -      (goto-address-mode t)
> +      (jit-lock-register #'notmuch-show-buttonise-links)
> +
>        ;; Act on visual lines rather than logical lines.
>        (visual-line-mode t)
>  
> -- 
> 1.7.5.4
> 
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch


Signed-off-by: Pieter Praet <pieter@praet.org>

Fantastic usability improvement Daniel!


Peace

-- 
Pieter