From: david@tethera.net
To: notmuch@notmuchmail.org
Date: Fri, 25 Dec 2009 16:09:06 -0400
Message-Id: <1261771748-23687-1-git-send-email-david@tethera.net>
In-Reply-To: <87my17965l.fsf@anar.kanru.info>
References: <87my17965l.fsf@anar.kanru.info>
Subject: [notmuch] Update Patch Series. Only collapse part of citations.
Precedence: list

On Fri, 25 Dec 2009 09:47:02 +0800, Kan-Ru Chen <kanru@kanru.info> wrote:

> I like this idea, but this patch hides all citations larger than the
> threshold. I'd like to see limited lines of citations been displayed.

Since you ask nicely, OK :).

I kept it as two patches so that people could compare the usability of
both versions.  

[PATCH 1/2] notmuch.el: Refactor citation markup. Variables for minimum size, button text.

This is as before, except one stray '*' deleted from the regex for
matching citations. This bug was only detected when I started
partially hiding citations.

[PATCH 2/2] notmuch.el: show some of citation even when hiding.  

This implements Kan-Ru's suggested format. I also changed the default
text on the citation buttons so that it made a bit more sense when the
citation was expanded.

At first I thought having the buttons in the middle of an expanded
citation would be really irritating, but I got used to it pretty
quickly. So give it a chance ;).  Hiding buttons or replacing them
looks a bit messy to me.

David