Re: [notmuch] [PATCH] Create a default notmuch-show-hook that highlights URLs and uses word-wrap

Subject: Re: [notmuch] [PATCH] Create a default notmuch-show-hook that highlights URLs and uses word-wrap

Date: Thu, 19 Nov 2009 00:05:51 +0100

To: Alexander Botero-Lowry, Keith Packard, notmuch@notmuchmail.org

Cc:

From: Carl Worth


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

Thread: