[PATCH 4/4] Update NEWS for user.other_name

[notmuch-archives.git] / a8 / b56a54cc79d335ca268e763e771d20c128a15a

Return-Path: <cworth@cworth.org>\r
X-Original-To: notmuch@notmuchmail.org\r
Delivered-To: notmuch@notmuchmail.org\r
Received: from localhost (localhost [127.0.0.1])\r
        by olra.theworths.org (Postfix) with ESMTP id 24DDF4196F2\r
        for <notmuch@notmuchmail.org>; Wed, 21 Apr 2010 14:03:05 -0700 (PDT)\r
X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
X-Spam-Flag: NO\r
X-Spam-Score: -2.89\r
X-Spam-Level: \r
X-Spam-Status: No, score=-2.89 tagged_above=-999 required=5\r
        tests=[ALL_TRUSTED=-1, BAYES_00=-1.9, T_MIME_NO_TEXT=0.01]\r
        autolearn=ham\r
Received: from olra.theworths.org ([127.0.0.1])\r
        by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
        with ESMTP id 7yTX8nBIqY00; Wed, 21 Apr 2010 14:03:03 -0700 (PDT)\r
Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
        by olra.theworths.org (Postfix) with ESMTP id 832CA431FC1;\r
        Wed, 21 Apr 2010 14:03:03 -0700 (PDT)\r
Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
        id 34280568DE4; Wed, 21 Apr 2010 14:03:03 -0700 (PDT)\r
From: Carl Worth <cworth@cworth.org>\r
To: David Edmondson <dme@dme.org>, notmuch <notmuch@notmuchmail.org>\r
Subject: Re: pull request\r
In-Reply-To: <87eiibq22s.fsf@ut.hh.sledj.net>\r
References: <87sk722sfq.fsf@ut.hh.sledj.net> <87eiibq22s.fsf@ut.hh.sledj.net>\r
Date: Wed, 21 Apr 2010 14:03:03 -0700\r
Message-ID: <87iq7kjz7c.fsf@yoom.home.cworth.org>\r
MIME-Version: 1.0\r
Content-Type: multipart/signed; boundary="=-=-=";\r
        micalg=pgp-sha1; protocol="application/pgp-signature"\r
X-BeenThere: notmuch@notmuchmail.org\r
X-Mailman-Version: 2.1.13\r
Precedence: list\r
List-Id: "Use and development of the notmuch mail system."\r
        <notmuch.notmuchmail.org>\r
List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
        <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
List-Post: <mailto:notmuch@notmuchmail.org>\r
List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
        <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
