* [PATCH] contrib: pick: bugfix. use notmuch-show-only-matching-messages rather than prefix
@ 2013-06-01 21:51 Mark Walters
2013-06-02 14:34 ` Tomi Ollila
2013-06-03 0:10 ` David Bremner
0 siblings, 2 replies; 3+ messages in thread
From: Mark Walters @ 2013-06-01 21:51 UTC (permalink / raw)
To: notmuch
Previously pick set a prefix argument prior to calling show in the
message pane to tell show to only show matching messages. This sets
notmuch-show-only-matching-messages instead which is much cleaner and
will work even if the user has configured show to default to showing
only matching messages.
---
contrib/notmuch-pick/notmuch-pick.el | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/contrib/notmuch-pick/notmuch-pick.el b/contrib/notmuch-pick/notmuch-pick.el
index 9079acf..2da43ea 100644
--- a/contrib/notmuch-pick/notmuch-pick.el
+++ b/contrib/notmuch-pick/notmuch-pick.el
@@ -376,8 +376,8 @@ Does NOT change the database."
(split-window-vertically (/ (window-height) 4)))
(with-selected-window notmuch-pick-message-window
;; Since we are only displaying one message do not indent.
- (let ((notmuch-show-indent-messages-width 0))
- (setq current-prefix-arg '(4))
+ (let ((notmuch-show-indent-messages-width 0)
+ (notmuch-show-only-matching-messages t))
(setq buffer (notmuch-show id nil nil nil))))
;; We need the `let' as notmuch-pick-message-window is buffer local.
(let ((window notmuch-pick-message-window))
--
1.7.9.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] contrib: pick: bugfix. use notmuch-show-only-matching-messages rather than prefix
2013-06-01 21:51 [PATCH] contrib: pick: bugfix. use notmuch-show-only-matching-messages rather than prefix Mark Walters
@ 2013-06-02 14:34 ` Tomi Ollila
2013-06-03 0:10 ` David Bremner
1 sibling, 0 replies; 3+ messages in thread
From: Tomi Ollila @ 2013-06-02 14:34 UTC (permalink / raw)
To: Mark Walters, notmuch
On Sun, Jun 02 2013, Mark Walters <markwalters1009@gmail.com> wrote:
> Previously pick set a prefix argument prior to calling show in the
> message pane to tell show to only show matching messages. This sets
> notmuch-show-only-matching-messages instead which is much cleaner and
> will work even if the user has configured show to default to showing
> only matching messages.
> ---
LGTM.
Tomi
> contrib/notmuch-pick/notmuch-pick.el | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/contrib/notmuch-pick/notmuch-pick.el b/contrib/notmuch-pick/notmuch-pick.el
> index 9079acf..2da43ea 100644
> --- a/contrib/notmuch-pick/notmuch-pick.el
> +++ b/contrib/notmuch-pick/notmuch-pick.el
> @@ -376,8 +376,8 @@ Does NOT change the database."
> (split-window-vertically (/ (window-height) 4)))
> (with-selected-window notmuch-pick-message-window
> ;; Since we are only displaying one message do not indent.
> - (let ((notmuch-show-indent-messages-width 0))
> - (setq current-prefix-arg '(4))
> + (let ((notmuch-show-indent-messages-width 0)
> + (notmuch-show-only-matching-messages t))
> (setq buffer (notmuch-show id nil nil nil))))
> ;; We need the `let' as notmuch-pick-message-window is buffer local.
> (let ((window notmuch-pick-message-window))
> --
> 1.7.9.1
>
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] contrib: pick: bugfix. use notmuch-show-only-matching-messages rather than prefix
2013-06-01 21:51 [PATCH] contrib: pick: bugfix. use notmuch-show-only-matching-messages rather than prefix Mark Walters
2013-06-02 14:34 ` Tomi Ollila
@ 2013-06-03 0:10 ` David Bremner
1 sibling, 0 replies; 3+ messages in thread
From: David Bremner @ 2013-06-03 0:10 UTC (permalink / raw)
To: Mark Walters, notmuch
Mark Walters <markwalters1009@gmail.com> writes:
> Previously pick set a prefix argument prior to calling show in the
> message pane to tell show to only show matching messages. This sets
> notmuch-show-only-matching-messages instead which is much cleaner and
> will work even if the user has configured show to default to showing
> only matching messages.
pushed.
d
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-06-03 0:10 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-06-01 21:51 [PATCH] contrib: pick: bugfix. use notmuch-show-only-matching-messages rather than prefix Mark Walters
2013-06-02 14:34 ` Tomi Ollila
2013-06-03 0:10 ` David Bremner
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).