* [EMACS] New commands notmuch-tree-filter{,-by-tag}
@ 2021-08-04 0:23 jao
2021-08-04 0:23 ` [PATCH v3 1/2] emacs: new command notmuch-tree-filter jao
2021-08-04 0:23 ` [PATCH v3 2/2] emacs: new command notmuch-tree-filter-by-tag jao
0 siblings, 2 replies; 5+ messages in thread
From: jao @ 2021-08-04 0:23 UTC (permalink / raw)
To: notmuch
The patches below supersede 20210801021410.261278-2-jao@gnu.org and
add, besides notmuch-tree-filter, the companion command
notmuch-tree-filter-by-tag.
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v3 1/2] emacs: new command notmuch-tree-filter
2021-08-04 0:23 [EMACS] New commands notmuch-tree-filter{,-by-tag} jao
@ 2021-08-04 0:23 ` jao
2021-08-21 14:32 ` David Bremner
2021-08-04 0:23 ` [PATCH v3 2/2] emacs: new command notmuch-tree-filter-by-tag jao
1 sibling, 1 reply; 5+ messages in thread
From: jao @ 2021-08-04 0:23 UTC (permalink / raw)
To: notmuch; +Cc: jao
This command is analogous to notmuch-filter, but is defined on tree
mode buffers.
---
doc/notmuch-emacs.rst | 3 +++
emacs/notmuch-tree.el | 16 ++++++++++++++++
2 files changed, 19 insertions(+)
diff --git a/doc/notmuch-emacs.rst b/doc/notmuch-emacs.rst
index 952fe2a5..5accfa60 100644
--- a/doc/notmuch-emacs.rst
+++ b/doc/notmuch-emacs.rst
@@ -302,6 +302,9 @@ tags.
``o`` ``notmuch-tree-toggle-order``
|docstring::notmuch-tree-toggle-order|
+``l`` ``notmuch-tree-filter``
+ Filter or LIMIT the current search results based on an additional query string
+
``g`` ``=``
Refresh the buffer
diff --git a/emacs/notmuch-tree.el b/emacs/notmuch-tree.el
index ef1ca4c5..f2938330 100644
--- a/emacs/notmuch-tree.el
+++ b/emacs/notmuch-tree.el
@@ -349,6 +349,7 @@ then NAME behaves like CMD."
(define-key map "r" 'notmuch-tree-reply-sender)
(define-key map "R" 'notmuch-tree-reply)
(define-key map "V" 'notmuch-tree-view-raw-message)
+ (define-key map "l" 'notmuch-tree-filter)
;; The main tree view bindings
(define-key map (kbd "RET") 'notmuch-tree-show-message)
@@ -1168,6 +1169,21 @@ The arguments are:
(interactive)
(notmuch-tree query query-context target buffer-name open-target t))
+(defun notmuch-tree-filter (query)
+ "Filter or LIMIT the current search results based on an additional query string.
+
+Runs a new tree search matching only messages that match both the
+current search results AND the additional query string provided."
+ (interactive (list (notmuch-read-query "Filter search: ")))
+ (let ((notmuch-show-process-crypto (notmuch-tree--message-process-crypto))
+ (grouped-query (notmuch-group-disjunctive-query-string query))
+ (grouped-original-query (notmuch-group-disjunctive-query-string
+ (notmuch-tree-get-query))))
+ (notmuch-tree-close-message-window)
+ (notmuch-tree (if (string= grouped-original-query "*")
+ grouped-query
+ (concat grouped-original-query " and " grouped-query)))))
+
;;; _
(provide 'notmuch-tree)
--
2.32.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v3 1/2] emacs: new command notmuch-tree-filter
2021-08-04 0:23 ` [PATCH v3 1/2] emacs: new command notmuch-tree-filter jao
@ 2021-08-21 14:32 ` David Bremner
0 siblings, 0 replies; 5+ messages in thread
From: David Bremner @ 2021-08-21 14:32 UTC (permalink / raw)
To: jao, notmuch; +Cc: jao
jao <jao@gnu.org> writes:
> This command is analogous to notmuch-filter, but is defined on tree
> mode buffers.
> ---
> doc/notmuch-emacs.rst | 3 +++
> emacs/notmuch-tree.el | 16 ++++++++++++++++
> 2 files changed, 19 insertions(+)
>
> diff --git a/doc/notmuch-emacs.rst b/doc/notmuch-emacs.rst
> index 952fe2a5..5accfa60 100644
> --- a/doc/notmuch-emacs.rst
> +++ b/doc/notmuch-emacs.rst
> @@ -302,6 +302,9 @@ tags.
> ``o`` ``notmuch-tree-toggle-order``
> |docstring::notmuch-tree-toggle-order|
>
> +``l`` ``notmuch-tree-filter``
> + Filter or LIMIT the current search results based on an additional query string
> +
> ``g`` ``=``
> Refresh the buffer
Please also update devel/emacs-keybindings.org, which is to help
developers avoid collisions or bad keybinding choices.
Other than that this patch looks OK to me.
d
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v3 2/2] emacs: new command notmuch-tree-filter-by-tag
2021-08-04 0:23 [EMACS] New commands notmuch-tree-filter{,-by-tag} jao
2021-08-04 0:23 ` [PATCH v3 1/2] emacs: new command notmuch-tree-filter jao
@ 2021-08-04 0:23 ` jao
2021-08-21 14:35 ` David Bremner
1 sibling, 1 reply; 5+ messages in thread
From: jao @ 2021-08-04 0:23 UTC (permalink / raw)
To: notmuch; +Cc: jao
---
doc/notmuch-emacs.rst | 4 ++++
emacs/notmuch-tree.el | 28 ++++++++++++++++++++++++----
2 files changed, 28 insertions(+), 4 deletions(-)
diff --git a/doc/notmuch-emacs.rst b/doc/notmuch-emacs.rst
index 5accfa60..36dcb116 100644
--- a/doc/notmuch-emacs.rst
+++ b/doc/notmuch-emacs.rst
@@ -305,6 +305,10 @@ tags.
``l`` ``notmuch-tree-filter``
Filter or LIMIT the current search results based on an additional query string
+``t`` ``notmuch-tree-filter-by-tag``
+ Filter the current search results based on an additional tag
+
+
``g`` ``=``
Refresh the buffer
diff --git a/emacs/notmuch-tree.el b/emacs/notmuch-tree.el
index f2938330..01a77b71 100644
--- a/emacs/notmuch-tree.el
+++ b/emacs/notmuch-tree.el
@@ -321,10 +321,10 @@ then NAME behaves like CMD."
;; These bindings shadow common bindings with variants
;; that additionally close the message window.
(define-key map [remap notmuch-bury-or-kill-this-buffer] 'notmuch-tree-quit)
- (define-key map [remap notmuch-search] 'notmuch-tree-to-search)
- (define-key map [remap notmuch-help] 'notmuch-tree-help)
- (define-key map [remap notmuch-mua-new-mail] 'notmuch-tree-new-mail)
- (define-key map [remap notmuch-jump-search] 'notmuch-tree-jump-search)
+ (define-key map [remap notmuch-search] 'notmuch-tree-to-search)
+ (define-key map [remap notmuch-help] 'notmuch-tree-help)
+ (define-key map [remap notmuch-mua-new-mail] 'notmuch-tree-new-mail)
+ (define-key map [remap notmuch-jump-search] 'notmuch-tree-jump-search)
(define-key map "o" 'notmuch-tree-toggle-order)
(define-key map "S" 'notmuch-search-from-tree-current-query)
@@ -350,6 +350,7 @@ then NAME behaves like CMD."
(define-key map "R" 'notmuch-tree-reply)
(define-key map "V" 'notmuch-tree-view-raw-message)
(define-key map "l" 'notmuch-tree-filter)
+ (define-key map "t" 'notmuch-tree-filter-by-tag)
;; The main tree view bindings
(define-key map (kbd "RET") 'notmuch-tree-show-message)
@@ -1184,6 +1185,25 @@ current search results AND the additional query string provided."
grouped-query
(concat grouped-original-query " and " grouped-query)))))
+(defun notmuch-tree-filter-by-tag (tag)
+ "Filter the current search results based on a single TAG.
+
+Run a new search matching only messages that match the current
+search results and that are also tagged with the given TAG."
+ (interactive
+ (list (notmuch-select-tag-with-completion "Filter by tag: "
+ notmuch-tree-basic-query)))
+ (let ((notmuch-show-process-crypto (notmuch-tree--message-process-crypto)))
+ (notmuch-tree-close-message-window)
+ (notmuch-tree (concat notmuch-tree-basic-query " and tag:" tag)
+ notmuch-tree-query-context
+ nil
+ nil
+ nil
+ notmuch-tree-unthreaded
+ nil
+ notmuch-search-oldest-first)))
+
;;; _
(provide 'notmuch-tree)
--
2.32.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v3 2/2] emacs: new command notmuch-tree-filter-by-tag
2021-08-04 0:23 ` [PATCH v3 2/2] emacs: new command notmuch-tree-filter-by-tag jao
@ 2021-08-21 14:35 ` David Bremner
0 siblings, 0 replies; 5+ messages in thread
From: David Bremner @ 2021-08-21 14:35 UTC (permalink / raw)
To: jao, notmuch; +Cc: jao
jao <jao@gnu.org> writes:
> ---
> doc/notmuch-emacs.rst | 4 ++++
> emacs/notmuch-tree.el | 28 ++++++++++++++++++++++++----
> 2 files changed, 28 insertions(+), 4 deletions(-)
>
> diff --git a/doc/notmuch-emacs.rst b/doc/notmuch-emacs.rst
> index 5accfa60..36dcb116 100644
> --- a/doc/notmuch-emacs.rst
> +++ b/doc/notmuch-emacs.rst
> @@ -305,6 +305,10 @@ tags.
> ``l`` ``notmuch-tree-filter``
> Filter or LIMIT the current search results based on an additional query string
>
> +``t`` ``notmuch-tree-filter-by-tag``
> + Filter the current search results based on an additional tag
> +
Here again devel/emacs-keybindings.org needs to be updated, and the
commit message needs a brief explanation about the replacement of the
function bound to 't'.
d
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-08-21 14:36 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-04 0:23 [EMACS] New commands notmuch-tree-filter{,-by-tag} jao
2021-08-04 0:23 ` [PATCH v3 1/2] emacs: new command notmuch-tree-filter jao
2021-08-21 14:32 ` David Bremner
2021-08-04 0:23 ` [PATCH v3 2/2] emacs: new command notmuch-tree-filter-by-tag jao
2021-08-21 14:35 ` 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).