Re: [PATCH] Add Emacs' imenu support in notmuch-show and notmuch-search

Subject: Re: [PATCH] Add Emacs' imenu support in notmuch-show and notmuch-search

Date: Sun, 11 Jun 2017 08:00:26 -0300

To: Damien Cassou, notmuch@notmuchmail.org

Cc:

From: David Bremner


Damien Cassou <damien@cassou.me> writes:

> David Bremner <david@tethera.net> writes:
>> I am indeed using the default. I think you forgot the screen 
>> shot. 
>
>
> indeed. Attached to this email.
>
>
>>> I can still get rid of indentation if you confirm you don't 
>>> want  it. 
>> 
>> I think so, although to be honest I never tried imenu before 
>> testing your patches, perhaps we should wait for other opinions. 
>
>
> I advise you to install counsel at least for that (I don't use it 
> for anything else).
>

OK, I see with counsel-imenu the current indexing by header lines is
reasonable. It might be improvable by adding the subject, but I'm not
sure about line lengths.

- maybe the docstrings should recomment counsel-imenu?
- I think the indentation should probably go to make it more
  usable with the builtin imenu



Thread: