1 Return-Path: <pieter@praet.org>
\r
2 X-Original-To: notmuch@notmuchmail.org
\r
3 Delivered-To: notmuch@notmuchmail.org
\r
4 Received: from localhost (localhost [127.0.0.1])
\r
5 by olra.theworths.org (Postfix) with ESMTP id 531F6431FBD
\r
6 for <notmuch@notmuchmail.org>; Wed, 18 Jan 2012 02:10:41 -0800 (PST)
\r
7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org
\r
11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5
\r
12 tests=[RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled
\r
13 Received: from olra.theworths.org ([127.0.0.1])
\r
14 by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)
\r
15 with ESMTP id pLGlHYDhVjox for <notmuch@notmuchmail.org>;
\r
16 Wed, 18 Jan 2012 02:10:40 -0800 (PST)
\r
17 Received: from mail-ww0-f45.google.com (mail-ww0-f45.google.com
\r
18 [74.125.82.45]) (using TLSv1 with cipher RC4-SHA (128/128 bits)) (No client
\r
19 certificate requested) by olra.theworths.org (Postfix) with ESMTPS id
\r
20 52D82431FAE for <notmuch@notmuchmail.org>; Wed, 18 Jan 2012 02:10:40 -0800
\r
22 Received: by wgbdr13 with SMTP id dr13so2155027wgb.2
\r
23 for <notmuch@notmuchmail.org>; Wed, 18 Jan 2012 02:10:39 -0800 (PST)
\r
24 Received: by 10.180.81.72 with SMTP id y8mr35421554wix.14.1326881439197;
\r
25 Wed, 18 Jan 2012 02:10:39 -0800 (PST)
\r
26 Received: from localhost ([109.131.97.13])
\r
27 by mx.google.com with ESMTPS id gy6sm21588850wib.11.2012.01.18.02.10.37
\r
28 (version=TLSv1/SSLv3 cipher=OTHER);
\r
29 Wed, 18 Jan 2012 02:10:38 -0800 (PST)
\r
30 From: Pieter Praet <pieter@praet.org>
\r
31 To: Aaron Ecay <aaronecay@gmail.com>, David Edmondson <dme@dme.org>,
\r
32 Jameson Graef Rollins <jrollins@finestructure.net>
\r
33 Subject: Re: [PATCH] v2 emacs: colorize buttonized 'id:' links depending on
\r
34 the target message's state
\r
35 In-Reply-To: <m2sjjfb9xx.fsf@gmail.com>
\r
36 References: <87hazva8qe.fsf@praet.org>
\r
37 <1326733053-25185-1-git-send-email-pieter@praet.org>
\r
38 <m2sjjfb9xx.fsf@gmail.com>
\r
39 User-Agent: Notmuch/0.11+86~g67f4140 (http://notmuchmail.org) Emacs/23.3.1
\r
40 (x86_64-unknown-linux-gnu)
\r
41 Date: Wed, 18 Jan 2012 11:08:54 +0100
\r
42 Message-ID: <87aa5l8gqx.fsf@praet.org>
\r
44 Content-Type: text/plain; charset=utf-8
\r
45 Content-Transfer-Encoding: quoted-printable
\r
46 Cc: Notmuch Mail <notmuch@notmuchmail.org>
\r
47 X-BeenThere: notmuch@notmuchmail.org
\r
48 X-Mailman-Version: 2.1.13
\r
50 List-Id: "Use and development of the notmuch mail system."
\r
51 <notmuch.notmuchmail.org>
\r
52 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,
\r
53 <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>
\r
54 List-Archive: <http://notmuchmail.org/pipermail/notmuch>
\r
55 List-Post: <mailto:notmuch@notmuchmail.org>
\r
56 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>
\r
57 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,
\r
58 <mailto:notmuch-request@notmuchmail.org?subject=subscribe>
\r
59 X-List-Received-Date: Wed, 18 Jan 2012 10:10:41 -0000
\r
61 On Mon, 16 Jan 2012 16:43:06 -0500, Aaron Ecay <aaronecay@gmail.com> wrote:
\r
62 > On Mon, 16 Jan 2012 17:57:33 +0100, Pieter Praet <pieter@praet.org> wrote:
\r
63 > > * emacs/notmuch-show.el (notmuch-show-buttonized-link-colors):
\r
64 > > new defcustom, allows toggling colorization of buttonized links.
\r
66 > > * emacs/notmuch-show.el (notmuch-show-buttonized-link-present),
\r
67 > > * emacs/notmuch-show.el (notmuch-show-buttonized-link-present-and-unrea=
\r
69 > > * emacs/notmuch-show.el (notmuch-show-buttonized-link-missing):
\r
70 > > new faces for buttonized id: links.
\r
72 > > * emacs/notmuch-show.el (notmuch-show-found-target-p): add optional arg
\r
73 > > VERIFY-UNREAD which causes results to be filtered by "tag:unread".
\r
75 > > * emacs/notmuch-show.el (notmuch-show-buttonize-links): use different
\r
76 > > face property depending on the result of `notmuch-show-found-target-p=
\r
78 > > causing buttons to available, available-and-unread and missing messag=
\r
80 > > to be displayed in a different color.
\r
82 > I really like the idea behind this patch, but it has the very small
\r
83 > problem that it colorizes too much. So in reading this thread, there
\r
84 > are things like =E2=80=9Cid:=E2=80=99s=E2=80=9D and =E2=80=9Cid:?=E2=80=
\r
85 =9D that get colored the missing-message
\r
86 > color (a very angry red, by default). Though this isn=E2=80=99t likely t=
\r
88 > very frequent problem with email messages that are not on this listserv
\r
89 > :), it would be nice to fix it. [...]
\r
91 Excellent suggestion!
\r
93 Amended patch follows.
\r
95 > [...] Maybe you could change the regex that
\r
96 > matches id:=E2=80=99s to require a little more structure =E2=80=93 an at-=
\r
98 > Or even requiring more than (say) 5 non-space characters after the
\r
99 > message id would cut down sharply on the false positive rate.
\r
102 Not sure how that would pan out. It's fairly common behaviour to put
\r
103 one or more spaces after a inline Message-Id, so I don't think such a
\r
104 limitation would be warmly recepted.
\r