unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
From: Mark Walters <markwalters1009@gmail.com>
To: notmuch@notmuchmail.org
Subject: [PATCH 0/4] Allow oldest-first view in tree view
Date: Fri, 26 Jun 2015 23:50:31 +0100	[thread overview]
Message-ID: <1435359035-6767-1-git-send-email-markwalters1009@gmail.com> (raw)

This series allows the user to toggle the sort order in tree view and,
in particular, allows oldest first view.

Note that in all cases messages within a thread are displayed in
thread order: the option only affects the order when multiple threads
are viewed. If you only use tree view for single threads then nothing
will change.

Since tree view uses the notmuch-show command line as a backend this
means we need to add a --sort option tonotmuch show. This is the first
patch.

The second patch implements the toggle sort order option bound to 'o'
(so the same as the corresponding command in search view).

These third and fourth patches are more optional: they make notmuch
tree view inherit the notmuch search order when called via Z
(notmuch-tree-from-search-current-query), and use the default
notmuch-search sort order (notmuch-search-oldest-first) when called
interactively. The third patch does the code change and the fourth the
test update.

I think these last two patches make sense for consistency but I have
no strong feelings otherwise.

Best wishes

Mark




Mark Walters (4):
  cli: show: allow sort order to be specified
  emacs: tree: bind o to toggle sort order
  emacs: tree: add sort argument to notmuch-tree
  test: tree: fix and a test for tree sort order changes

 doc/man1/notmuch-show.rst                          | 17 +++++++
 emacs/notmuch-tree.el                              | 44 +++++++++++++++---
 emacs/notmuch.el                                   |  2 +-
 notmuch-show.c                                     |  8 ++++
 test/T460-emacs-tree.sh                            | 11 ++++-
 .../notmuch-tree-tag-inbox-oldest-first            | 53 ++++++++++++++++++++++
 6 files changed, 126 insertions(+), 9 deletions(-)
 create mode 100644 test/tree.expected-output/notmuch-tree-tag-inbox-oldest-first

-- 
2.1.4

             reply	other threads:[~2015-06-26 22:50 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-26 22:50 Mark Walters [this message]
2015-06-26 22:50 ` [PATCH 1/4] cli: show: allow sort order to be specified Mark Walters
2015-07-31 19:11   ` David Bremner
2015-06-26 22:50 ` [PATCH 2/4] emacs: tree: bind o to toggle sort order Mark Walters
2015-07-31 19:39   ` David Bremner
2015-06-26 22:50 ` [PATCH 3/4] emacs: tree: add sort argument to notmuch-tree Mark Walters
2015-08-01  5:49   ` David Bremner
2015-06-26 22:50 ` [PATCH 4/4] test: tree: fix and a test for tree sort order changes Mark Walters

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://notmuchmail.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1435359035-6767-1-git-send-email-markwalters1009@gmail.com \
    --to=markwalters1009@gmail.com \
    --cc=notmuch@notmuchmail.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://yhetil.org/notmuch.git/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).