Re: [PATCH] emacs: Add notmuch-hello.el, a friendly frontend to notmuch

[Carl Worth <cworth@cworth.org>]

[-- Attachment #1: Type: text/plain, Size: 2807 bytes --]


-- 
carl.d.worth@intel.com
On Mon, 26 Apr 2010 10:36:28 +0100, David Edmondson <dme@dme.org> wrote:
> I'll do this, but I wonder if it is actually good advice.

My primary point is that all of the implemented functionality,
(including notmuch-hello and notmuch-addresses), needs to be made
available with our documented approach for loading notmuch in
emacs. Currently that's "(require 'notmuch)".

We can certainly come up with another approach if there's a better one.

> So if `notmuch-folders' is set before notmuch-hello.el is loaded, you
> get some saved-searches copied from your folders (but this won't
> override any specific settings you made for
> `notmuch-hello-saved-searches').

OK. That is nice. But also a trick order dependency, yes.

> of autoloads in it. Oh, or "(require 'notmuch)" should just load
> something that has the autoloads (d'oh!) and some
> `define-mail-user-agent' goop so that `compose-mail' does the right
> thing.
> 
> I'll try to produce a patch for this last thing.

Yes, that sounds perfect. I'll look forward to that.

> >   * The logo background is gray on my white-background emacs. That
> >     looks odd.
> 
> Patch sent.

Thanks. Pushed.

> >   * It would be great if point were in the search bar right when this
> >     mode started.
> 
> This is customisable, and I see that you changed the default ;-)

Yes. I'm still on the fence about this change, (as we are talking in the
other thread).

> >   * I'd like the saved searches to appear before the recent searches I
> >     think.
> 
> Did you come to any conclusion about this?

The thing I most want is for my saved searches to be extremely easy to
select. With notmuch-folder, the view wasn't fancy at all, but it was
extremely functional in that the first saved search was under point
right away and just C-n/C-p (or arrow keys) could be used to select
others.

I haven't gotten that same convenience with notmuch-hello yet.

> >   * I'm not sure how useful the numbered shortcuts are for the recent
> >     searches
> 
> Patch sent which removes them.

Pushed.

> >   * Repeating a recent search by pressing RET on it creates a new
> >     identical search which doesn't seem all that useful.
> 
> Patch sent.

And pushed.

> We could have `notmuch-hello' use `notmuch-folders' if
> `notmuch-hello-saved-searches' is `nil'. Would that help?

Yes, that's what I had in mind here.

> > Finally, I'm quite inclined to make `notmuch' invoke this mode, so
> > think about that.
> 
> It seems straightforward, I'll produce a patch.

Excellent. A lot of these would have been easy for me to do of course,
but it seems to be working well for you to "own" the coding of our emacs
code for now. I really appreciate it!

-Carl

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]