X-List-Received-Date: Wed, 21 Apr 2010 21:03:05 -0000\r
\r
--=-=-=\r
Content-Transfer-Encoding: quoted-printable\r
\r
On Mon, 19 Apr 2010 09:27:39 +0100, David Edmondson <dme@dme.org> wrote:\r
> commit 7dedc95af671173a57bafd973604614c03121ce6\r
>=20\r
>     emacs: JSON based implementation\r
\r
I merged this and followed it up immediately with a commit to remove the\r
ability to toggle the body visibility separate from the message\r
visibility. That was just confusing and none too useful.\r
\r
> commit 514e14c42e214718768a1ec94cb869cd3eb47114\r
>\r
>     emacs: Move body markup to a separate file\r
\r
This is pushed.\r
\r
> commit 2b6201fbf9209a875f216d48c30b95a6f583c575\r
>=20\r
>     emacs: Add more functions to clean up text/plain parts\r
>=20=20=20=20=20\r
>     Add:\r
>     - notmuch-wash-wrap-long-lines: Wrap lines longer than the width of\r
>       the current window whilst maintaining any citation prefix.\r
>     - notmuch-wash-tidy-citations: Tidy up citations by:\r
>       - compress repeated otherwise blank citation lines,\r
>       - remove otherwise blank citation lines at the head and tail of a\r
>         citation and remove blank lines between attribution statements and\r
>         the citation,\r
>     - notmuch-wash-compress-blanks: Compress repeated blank lines and\r
>       remove leading and trailing blank lines.\r
\r
I did not push this one. I think this munges messages too much to be on\r
by default.\r
\r
Consider a window of 80 columns and a thread with messages all wrapped\r
at a reasonable boundary, (75 columns or whatever). And consider that\r
the thread causes indentation such that messages start running into the\r
right edge of the window.\r
\r
Before the above patch, this scenario will cause the message to be\r
displayed in an obviously incorrect fashion, (some lines will only have\r
one or two words displayed on them). But since the content is unchanged\r
From=20the original message (other than the extra indentation whitespace)\r
I can just widen my emacs window and view the message correctly.\r
\r
After the patch, the broken case looks a little nicer, (lines that have\r
only one or two words are now indented and citation markers are added),\r
but if I widen the window nothing gets fixed. So that's annoying.\r
\r
With sup, the approach for this scenarios was to effectively make the\r
buffer much wider, and to automatically scroll the buffer to the left\r
when necessary so that the current message was made visible with no\r
wrapping. I liked that.\r
\r
Dealing with broken messages that have entire paragraphs unwrapped is a\r
separate issue from displaying correct messages nicely. I'd like to make\r
correct messages work well first, and then worry about the broken\r
messages. (For those, I'm much more willing to accept munged display of\r
the message.)\r
\r
> commit c7872f5e1f11cfa10d93cb818c5f6f6c0835b918\r
>\r
>     emacs/notmuch-show.el: Improved part labelling\r
...=20\r
> commit 951db85a55a1893e766b26de1377dda5b4573366\r
>=20\r
>     emacs: Use `mm-display-part' when possible\r
...\r
> commit 5c060ded87ec4dc479625348708ef73852d60b36\r
>=20\r
>     emacs: Use mailcap.el to guess the type of application/octet-stream p=\r
arts\r
...\r
> commit 5fcbb528384b7bda838f7c77434def15f85c7382\r
>=20\r
>     emacs: Display all body parts using `notmuch part --part=3D<n>'\r
\r
All four of these are pushed now. I like the inline display of\r
images. (Though I wish that emacs would allow the scrollbar to display\r
partial images---the sudden disappearance of the message once it hits\r
the top of the window is annoying.)\r
\r
> commit 9e193a3998b7503e35d21013c71cc4ecaf6c9d50\r
>=20\r
>     emacs/notmuch-wash.el: Add `notmuch-wash-inline-patch'\r
\r
The commit message for this one was enough to make me skip it for now:\r
\r
>     Due to the scope for error in detecting inline patches (and their\r
>     extent), this function is not enabled by default.\r
\r
For this entire series it would be very nice to have some emacs-based\r
testing in the test suite. And for a patch like this with heuristics\r
that are known to not be perfect, I think it's essential to have that\r
testing in place first.\r
\r
> commit 444de7e73d988cab9b8d1fef851c8ad26174a996\r
>=20\r
>     emacs/notmuch-show.el: Part headers are real buttons that save the pa=\r
rt\r
\r
Pushed. This is a delightful improvement to our handling of attachments.\r
\r
I'd like the text on the button to give some more instruction, such as:\r
\r
        "Click or press Enter to save."\r
\r
And if we could pull it off, (I don't know how flexible the button\r
widget is in emacs), it would be nice to have something like this as\r
well:\r
\r
        "Or press V to view."\r
\r
so that one could easily use an external viewer for a single attachment.\r
\r
Meanwhile, another issue with the result of this series is that I now\r
seem to get rendering for both the text/plain and the text/html\r
alternatives when a message has both. For now, the paragraphs are\r
wrapped much more nicely in the rendering of the html portion, but links\r
are apparently entirely missing. The link URLs at least appear in the\r
text/plain rendering, (which is pretty ugly, but at least not impossible\r
to use).\r
\r
If we could get one version or the other working completely, then it\r
would be nice to display only one.\r
\r
>     Don't attempt to save 'fake' parts generated by\r
>     `notmuch-wash-inline-patch'.\r
\r
There was one portion of the patch that did not apply since I hadn't\r
applied an earlier patch. It looked to me like it was the portion\r
described by the above sentence, so I dropped that sentence from the\r
commit message.\r
\r
> commit e9d737feb5a49fd59e1f27bccd24cac2fd1ef749\r
>=20\r
>     emacs/notmuch-show.el: Add `notmuch-show-toggle-all' bound to M-RET\r
>=20=20=20=20=20\r
>     `notmuch-show-toggle-all' changes the visibility all of the messages\r
>     in the current thread. By default it makes all of the messages not\r
>     visible. With a prefix argument, it makes them all visible.\r
\r
I didn't push this one yet.\r
\r
The feature is *almost* what I want. It's just that I often want to open\r
all messages in a thread, (and I've never found myself wanting to close\r
all messages). So I'd like to switch which behavior requires the prefix\r
argument here.\r
\r
Also, care should be taken in the documentation of this function so that\r
the first line reads as a complete sentence. This is because our help\r
screen (with the "?" key) displays only the first line of help for each\r
function.\r
\r
[Aside: We could improve our help screen by making RET on any line then\r
bring up the full help message for that function.]\r
\r
> commit b3be927b54956a7258f203159e0bdb954e686c80\r
>=20\r
>     emacs: Support for customizing search result display\r
\r
Finally, this is pushed. It's a quite lovely feature that's quite easy\r
to take advantage of from "M-x customize-group" "notmuch".\r
\r
Anyone, if you haven't tried that mode yet for customizing notmuch, you\r
should give it a try. (And we should add some hint somewhere to make it\r
easier to find. Perhaps a keybinding to get to the customization\r
buffer.)\r
\r
Thanks again, David!\r
\r
=2DCarl\r
\r
--=-=-=\r
Content-Type: application/pgp-signature\r
\r
-----BEGIN PGP SIGNATURE-----\r
Version: GnuPG v1.4.10 (GNU/Linux)\r
\r
iD8DBQFLz2gH6JDdNq8qSWgRAgbPAJ9n2AIL6ub0MNvvTsygQo7/Gran1QCfTYl4\r
BEV4i7MTom80zbdLqF35GtY=\r
=mHpC\r
-----END PGP SIGNATURE-----\r
--=-=-=--\r