From: Carl Worth <cworth@cworth.org>
Date: Thu, 5 Nov 2009 00:22:42 +0000 (-0800)
Subject: notmuch.el: Don't skip read messages when they are open.
X-Git-Tag: 0.1~572
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=3a330cb84a4f42d0013a772609fd10330a8413b0;p=notmuch.git

notmuch.el: Don't skip read messages when they are open.

More magic for the magic space bar: If a thread is entirely open,
(such as when viewing an old thread where every message is read), the
space bar now visits each message in turn (rather than skipping all of
the unread messages).
---

diff --git a/notmuch.el b/notmuch.el
index 68a01187..50e573dd 100644
--- a/notmuch.el
+++ b/notmuch.el
@@ -256,6 +256,16 @@ there are no more unread messages past the current point."
   (if (not (notmuch-show-message-unread-p))
       (notmuch-show-next-message)))
 
+(defun notmuch-show-next-open-message ()
+  "Advance to the the next message which is not hidden.
+
+If read messages are currently hidden, advance to the next unread
+message. Otherwise, advance to the next message."
+  (if (or (memq 'notmuch-show-body-read buffer-invisibility-spec)
+	  (assq 'notmuch-show-body-read buffer-invisibility-spec))
+      (notmuch-show-next-unread-message)
+    (notmuch-show-next-message)))
+
 (defun notmuch-show-previous-message ()
   "Backup to the beginning of the previous message in the buffer.
 
@@ -287,11 +297,11 @@ it."
       (notmuch-show-previous-message)
       (point))))
 
-(defun notmuch-show-mark-read-then-next-unread-message ()
+(defun notmuch-show-mark-read-then-next-open-message ()
   "Remove unread tag from current message, then advance to next unread message."
   (interactive)
   (notmuch-show-remove-tag "unread")
-  (notmuch-show-next-unread-message))
+  (notmuch-show-next-open-message))
 
 (defun notmuch-show-rewind ()
   "Do reverse scrolling compared to `notmuch-show-advance-marking-read-and-archiving'
@@ -326,7 +336,7 @@ scroll by a near screenful to read more of the message.
 
 Otherwise, (the end of the current message is already within the
 current window), remove the \"unread\" tag (if present) from the
-current message and advance to the next message.
+current message and advance to the next open message.
 
 Finally, if there is no further message to advance to, and this
 last message is already read, then archive the entire current
@@ -339,10 +349,10 @@ which this thread was originally shown."
     (if (> next (window-end))
 	(scroll-up nil)
       (if unread
-	  (notmuch-show-mark-read-then-next-unread-message)
+	  (notmuch-show-mark-read-then-next-open-message)
 	(if (notmuch-show-last-message-p)
 	    (notmuch-show-archive-thread)
-	  (notmuch-show-next-unread-message))))))
+	  (notmuch-show-next-open-message))))))
 
 (defun notmuch-show-markup-citations-region (beg end)
   (goto-char beg)