Hi On Sun, 06 Apr 2014, Austin Clements <amdragon at MIT.EDU> wrote: > Quoth Mark Walters on Apr 05 at 10:24 pm: >> This adds a sort-order option to saved-searches, stores it in the >> saved-search buttons (widgets), and uses the stored value when the >> button is pressed. >> >> Storing the sort-order in the widget was suggested by Jani in >> id:4c3876274126985683e888641b29cf18142a5eb8.1391771337.git.jani at >> nikula.org. >> --- >> emacs/notmuch-hello.el | 11 ++++++++++- >> emacs/notmuch-lib.el | 7 ++++++- >> 2 files changed, 16 insertions(+), 2 deletions(-) >> >> diff --git a/emacs/notmuch-hello.el b/emacs/notmuch-hello.el >> index aa40e6f..6a28372 100644 >> --- a/emacs/notmuch-hello.el >> +++ b/emacs/notmuch-hello.el >> @@ -364,7 +364,8 @@ (defun notmuch-hello-reflect (list ncols) >> (defun notmuch-hello-widget-search (widget &rest ignore) >> (notmuch-search (widget-get widget >> :notmuch-search-terms) >> - notmuch-search-oldest-first)) >> + (widget-get widget >> + :notmuch-search-oldest-first))) >> >> (defun notmuch-saved-search-count (search) >> (car (process-lines notmuch-command "count" search))) >> @@ -496,12 +497,20 @@ (defun notmuch-hello-insert-buttons (searches) >> (widget-insert (make-string column-indent ? ))) >> (let* ((name (plist-get elem :name)) >> (query (plist-get elem :query)) >> + (oldest-first (cond >> + ((eq (plist-get elem :sort-order) >> 'newest-first) >> + nil) >> + ((eq (plist-get elem :sort-order) >> 'oldest-first) >> + t) >> + (t >> + notmuch-search-oldest-first))) > > (case (plist-get elem :sort-order) > (newest-first nil) > (oldest-first t) > (otherwise notmuch-search-oldest-first))
This is much better. > >> (msg-count (plist-get elem :count))) >> (widget-insert (format "%8s " >> (notmuch-hello-nice-number msg-count))) >> (widget-create 'push-button >> :notify #'notmuch-hello-widget-search >> :notmuch-search-terms query >> + :notmuch-search-oldest-first oldest-first >> name) >> (setq column-indent >> (1+ (max 0 (- column-width (length name))))))) >> diff --git a/emacs/notmuch-lib.el b/emacs/notmuch-lib.el >> index 8a12f91..8aa8cfc 100644 >> --- a/emacs/notmuch-lib.el >> +++ b/emacs/notmuch-lib.el >> @@ -125,7 +125,12 @@ (define-widget 'notmuch-saved-search-plist 'list >> (group :format "%v" :inline t (const :format " Query: " >> :query) (string :format "%v"))) >> (checklist :inline t >> :format "%v" >> - (group :format "%v" :inline t (const :format "Count-Query: >> " :count-query) (string :format "%v"))))) >> + (group :format "%v" :inline t (const :format "Count-Query: >> " :count-query) (string :format "%v")) >> + (group :format "%v" :inline t (const :format "" >> :sort-order) >> + (choice :tag " Sort Order" > > Should there be a colon? (I haven't applied the patches and I'm > afraid this is beyond my mental implementation of define-widget!) I think it is OK without: since this is using :tag rather than :format the colon is supplied automatically. Many thanks Mark > >> + (const :tag "Default" nil) >> + (const :tag "Oldest-first" oldest-first) >> + (const :tag "Newest-first" >> newest-first)))))) >> >> (defcustom notmuch-saved-searches '((:name "inbox" :query "tag:inbox") >> (:name "unread" :query "tag:unread"))