* [PATCH -V2] notmuch.el: Support for customizing search result display
@ 2010-01-23 10:09 Aneesh Kumar K.V
2010-03-10 16:33 ` [PATCH -V3] " Aneesh Kumar K.V
0 siblings, 1 reply; 4+ messages in thread
From: Aneesh Kumar K.V @ 2010-01-23 10:09 UTC (permalink / raw)
To: cworth; +Cc: Aneesh Kumar K.V, notmuch
From: Aneesh Kumar K.V <aneesh.kumar@gmail.com>
This patch helps in customizing search result display
similar to mutt's index_format. The customization is done
by defining an alist as below
(setq notmuch-search-result-format '(("date" . "%s ")
("authors" . "%-40s ")
("subject" . "%s ")
("tags" . "(%s)")))
The supported keywords are date, count, authors, subject and tags.
Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@gmail.com>
---
notmuch.el | 56 +++++++++++++++++++++++++++++++++++++++-----------------
1 files changed, 39 insertions(+), 17 deletions(-)
diff --git a/notmuch.el b/notmuch.el
index 97914f2..89dc32a 100644
--- a/notmuch.el
+++ b/notmuch.el
@@ -119,6 +119,14 @@ pattern can still test against the entire line).")
(defvar notmuch-show-body-read-visible nil)
(defvar notmuch-show-citations-visible nil)
(defvar notmuch-show-signatures-visible nil)
+(defcustom notmuch-search-result-format nil
+ "Search result formating. Supported fields are
+ date, count, authors, subject, tags
+ex: (setq notmuch-search-result-format \(\(\"authors\" . \"%-40s\"\)
+ \(\"subject\" . \"%s\"\)\)\)"
+:type '(alist :key-type (string) :value-type (string))
+:group 'notmuch)
+
(defvar notmuch-show-headers-visible nil)
; XXX: This should be a generic function in emacs somewhere, not here
@@ -1065,11 +1073,6 @@ matching this search term are shown if non-nil. "
"Notmuch search mode face used to highligh tags."
:group 'notmuch)
-(defvar notmuch-tag-face-alist nil
- "List containing the tag list that need to be highlighed")
-
-(defvar notmuch-search-font-lock-keywords nil)
-
;;;###autoload
(defun notmuch-search-mode ()
"Major mode displaying results of a notmuch search.
@@ -1105,17 +1108,7 @@ Complete list of currently available key bindings:
(setq truncate-lines t)
(setq major-mode 'notmuch-search-mode
mode-name "notmuch-search")
- (setq buffer-read-only t)
- (if (not notmuch-tag-face-alist)
- (add-to-list 'notmuch-search-font-lock-keywords (list
- "(\\([^)]*\\))$" '(1 'notmuch-tag-face)))
- (let ((notmuch-search-tags (mapcar 'car notmuch-tag-face-alist)))
- (loop for notmuch-search-tag in notmuch-search-tags
- do (add-to-list 'notmuch-search-font-lock-keywords (list
- (concat "([^)]*\\(" notmuch-search-tag "\\)[^)]*)$")
- `(1 ,(cdr (assoc notmuch-search-tag notmuch-tag-face-alist))))))))
- (set (make-local-variable 'font-lock-defaults)
- '(notmuch-search-font-lock-keywords t)))
+ (setq buffer-read-only t))
(defun notmuch-search-find-thread-id ()
"Return the thread for the current thread"
@@ -1230,6 +1223,30 @@ This function advances the next thread when finished."
(insert (format " (process returned %d)" exit-status)))
(insert "\n"))))))))))
+(defun insert-tags (tags)
+ (insert (concat "(" (propertize tags
+ 'font-lock-face 'notmuch-tag-face) ")")))
+
+(defun insert-field (field date count authors subject tags)
+(if (string-equal field "date")
+ (insert (format (cdr (assoc field notmuch-search-result-format)) date))
+ (if (string-equal field "count")
+ (insert (format (cdr (assoc field notmuch-search-result-format)) count))
+ (if (string-equal field "authors")
+ (insert (format (cdr (assoc field notmuch-search-result-format)) authors))
+ (if (string-equal field "subject")
+ (insert (format (cdr (assoc field notmuch-search-result-format)) subject))
+ (if (string-equal field "tags")
+ (insert-tags (format (cdr (assoc field notmuch-search-result-format)) tags)))
+)))))
+
+(defun notmuch-search-show-result (date count authors subject tags)
+(let ((fields) (field))
+ (setq fields (mapcar 'car notmuch-search-result-format))
+ (loop for field in fields
+ do (insert-field field date count authors subject tags)))
+(insert "\n"))
+
(defun notmuch-search-process-filter (proc string)
"Process and filter the output of \"notmuch search\""
(let ((buffer (process-buffer proc)))
@@ -1252,7 +1269,12 @@ This function advances the next thread when finished."
(set 'authors (concat (substring authors 0 (- 40 3)) "...")))
(goto-char (point-max))
(let ((beg (point-marker)))
- (insert (format "%s %-7s %-40s %s (%s)\n" date count authors subject tags))
+ (if (not notmuch-search-result-format)
+ (progn (insert (format "%s %-7s %-40s %s" date count authors subject))
+ ;; insert the fontified tag
+ (insert-tags (format "%s" tags))
+ (insert "\n"))
+ (notmuch-search-show-result date count authors subject tags))
(put-text-property beg (point-marker) 'notmuch-search-thread-id thread-id)
(put-text-property beg (point-marker) 'notmuch-search-authors authors)
(put-text-property beg (point-marker) 'notmuch-search-subject subject))
--
1.6.6.1.394.gdedc0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH -V3] notmuch.el: Support for customizing search result display
2010-01-23 10:09 [PATCH -V2] notmuch.el: Support for customizing search result display Aneesh Kumar K.V
@ 2010-03-10 16:33 ` Aneesh Kumar K.V
0 siblings, 0 replies; 4+ messages in thread
From: Aneesh Kumar K.V @ 2010-03-10 16:33 UTC (permalink / raw)
To: cworth; +Cc: Aneesh Kumar K.V, notmuch
From: Aneesh Kumar K.V <aneesh.kumar@gmail.com>
This patch helps in customizing search result display
similar to mutt's index_format. The customization is done
by defining an alist as below
(setq notmuch-search-result-format '(("date" . "%s ")
("authors" . "%-40s ")
("subject" . "%s ")))
The supported keywords are date, count, authors, subject.
Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@gmail.com>
---
emacs/notmuch.el | 55 +++++++++++++++++++++++++++++++++++++----------------
1 files changed, 38 insertions(+), 17 deletions(-)
diff --git a/emacs/notmuch.el b/emacs/notmuch.el
index 5b553bb..cc5167f 100644
--- a/emacs/notmuch.el
+++ b/emacs/notmuch.el
@@ -157,6 +157,18 @@ remaining lines into a button.")
(defvar notmuch-show-body-read-visible nil)
(defvar notmuch-show-citations-visible nil)
(defvar notmuch-show-signatures-visible nil)
+(defcustom notmuch-search-result-format
+ '(("date" . "%s")
+ ("count" . "%-7s")
+ ("authors" . "%-40s")
+ ("subject" . "%s"))
+ "Search result formating. Supported fields are
+ date, count, authors, subject
+ex: (setq notmuch-search-result-format \(\(\"authors\" . \"%-40s\"\)
+ \(\"subject\" . \"%s\"\)\)\)"
+:type '(alist :key-type (string) :value-type (string))
+:group 'notmuch)
+
(defvar notmuch-show-headers-visible nil)
; XXX: This should be a generic function in emacs somewhere, not here
@@ -1314,11 +1326,6 @@ matching this search term are shown if non-nil. "
"Notmuch search mode face used to highligh tags."
:group 'notmuch)
-(defvar notmuch-tag-face-alist nil
- "List containing the tag list that need to be highlighed")
-
-(defvar notmuch-search-font-lock-keywords nil)
-
;;;###autoload
(defun notmuch-search-mode ()
"Major mode displaying results of a notmuch search.
@@ -1356,17 +1363,7 @@ Complete list of currently available key bindings:
(setq truncate-lines t)
(setq major-mode 'notmuch-search-mode
mode-name "notmuch-search")
- (setq buffer-read-only t)
- (if (not notmuch-tag-face-alist)
- (add-to-list 'notmuch-search-font-lock-keywords (list
- "(\\([^)]*\\))$" '(1 'notmuch-tag-face)))
- (let ((notmuch-search-tags (mapcar 'car notmuch-tag-face-alist)))
- (loop for notmuch-search-tag in notmuch-search-tags
- do (add-to-list 'notmuch-search-font-lock-keywords (list
- (concat "([^)]*\\(" notmuch-search-tag "\\)[^)]*)$")
- `(1 ,(cdr (assoc notmuch-search-tag notmuch-tag-face-alist))))))))
- (set (make-local-variable 'font-lock-defaults)
- '(notmuch-search-font-lock-keywords t)))
+ (setq buffer-read-only t))
(defun notmuch-search-find-thread-id ()
"Return the thread for the current thread"
@@ -1487,6 +1484,30 @@ This function advances the next thread when finished."
(if never-found-target-thread
(goto-char notmuch-search-target-position)))))))
+(defun insert-tags (tags)
+ (insert (concat "(" (propertize tags
+ 'font-lock-face 'notmuch-tag-face) ")")))
+
+(defun insert-field (field date count authors subject tags)
+(if (string-equal field "date")
+ (insert (format (cdr (assoc field notmuch-search-result-format)) date))
+ (if (string-equal field "count")
+ (insert (format (cdr (assoc field notmuch-search-result-format)) count))
+ (if (string-equal field "authors")
+ (insert (format (cdr (assoc field notmuch-search-result-format)) authors))
+ (if (string-equal field "subject")
+ (insert (format (cdr (assoc field notmuch-search-result-format)) subject))
+ (if (string-equal field "tags")
+ (insert-tags (format "%s" tags)))
+)))))
+
+(defun notmuch-search-show-result (date count authors subject tags)
+(let ((fields) (field))
+ (setq fields (mapcar 'car notmuch-search-result-format))
+ (loop for field in fields
+ do (insert-field field date count authors subject tags)))
+(insert "\n"))
+
(defun notmuch-search-process-filter (proc string)
"Process and filter the output of \"notmuch search\""
(let ((buffer (process-buffer proc))
@@ -1510,7 +1531,7 @@ This function advances the next thread when finished."
(set 'authors (concat (substring authors 0 (- 40 3)) "...")))
(goto-char (point-max))
(let ((beg (point-marker)))
- (insert (format "%s %-7s %-40s %s (%s)\n" date count authors subject tags))
+ (notmuch-search-show-result date count authors subject tags)
(put-text-property beg (point-marker) 'notmuch-search-thread-id thread-id)
(put-text-property beg (point-marker) 'notmuch-search-authors authors)
(put-text-property beg (point-marker) 'notmuch-search-subject subject)
--
1.7.0.2.157.gb7e7f
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH -v2] notmuch.el: Support for customizing search result display
@ 2010-02-05 19:55 Carl Worth
2010-02-11 16:41 ` [PATCH -V3] " Aneesh Kumar K.V
0 siblings, 1 reply; 4+ messages in thread
From: Carl Worth @ 2010-02-05 19:55 UTC (permalink / raw)
To: Aneesh Kumar K.V, aneesh.kumar; +Cc: Aneesh Kumar K.V, notmuch
[-- Attachment #1: Type: text/plain, Size: 2064 bytes --]
On Wed, 2 Dec 2009 18:19:38 +0530, "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com> wrote:
> From: Aneesh Kumar K.V <aneesh.kumar@gmail.com>
>
> This patch helps in customizing search result display
> similar to mutt's index_format. The customization is done
> by defining an alist as below
>
> (setq notmuch-search-result-format '(("date" . "%s ")
> ("authors" . "%-40s ")
> ("subject" . "%s ")
> ("tags" . "(%s)")))
>
> The supported keywords are date, count, authors, subject and tags.
>
> Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@gmail.com>
Hi Aneesh,
I'm sorry this patch has lingered so long without comment. There's
really only one problem I see with it:
> +(defcustom notmuch-search-result-format nil
> + "Search result formating. Supported fields are
> + date, count, authors, subject, tags
> +ex: (setq notmuch-search-result-format \(\(\"authors\" . \"%-40s\"\)
> + \(\"subject\" . \"%s\"\)\)\)"
> +:type '(alist :key-type (string) :value-type (string))
> +:group 'notmuch)
...
> - (insert (format "%s %-7s %-40s %s (%s)\n" date count authors subject tags))
> + (if (not notmuch-search-result-format)
> + (progn (insert (format "%s %-7s %-40s %s" date count authors subject))
> + ;; insert the fontified tag
> + (insert-tags (format "%s" tags))
> + (insert "\n"))
> + (notmuch-search-show-result date count authors subject tags))
I don't like that the new format variable is nil by default and then
there's an open-coded implementation of the default formatting. This has
a couple of problems:
1. The new code is not being exercised by default, so it would be easy
to break it without realizing.
2. The system is not very self-documenting.
If a new user wants to tweak the default format, that will be a lot
easier if they find a customizable variable that describes the
current format. *That* will be a lot easier to modify rather than
trying to learn what should be used instead of "nil".
-Carl
[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH -V3] notmuch.el: Support for customizing search result display
2010-02-05 19:55 [PATCH -v2] " Carl Worth
@ 2010-02-11 16:41 ` Aneesh Kumar K.V
2010-02-11 17:17 ` Aneesh Kumar K. V
0 siblings, 1 reply; 4+ messages in thread
From: Aneesh Kumar K.V @ 2010-02-11 16:41 UTC (permalink / raw)
To: cworth; +Cc: Aneesh Kumar K.V, notmuch
From: Aneesh Kumar K.V <aneesh.kumar@gmail.com>
This patch helps in customizing search result display
similar to mutt's index_format. The customization is done
by defining an alist as below
(setq notmuch-search-result-format '(("date" . "%s ")
("authors" . "%-40s ")
("subject" . "%s ")
("tags" . "(%s)")))
The supported keywords are date, count, authors, subject and tags.
Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@gmail.com>
---
notmuch.el | 56 +++++++++++++++++++++++++++++++++++++++-----------------
1 files changed, 39 insertions(+), 17 deletions(-)
diff --git a/notmuch.el b/notmuch.el
index 040997e..a4a89ac 100644
--- a/notmuch.el
+++ b/notmuch.el
@@ -141,6 +141,19 @@ remaining lines into a button.")
(defvar notmuch-show-body-read-visible nil)
(defvar notmuch-show-citations-visible nil)
(defvar notmuch-show-signatures-visible nil)
+(defcustom notmuch-search-result-format
+ '(("date" . "%s")
+ ("count" . "%-7s")
+ ("authors" . "%-40s")
+ ("subject" . "%s")
+ ("tags" . "%s"))
+ "Search result formating. Supported fields are
+ date, count, authors, subject, tags
+ex: (setq notmuch-search-result-format \(\(\"authors\" . \"%-40s\"\)
+ \(\"subject\" . \"%s\"\)\)\)"
+:type '(alist :key-type (string) :value-type (string))
+:group 'notmuch)
+
(defvar notmuch-show-headers-visible nil)
; XXX: This should be a generic function in emacs somewhere, not here
@@ -1175,11 +1188,6 @@ matching this search term are shown if non-nil. "
"Notmuch search mode face used to highligh tags."
:group 'notmuch)
-(defvar notmuch-tag-face-alist nil
- "List containing the tag list that need to be highlighed")
-
-(defvar notmuch-search-font-lock-keywords nil)
-
;;;###autoload
(defun notmuch-search-mode ()
"Major mode displaying results of a notmuch search.
@@ -1215,17 +1223,7 @@ Complete list of currently available key bindings:
(setq truncate-lines t)
(setq major-mode 'notmuch-search-mode
mode-name "notmuch-search")
- (setq buffer-read-only t)
- (if (not notmuch-tag-face-alist)
- (add-to-list 'notmuch-search-font-lock-keywords (list
- "(\\([^)]*\\))$" '(1 'notmuch-tag-face)))
- (let ((notmuch-search-tags (mapcar 'car notmuch-tag-face-alist)))
- (loop for notmuch-search-tag in notmuch-search-tags
- do (add-to-list 'notmuch-search-font-lock-keywords (list
- (concat "([^)]*\\(" notmuch-search-tag "\\)[^)]*)$")
- `(1 ,(cdr (assoc notmuch-search-tag notmuch-tag-face-alist))))))))
- (set (make-local-variable 'font-lock-defaults)
- '(notmuch-search-font-lock-keywords t)))
+ (setq buffer-read-only t))
(defun notmuch-search-find-thread-id ()
"Return the thread for the current thread"
@@ -1340,6 +1338,30 @@ This function advances the next thread when finished."
(insert (format " (process returned %d)" exit-status)))
(insert "\n"))))))))))
+(defun insert-tags (tags)
+ (insert (concat "(" (propertize tags
+ 'font-lock-face 'notmuch-tag-face) ")")))
+
+(defun insert-field (field date count authors subject tags)
+(if (string-equal field "date")
+ (insert (format (cdr (assoc field notmuch-search-result-format)) date))
+ (if (string-equal field "count")
+ (insert (format (cdr (assoc field notmuch-search-result-format)) count))
+ (if (string-equal field "authors")
+ (insert (format (cdr (assoc field notmuch-search-result-format)) authors))
+ (if (string-equal field "subject")
+ (insert (format (cdr (assoc field notmuch-search-result-format)) subject))
+ (if (string-equal field "tags")
+ (insert-tags (format (cdr (assoc field notmuch-search-result-format)) tags)))
+)))))
+
+(defun notmuch-search-show-result (date count authors subject tags)
+(let ((fields) (field))
+ (setq fields (mapcar 'car notmuch-search-result-format))
+ (loop for field in fields
+ do (insert-field field date count authors subject tags)))
+(insert "\n"))
+
(defun notmuch-search-process-filter (proc string)
"Process and filter the output of \"notmuch search\""
(let ((buffer (process-buffer proc)))
@@ -1362,7 +1384,7 @@ This function advances the next thread when finished."
(set 'authors (concat (substring authors 0 (- 40 3)) "...")))
(goto-char (point-max))
(let ((beg (point-marker)))
- (insert (format "%s %-7s %-40s %s (%s)\n" date count authors subject tags))
+ (notmuch-search-show-result date count authors subject tags)
(put-text-property beg (point-marker) 'notmuch-search-thread-id thread-id)
(put-text-property beg (point-marker) 'notmuch-search-authors authors)
(put-text-property beg (point-marker) 'notmuch-search-subject subject))
--
1.7.0.rc2
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH -V3] notmuch.el: Support for customizing search result display
2010-02-11 16:41 ` [PATCH -V3] " Aneesh Kumar K.V
@ 2010-02-11 17:17 ` Aneesh Kumar K. V
0 siblings, 0 replies; 4+ messages in thread
From: Aneesh Kumar K. V @ 2010-02-11 17:17 UTC (permalink / raw)
To: cworth; +Cc: Aneesh Kumar K.V, notmuch
On Thu, 11 Feb 2010 22:11:46 +0530, "Aneesh Kumar K.V" <aneesh.kumar@linux.vnet.ibm.com> wrote:
> From: Aneesh Kumar K.V <aneesh.kumar@gmail.com>
>
> This patch helps in customizing search result display
> similar to mutt's index_format. The customization is done
> by defining an alist as below
>
> (setq notmuch-search-result-format '(("date" . "%s ")
> ("authors" . "%-40s ")
> ("subject" . "%s ")
> ("tags" . "(%s)")))
>
> The supported keywords are date, count, authors, subject and tags.
>
> Signed-off-by: Aneesh Kumar K.V <aneesh.kumar@gmail.com>
> ---
> notmuch.el | 56 +++++++++++++++++++++++++++++++++++++++-----------------
> 1 files changed, 39 insertions(+), 17 deletions(-)
>
> diff --git a/notmuch.el b/notmuch.el
> index 040997e..a4a89ac 100644
> --- a/notmuch.el
> +++ b/notmuch.el
> @@ -141,6 +141,19 @@ remaining lines into a button.")
> (defvar notmuch-show-body-read-visible nil)
> (defvar notmuch-show-citations-visible nil)
> (defvar notmuch-show-signatures-visible nil)
> +(defcustom notmuch-search-result-format
> + '(("date" . "%s")
> + ("count" . "%-7s")
> + ("authors" . "%-40s")
> + ("subject" . "%s")
> + ("tags" . "%s"))
I missed a '(' around tags. So the last line should be
("tags" . "(%s)"))
-aneesh
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2010-03-10 16:33 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-01-23 10:09 [PATCH -V2] notmuch.el: Support for customizing search result display Aneesh Kumar K.V
2010-03-10 16:33 ` [PATCH -V3] " Aneesh Kumar K.V
-- strict thread matches above, loose matches on Subject: below --
2010-02-05 19:55 [PATCH -v2] " Carl Worth
2010-02-11 16:41 ` [PATCH -V3] " Aneesh Kumar K.V
2010-02-11 17:17 ` Aneesh Kumar K. V
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).