From: Carl Worth <cworth@cworth.org>
To: Alexander Botero-Lowry <alex.boterolowry@gmail.com>, Keith Packard
	<keithp@keithp.com>, notmuch@notmuchmail.org
In-Reply-To: <1258532999-9316-1-git-send-email-keithp@keithp.com>
References: <1258532999-9316-1-git-send-email-keithp@keithp.com>
	<867hto2p0t.fsf@fortitudo.i-did-not-set--mail-host-address--so-tickle-me>
Date: Thu, 19 Nov 2009 00:05:51 +0100
Message-ID: <87ws1nmo1c.fsf@yoom.home.cworth.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Subject: Re: [notmuch] [PATCH] Create a default notmuch-show-hook that
 highlights URLs and uses word-wrap
Precedence: list

On Wed, 18 Nov 2009 00:29:59 -0800, Keith Packard <keithp@keithp.com> wrote:
> I created the notmuch-show-hook precisely so I could add these two
> options, but I suspect most people will want them, so I just made them
> the default. If you don't want them, you can use remove-hook to get
> rid of this.

Thanks, Keith!

I had actually held off on pushing your original commit to add the hook
because I didn't like that it wasn't documenting what the hook was good
for. But now that we've got some good, suggested behavior there, I'm
much happier. So both are pushed now.

On Wed, 18 Nov 2009 00:52:34 -0800, Alexander Botero-Lowry <alex.boterolowry@gmail.com> wrote:
> Yes, hooks should be added for search as well. :)

Not a bad idea at all.

> visual-line-mode turns out to make subject look pretty ugly if there is a
> continuation. It doesn't do much good for the citation headers
> either. We probably need to do our own intelligent wrapping rather then
> use visual-line-mode to make this actually look right.

Yes, this stuff isn't perfect yet, but I think it's better than nothing.
We can do our own wrapping if these things get too annoying.

-Carl