* todos.el: Fixes and improvements related to item relocation.
[bpt/emacs.git] / lisp / calendar / todos.el
index b956e78..30c64ef 100644 (file)
@@ -79,7 +79,6 @@ This lacks the extension and directory components."
                                  (funcall todos-files-function))))
   :group 'todos)
 
-;; FIXME: is there a better alternative to this?
 (defun todos-reevaluate-default-file-defcustom ()
   "Reevaluate defcustom of `todos-default-todos-file'.
 Called after adding or deleting a Todos file."
@@ -112,7 +111,6 @@ Otherwise, `todos-show' always visits `todos-default-todos-file'."
                                (funcall todos-files-function))))
   :group 'todos)
 
-;; FIXME: is there a better alternative to this?
 (defun todos-reevaluate-category-completions-files-defcustom ()
   "Reevaluate defcustom of `todos-category-completions-files'.
 Called after adding or deleting a Todos file."
@@ -141,9 +139,13 @@ displayed correctly."
   :type 'string
   :group 'todos)
 
-(defcustom todos-display-categories-first nil
-  "Non-nil to display category list on first visit to a Todos file."
-  :type 'boolean
+(defcustom todos-show-first 'first
+  "What action to take on first use of `todos-show' on a file."
+  :type '(choice (const :tag "Show first category" first)
+                (const :tag "Show table of categories" table)
+                (const :tag "Show top priorities" top)
+                (const :tag "Show diary items" diary)
+                (const :tag "Show regexp items" regexp))
   :group 'todos)
 
 (defcustom todos-completion-ignore-case nil
@@ -190,7 +192,13 @@ todo-mode.el."
 
 (defcustom todos-prefix ""
   "String prefixed to todo items for visual distinction."
-  :type 'string
+  :type '(string :validate
+                (lambda (widget)
+                  (when (string= (widget-value widget) todos-item-mark)
+                    (widget-put
+                     widget :error
+                     "Invalid value: must be distinct from `todos-item-mark'")
+                    widget)))
   :initialize 'custom-initialize-default
   :set 'todos-reset-prefix
   :group 'todos-mode-display)
@@ -206,23 +214,30 @@ These reflect the priorities of the items in each category."
 (defun todos-reset-prefix (symbol value)
   "The :set function for `todos-prefix' and `todos-number-priorities'."
   (let ((oldvalue (symbol-value symbol))
-       (files (append todos-files todos-archives)))
+       (files todos-file-buffers))
     (custom-set-default symbol value)
     (when (not (equal value oldvalue))
       (dolist (f files)
        (with-current-buffer (find-file-noselect f)
-         (save-window-excursion
-           (todos-show)
-           (save-excursion
-             (widen)
-             (goto-char (point-min))
-             (while (not (eobp))
-               (remove-overlays (point) (point)); 'before-string prefix)
-               (forward-line)))
-           ;; Activate the new setting (save-restriction does not help).
-           (save-excursion (todos-category-select))))))))
+         ;; Activate the new setting in the current category.
+         (save-excursion (todos-category-select)))))))
+
+(defcustom todos-item-mark "*"
+  "String used to mark items.
+To ensure item marking works, change the value of this option
+only when no items are marked."
+  :type '(string :validate
+                (lambda (widget)
+                  (when (string= (widget-value widget) todos-prefix)
+                    (widget-put
+                     widget :error
+                     "Invalid value: must be distinct from `todos-prefix'")
+                    widget)))
+  :set (lambda (symbol value)
+        (custom-set-default symbol (propertize value 'face 'todos-mark)))
+  :group 'todos-mode-display)
 
-(defcustom todos-done-separator-string "_"
+(defcustom todos-done-separator-string "="
   "String for generating `todos-done-separator'.
 
 If the string consists of a single character,
@@ -242,9 +257,14 @@ the value of `todos-done-separator'."
        (files todos-file-buffers)
        (sep todos-done-separator))
     (custom-set-default symbol value)
-    (setq todos-done-separator (todos-done-separator))
-    (when (= 1 (length value))
-      (todos-reset-done-separator sep))))
+    (when (not (equal value oldvalue))
+      (dolist (f files)
+       (with-current-buffer (find-file-noselect f)
+         (let (buffer-read-only)
+           (setq todos-done-separator (todos-done-separator))
+           (when (= 1 (length value))
+             (todos-reset-done-separator sep)))
+         (todos-category-select))))))
 
 (defcustom todos-done-string "DONE "
   "Identifying string appended to the front of done todos items."
@@ -379,8 +399,6 @@ The amount of indentation is given by user option
   (unless (member '(continuation) fringe-indicator-alist)
     (push '(continuation) fringe-indicator-alist)))
 
-;; FIXME: :set function to refill items with hard newlines and to immediately
-;; update wrapped prefix display
 (defcustom todos-indent-to-here 6
   "Number of spaces `todos-line-wrapping-function' indents to."
   :type '(integer :validate
@@ -480,26 +498,6 @@ current time, if nil, they include it."
   :version "24.2"
   :group 'todos)
 
-(defcustom todos-filtered-items-buffer "Todos filtered items"
-  "Initial name of buffer in Todos Filter Items mode."
-  :type 'string
-  :group 'todos-filtered)
-
-(defcustom todos-top-priorities-buffer "Todos top priorities"
-  "Buffer type string for `todos-filtered-buffer-name'."
-  :type 'string
-  :group 'todos-filtered)
-
-(defcustom todos-diary-items-buffer "Todos diary items"
-  "Buffer type string for `todos-filtered-buffer-name'."
-  :type 'string
-  :group 'todos-filtered)
-
-(defcustom todos-regexp-items-buffer "Todos regexp items"
-  "Buffer type string for `todos-filtered-buffer-name'."
-  :type 'string
-  :group 'todos-filtered)
-
 (defcustom todos-priorities-rules nil
   "List of rules giving how many items `todos-top-priorities' shows.
 This variable should be set interactively by
@@ -515,6 +513,8 @@ items in that category, which overrides NUM."
   :type 'sexp
   :group 'todos-filtered)
 
+;; FIXME: rename to todos-top-priorities AFTER renaming command
+;; todos-top-priorities to todos-filter-top-priorities
 (defcustom todos-show-priorities 1
   "Default number of top priorities shown by `todos-top-priorities'."
   :type 'integer
@@ -527,7 +527,6 @@ items in that category, which overrides NUM."
                                (funcall todos-files-function))))
   :group 'todos-filtered)
 
-;; FIXME: is there a better alternative to this?
 (defun todos-reevaluate-filter-files-defcustom ()
   "Reevaluate defcustom of `todos-filter-files'.
 Called after adding or deleting a Todos file."
@@ -595,7 +594,7 @@ categories display according to priority."
   :group 'todos-categories)
 
 ;; ---------------------------------------------------------------------------
-;;; Faces and font-lock matcher functions
+;;; Faces and font locking
 
 (defgroup todos-faces nil
   "Faces for the Todos modes."
@@ -618,24 +617,17 @@ categories display according to priority."
   :group 'todos-faces)
 
 (defface todos-top-priority
-  ;; '((t :inherit font-lock-comment-face))
-  '((((class grayscale) (background light))
-     :foreground "DimGray" :weight bold :slant italic)
-    (((class grayscale) (background dark))
-     :foreground "LightGray" :weight bold :slant italic)
-    (((class color) (min-colors 88) (background light))
-     :foreground "Firebrick" :weight bold)
-    (((class color) (min-colors 88) (background dark))
-     :foreground "chocolate1" :weight bold)
-    (((class color) (min-colors 16) (background light))
-     :foreground "red" :weight bold)
-    (((class color) (min-colors 16) (background dark))
-     :foreground "red1" :weight bold)
-    (((class color) (min-colors 8) (background light))
-     :foreground "red" :weight bold)
-    (((class color) (min-colors 8) (background dark))
-     :foreground "yellow" :weight bold)
-    (t :weight bold :slant italic))
+  ;; bold font-lock-comment-face
+  '((default :weight bold)
+    (((class grayscale) (background light)) :foreground "DimGray" :slant italic)
+    (((class grayscale) (background dark)) :foreground "LightGray" :slant italic)
+    (((class color) (min-colors 88) (background light)) :foreground "Firebrick")
+    (((class color) (min-colors 88) (background dark)) :foreground "chocolate1")
+    (((class color) (min-colors 16) (background light)) :foreground "red")
+    (((class color) (min-colors 16) (background dark)) :foreground "red1")
+    (((class color) (min-colors 8) (background light)) :foreground "red")
+    (((class color) (min-colors 8) (background dark)) :foreground "yellow")
+    (t :slant italic))
   "Face for top priority Todos item numerical priority string.
 The item's priority number string has this face if the number is
 less than or equal the category's top priority setting."
@@ -736,14 +728,19 @@ less than or equal the category's top priority setting."
   :group 'todos-faces)
 
 (defface todos-diary-expired
-  ;; '((t :inherit font-lock-warning-face))
-  '((((class color)
-      (min-colors 16))
-     (:weight bold :foreground "DarkOrange"))
-    (((class color))
-     (:weight bold :foreground "yellow"))
-    (t
-     (:weight bold)))
+  ;; Doesn't contrast enough with todos-date (= diary) face.
+  ;; ;; '((t :inherit warning))
+  ;; '((default :weight bold)
+  ;;   (((class color) (min-colors 16)) :foreground "DarkOrange")
+  ;;   (((class color)) :foreground "yellow"))
+  ;; bold font-lock-function-name-face
+  '((default :weight bold)
+    (((class color) (min-colors 88) (background light)) :foreground "Blue1")
+    (((class color) (min-colors 88) (background dark))  :foreground "LightSkyBlue")
+    (((class color) (min-colors 16) (background light)) :foreground "Blue")
+    (((class color) (min-colors 16) (background dark))  :foreground "LightSkyBlue")
+    (((class color) (min-colors 8)) :foreground "blue")
+    (t :inverse-video t))
   "Face for expired dates of diary items."
   :group 'todos-faces)
 (defvar todos-diary-expired-face 'todos-diary-expired)
@@ -760,7 +757,49 @@ less than or equal the category's top priority setting."
   :group 'todos-faces)
 (defvar todos-time-face 'todos-time)
 
+(defface todos-nondiary
+  ;; '((t :inherit font-lock-type-face))
+  '((((class grayscale) (background light)) :foreground "Gray90" :weight bold)
+    (((class grayscale) (background dark))  :foreground "DimGray" :weight bold)
+    (((class color) (min-colors 88) (background light)) :foreground "ForestGreen")
+    (((class color) (min-colors 88) (background dark))  :foreground "PaleGreen")
+    (((class color) (min-colors 16) (background light)) :foreground "ForestGreen")
+    (((class color) (min-colors 16) (background dark))  :foreground "PaleGreen")
+    (((class color) (min-colors 8)) :foreground "green")
+    (t :weight bold :underline t))
+  "Face for non-diary markers around todo item date/time header."
+  :group 'todos-faces)
+(defvar todos-nondiary-face 'todos-nondiary)
+
+(defface todos-category-string
+    ;; '((t :inherit font-lock-type-face))
+  '((((class grayscale) (background light)) :foreground "Gray90" :weight bold)
+    (((class grayscale) (background dark))  :foreground "DimGray" :weight bold)
+    (((class color) (min-colors 88) (background light)) :foreground "ForestGreen")
+    (((class color) (min-colors 88) (background dark))  :foreground "PaleGreen")
+    (((class color) (min-colors 16) (background light)) :foreground "ForestGreen")
+    (((class color) (min-colors 16) (background dark))  :foreground "PaleGreen")
+    (((class color) (min-colors 8)) :foreground "green")
+    (t :weight bold :underline t))
+  "Face for category file names in Todos Filtered Item."
+  :group 'todos-faces)
+(defvar todos-category-string-face 'todos-category-string)
+
 (defface todos-done
+  ;; '((t :inherit font-lock-keyword-face))
+  '((((class grayscale) (background light)) :foreground "LightGray" :weight bold)
+    (((class grayscale) (background dark))  :foreground "DimGray" :weight bold)
+    (((class color) (min-colors 88) (background light)) :foreground "Purple")
+    (((class color) (min-colors 88) (background dark))  :foreground "Cyan1")
+    (((class color) (min-colors 16) (background light)) :foreground "Purple")
+    (((class color) (min-colors 16) (background dark))  :foreground "Cyan")
+    (((class color) (min-colors 8)) :foreground "cyan" :weight bold)
+    (t :weight bold))
+  "Face for done Todos item header string."
+  :group 'todos-faces)
+(defvar todos-done-face 'todos-done)
+
+(defface todos-comment
   ;; '((t :inherit font-lock-comment-face))
   '((((class grayscale) (background light))
      :foreground "DimGray" :weight bold :slant italic)
@@ -779,34 +818,20 @@ less than or equal the category's top priority setting."
     (((class color) (min-colors 8) (background dark))
      :foreground "yellow")
     (t :weight bold :slant italic))
-  "Face for done Todos item header string."
-  :group 'todos-faces)
-(defvar todos-done-face 'todos-done)
-
-(defface todos-comment
-  ;; '((t :inherit font-lock-keyword-face))
-  '((((class grayscale) (background light)) :foreground "LightGray" :weight bold)
-    (((class grayscale) (background dark))  :foreground "DimGray" :weight bold)
-    (((class color) (min-colors 88) (background light)) :foreground "Purple")
-    (((class color) (min-colors 88) (background dark))  :foreground "Cyan1")
-    (((class color) (min-colors 16) (background light)) :foreground "Purple")
-    (((class color) (min-colors 16) (background dark))  :foreground "Cyan")
-    (((class color) (min-colors 8)) :foreground "cyan" :weight bold)
-    (t :weight bold))
   "Face for comments appended to done Todos items."
   :group 'todos-faces)
 (defvar todos-comment-face 'todos-comment)
 
 (defface todos-done-sep
-  ;; '((t :inherit font-lock-type-face))
-  '((((class grayscale) (background light)) :foreground "Gray90" :weight bold)
+  ;; '((t :inherit font-lock-builtin-face))
+  '((((class grayscale) (background light)) :foreground "LightGray" :weight bold)
     (((class grayscale) (background dark))  :foreground "DimGray" :weight bold)
-    (((class color) (min-colors 88) (background light)) :foreground "ForestGreen")
-    (((class color) (min-colors 88) (background dark))  :foreground "PaleGreen")
-    (((class color) (min-colors 16) (background light)) :foreground "ForestGreen")
-    (((class color) (min-colors 16) (background dark))  :foreground "PaleGreen")
-    (((class color) (min-colors 8)) :foreground "green")
-    (t :weight bold :underline t))
+    (((class color) (min-colors 88) (background light)) :foreground "dark slate blue")
+    (((class color) (min-colors 88) (background dark))  :foreground "LightSteelBlue")
+    (((class color) (min-colors 16) (background light)) :foreground "Orchid")
+    (((class color) (min-colors 16) (background dark)) :foreground "LightSteelBlue")
+    (((class color) (min-colors 8)) :foreground "blue" :weight bold)
+    (t :weight bold))
   "Face for separator string bewteen done and not done Todos items."
   :group 'todos-faces)
 (defvar todos-done-sep-face 'todos-done-sep)
@@ -871,8 +896,8 @@ less than or equal the category's top priority setting."
 
 (defun todos-category-string-matcher-1 (lim)
   "Search for Todos category name within LIM for font-locking.
-This is for fontifying category names appearing in Todos filter
-mode following done items."
+This is for fontifying category and file names appearing in Todos
+Filtered Items mode following done items."
   (if (eq major-mode 'todos-filtered-items-mode)
       (re-search-forward (concat todos-done-string-start todos-date-pattern
                                 "\\(?: " diary-time-regexp
@@ -884,8 +909,8 @@ mode following done items."
 
 (defun todos-category-string-matcher-2 (lim)
   "Search for Todos category name within LIM for font-locking.
-This is for fontifying category names appearing in Todos filter
-mode following todo (not done) items."
+This is for fontifying category and file names appearing in Todos
+Filtered Items mode following todo (not done) items."
   (if (eq major-mode 'todos-filtered-items-mode)
       (re-search-forward (concat todos-date-string-start todos-date-pattern
                                 "\\(?: " diary-time-regexp "\\)?\\(?:"
@@ -895,17 +920,16 @@ mode following todo (not done) items."
 
 (defvar todos-font-lock-keywords
   (list
-   '(todos-nondiary-marker-matcher 1 todos-done-sep-face t)
-   '(todos-nondiary-marker-matcher 2 todos-done-sep-face t)
-   ;; This is the face used by diary-lib.el.
+   '(todos-nondiary-marker-matcher 1 todos-nondiary-face t)
+   '(todos-nondiary-marker-matcher 2 todos-nondiary-face t)
+   ;; diary-lib.el uses font-lock-constant-face for diary-nonmarking-symbol.
    '(todos-diary-nonmarking-matcher 1 font-lock-constant-face t)
    '(todos-date-string-matcher 1 todos-date-face t)
    '(todos-time-string-matcher 1 todos-time-face t)
    '(todos-done-string-matcher 0 todos-done-face t)
    '(todos-comment-string-matcher 1 todos-comment-face t)
-   ;; '(todos-category-string-matcher 1 todos-done-sep-face t)
-   '(todos-category-string-matcher-1 1 todos-done-sep-face t t)
-   '(todos-category-string-matcher-2 1 todos-done-sep-face t t)
+   '(todos-category-string-matcher-1 1 todos-category-string-face t t)
+   '(todos-category-string-matcher-2 1 todos-category-string-face t t)
    '(todos-diary-expired-matcher 1 todos-diary-expired-face t)
    '(todos-diary-expired-matcher 2 todos-diary-expired-face t t)
    )
@@ -970,34 +994,18 @@ the Diary, of done items and of archived items.")
   "Variable holding the number of the current Todos category.
 Todos categories are numbered starting from 1.")
 
-(defvar todos-first-visit t
-  "Non-nil if first display of this file in the current session.
-See `todos-display-categories-first'.")
-
 (defvar todos-show-done-only nil
   "If non-nil display only done items in current category.
 Set by the command `todos-show-done-only' and used by
 `todos-category-select'.")
 
 (defun todos-reset-and-enable-done-separator ()
-  "Show resized catagory separator overlay after window size change.
+  "Show resized done items separator overlay after window change.
 Added to `window-configuration-change-hook' in `todos-mode'."
   (when (= 1 (length todos-done-separator-string))
     (let ((sep todos-done-separator))
       (setq todos-done-separator (todos-done-separator))
-      (save-match-data (todos-reset-done-separator sep)))
-    ;; FIXME: If this is called while the separator overlay is shown, the
-    ;; separator with deleted overlay becomes visible when waiting for user
-    ;; input and remains so.  The following workaround prevents this, but it
-    ;; also prevents widening category when edebugging todos.el.
-    ;; (save-excursion
-    ;;   (goto-char (point-min))
-    ;;   (when (re-search-forward todos-done-string-start nil t)
-    ;;         (let ((todos-show-with-done nil))
-    ;;           (todos-category-select))
-    ;;         (let ((todos-show-with-done t))
-    ;;           (todos-category-select))))
-    ))
+      (save-match-data (todos-reset-done-separator sep)))))
 
 ;; ---------------------------------------------------------------------------
 ;;; Global variables and helper functions for files and buffers
@@ -1008,6 +1016,11 @@ Added to `window-configuration-change-hook' in `todos-mode'."
 (defvar todos-archives (funcall todos-files-function t)
   "List of truenames of user's Todos archives.")
 
+(defvar todos-visited nil
+  "List of Todos files visited in this session by `todos-show'.
+Used to determine initial display according to the value of
+`todos-show-first'.")
+
 (defvar todos-file-buffers nil
   "List of file names of live Todos mode buffers.")
 
@@ -1037,15 +1050,19 @@ users option `todos-show-current-file' is non-nil).")
 (defvar todos-print-buffer "*Todos Print*"
   "Name of buffer containing printable Todos text.")
 
-(defun todos-absolute-file-name (name &optional archive)
+(defun todos-absolute-file-name (name &optional type)
   "Return the absolute file name of short Todos file NAME.
-With non-nil ARCHIVE return the absolute file name of the short
-Todos Archive name."
+With TYPE `archive' or `top' return the absolute file name of the
+short Todos Archive or Top Priorities file name, respectively."
   ;; NOP if there is no Todos file yet (i.e. don't concatenate nil).
   (when name
     (file-truename
      (concat todos-files-directory name
-            (if archive ".toda" ".todo")))))
+            (cond ((eq type 'archive) ".toda")
+                  ((eq type 'top) ".todt")
+                  ((eq type 'diary) ".tody")
+                  ((eq type 'regexp) ".todr")
+                  (t ".todo"))))))
 
 (defun todos-check-format ()
   "Signal an error if the current Todos file is ill-formatted.
@@ -1055,9 +1072,11 @@ where the invalid formatting was found."
     (save-restriction
       (widen)
       (goto-char (point-min))
-      ;; Check for `todos-categories' sexp as the first line
-      (let ((cats (prin1-to-string todos-categories)))
-       (unless (looking-at (regexp-quote cats))
+      (let ((cats (prin1-to-string todos-categories))
+           (sexp (buffer-substring-no-properties (line-beginning-position)
+                                                 (line-end-position))))
+       ;; Check for `todos-categories' sexp as the first line
+       (unless (string= sexp cats)
          (error "Invalid or missing todos-categories sexp")))
       (forward-line)
       (let ((legit (concat "\\(^" (regexp-quote todos-category-beg) "\\)"
@@ -1102,7 +1121,7 @@ number as its value."
          (1+ (- (length categories)
                 (length (member cat categories)))))))
 
-(defun todos-current-category ()       ;FIXME: arg FILE ?
+(defun todos-current-category ()
   "Return the name of the current category."
   (car (nth (1- todos-category-number) todos-categories)))
 
@@ -1115,9 +1134,17 @@ number as its value."
 (defun todos-done-separator ()
   "Return string used as value of variable `todos-done-separator'."
   (let ((sep todos-done-separator-string))
-    (if (= 1 (length sep))
-       (make-string (window-width) (string-to-char sep))
-      todos-done-separator-string)))
+    (propertize (if (= 1 (length sep))
+                   ;; If separator's length is window-width, then
+                   ;; with non-nil todos-wrap-lines and
+                   ;; todos-wrap-and-indent as value of
+                   ;; todos-line-wrapping-function, an indented empty
+                   ;; line appears between the separator and the
+                   ;; first done item.
+                   (make-string (1- (window-width)) (string-to-char sep))
+                   ;; (make-string (window-width) (string-to-char sep))
+                 todos-done-separator-string)
+               'face 'todos-done-sep)))
 
 (defvar todos-done-separator (todos-done-separator)
   "String used to visually separate done from not done items.
@@ -1135,15 +1162,16 @@ done items are shown.  Its value is determined by user option
              (concat "\n\\(" (regexp-quote todos-category-done) "\\)") nil t)
        (let* ((beg (match-beginning 1))
               (end (match-end 0))
-              (ovs (overlays-at beg))
-              old-sep new-sep)
-         (and ovs
-              (setq old-sep (overlay-get (car ovs) 'display))
-              (string= old-sep sep)
-              (delete-overlay (car ovs))
-              (setq new-sep (make-overlay beg end))
-              (overlay-put new-sep 'display
-                           todos-done-separator)))))))
+              (ov (progn (goto-char beg)
+                         (todos-get-overlay 'separator)))
+              (old-sep (when ov (overlay-get ov 'display)))
+              new-ov)
+         (when old-sep
+           (unless (string= old-sep sep)
+             (setq new-ov (make-overlay beg end))
+             (overlay-put new-ov 'todos 'separator)
+             (overlay-put new-ov 'display todos-done-separator)
+             (delete-overlay ov))))))))
 
 (defun todos-category-completions ()
   "Return a list of completions for `todos-read-category'.
@@ -1164,6 +1192,16 @@ Todos files named in `todos-category-completions-files'."
       (add-to-list 'files curfile)
       (dolist (f files listall)
        (with-current-buffer (find-file-noselect f 'nowarn)
+         ;; Ensure category is properly displayed in case user
+         ;; switches to file via a non-Todos command.  And if done
+         ;; items in category are visible, keep them visible.
+         (let ((done todos-show-with-done))
+           (when (> (buffer-size) (- (point-max) (point-min)))
+             (save-excursion
+               (goto-char (point-min))
+               (setq done (re-search-forward todos-done-string-start nil t))))
+           (let ((todos-show-with-done done))
+             (todos-category-select)))
          (save-excursion
            (save-restriction
              (widen)
@@ -1194,11 +1232,6 @@ Todos files named in `todos-category-completions-files'."
                    (point-max)))
     (setq mode-line-buffer-identification
          (funcall todos-mode-line-function name))
-    ;; FIXME: When, starting from `C-u i i' (and apparently only from
-    ;; this, e.g. `m' does not trigger the problem), after the
-    ;; following line is executed, the last line of the narrowed
-    ;; region (sometimes, always?) is at (window-start)... (continued
-    ;; below)
     (narrow-to-region cat-begin cat-end)
     (todos-prefix-overlays)
     (goto-char (point-min))
@@ -1219,20 +1252,12 @@ Todos files named in `todos-category-completions-files'."
        ;; Make display overlay for done items separator string, unless there
        ;; already is one.
        (let* ((done-sep todos-done-separator)
-              (ovs (overlays-at done-sep-start))
-              ov-sep)
-         ;; There should never be more than one overlay here, so car suffices.
-         (unless (and ovs (string= (overlay-get (car ovs) 'display) done-sep))
-           (setq ov-sep (make-overlay done-sep-start done-end))
-           (overlay-put ov-sep 'display done-sep))))
-      ;; FIXME: (continued) ...and after the following line, now the
-      ;; new last line of the narrowed region is (sometimes?) at
-      ;; (window-start), and after inserting the new item at the
-      ;; bottom of the list, the latter remains at (window-start).
-      ;; But `M-<' corrects the display, and since the narrowed region
-      ;; is shorter than (window-height), there is no way to
-      ;; interactively make Emacs show the last line at
-      ;; (window-start).
+              (ov (progn (goto-char done-sep-start)
+                         (todos-get-overlay 'separator))))
+         (unless ov
+           (setq ov (make-overlay done-sep-start done-end))
+           (overlay-put ov 'todos 'separator)
+           (overlay-put ov 'display done-sep))))
       (narrow-to-region (point-min) done-start)
       ;; Loading this from todos-mode, or adding it to the mode hook, causes
       ;; Emacs to hang in todos-item-start, at (looking-at todos-item-start).
@@ -1287,16 +1312,8 @@ With nil or omitted CATEGORY, default to the current category."
        (goto-char (point-min))
        (if (looking-at (concat "^" (regexp-quote todos-category-beg)))
            (progn (newline) (goto-char (point-min)) ; Make space for sexp.
-                  ;; No categories sexp means the first item was just added
-                  ;; to this file, so have to initialize Todos file and
-                  ;; categories variables in order e.g. to enable categories
-                  ;; display.
-                  ;; FIXME: is this right?
-                  (setq todos-default-todos-file (buffer-file-name))
                   (setq todos-categories (todos-make-categories-list t)))
-         ;; With empty buffer (e.g. with new archive in
-         ;; `todos-move-category') `kill-line' signals end of buffer.
-         (kill-region (line-beginning-position) (line-end-position)))
+         (delete-region (line-beginning-position) (line-end-position)))
        (prin1 todos-categories (current-buffer))))))
 
 (defun todos-make-categories-list (&optional force)
@@ -1373,7 +1390,11 @@ the file."
 (defun todos-repair-categories-sexp ()
   "Repair corrupt Todos categories sexp.
 This should only be needed as a consequence of careless manual
-editing or a bug in todos.el."
+editing or a bug in todos.el.
+
+*Warning*: Calling this command restores the category order to
+the list element order in the Todos categories sexp, so any order
+changes made in Todos Categories mode will have to be made again."
   (interactive)
   (let ((todos-categories (todos-make-categories-list t)))
     (todos-update-categories-sexp)))
@@ -1394,12 +1415,6 @@ The final element is \"*\", indicating an unspecified month.")
   (let ((dayname (diary-name-pattern calendar-day-name-array nil t)))
     (concat "\\(?5:" dayname "\\|"
            (let ((dayname)
-                 ;; FIXME: how to choose between abbreviated and unabbreviated
-                 ;; month name?
-                 ;; (monthname (format "\\(?6:%s\\|\\*\\)"
-                 ;;                 (diary-name-pattern
-                 ;;                  calendar-month-name-array
-                 ;;                  calendar-month-abbrev-array)))
                  (monthname (format "\\(?6:%s\\)" (diary-name-pattern
                                                    todos-month-name-array
                                                    todos-month-abbrev-array)))
@@ -1490,11 +1505,10 @@ The final element is \"*\", indicating an unspecified month.")
 
 (defun todos-remove-item ()
   "Internal function called in editing, deleting or moving items."
-  (let* ((beg (todos-item-start))
-        (end (progn (todos-item-end) (1+ (point))))
-        (ovs (overlays-in beg beg)))
-    ;; There can be both prefix/number and mark overlays.
-    (while ovs (delete-overlay (car ovs)) (pop ovs))
+  (let* ((end (progn (todos-item-end) (1+ (point))))
+        (beg (todos-item-start))
+        (ov (todos-get-overlay 'prefix)))
+    (when ov (delete-overlay ov))
     (delete-region beg end)))
 
 (defun todos-diary-item-p ()
@@ -1510,86 +1524,101 @@ The final element is \"*\", indicating an unspecified month.")
     (todos-item-start)
     (looking-at todos-done-string-start)))
 
-(defvar todos-item-mark (propertize (if (equal todos-prefix "*") "@" "*")
-                                   'face 'todos-mark)
-  "String used to mark items.")
+(defun todos-done-item-section-p ()
+  "Return non-nil if point is in category's done items section."
+  (save-excursion
+    (or (re-search-backward (concat "^" (regexp-quote todos-category-done))
+                           nil t)
+       (progn (goto-char (point-min))
+              (looking-at todos-done-string-start)))))
+
+(defun todos-get-overlay (val)
+  "Return the overlay at point whose `todos' property has value VAL."
+  ;; Use overlays-in to find prefix overlays and check over two
+  ;; positions to find done separator overlay.
+  (let ((ovs (overlays-in (point) (1+ (point))))
+       ov)
+    (catch 'done
+      (while ovs
+       (setq ov (pop ovs))
+       (when (eq (overlay-get ov 'todos) val)
+         (throw 'done ov))))))
 
 (defun todos-marked-item-p ()
-  "If this item begins with `todos-item-mark', return mark overlay."
-  (let ((ovs (overlays-in (line-beginning-position) (line-beginning-position)))
-       (mark todos-item-mark)
-       ov marked)
-    (catch 'stop
-      (while ovs
-       (setq ov (pop ovs))
-       (and (equal (overlay-get ov 'before-string) mark)
-            (throw 'stop (setq marked t)))))
+  "Non-nil if this item begins with `todos-item-mark'.
+ In that case, return the item's prefix overlay."
+  ;; If a todos-item-insert command is called on a Todos file before
+  ;; it is visited, it has no prefix overlays, so conditionalize:
+  (let* ((ov (todos-get-overlay 'prefix))
+        (pref (when ov (overlay-get ov 'before-string)))
+        (marked (when pref
+                  (string-match (concat "^" (regexp-quote todos-item-mark))
+                                pref))))
     (when marked ov)))
 
 (defun todos-insert-with-overlays (item)
   "Insert ITEM at point and update prefix/priority number overlays."
   (todos-item-start)
-  (insert item "\n")
+  ;; Insertion pushes item down but not its prefix overlay.  When the
+  ;; overlay includes a mark, this would now mark the inserted ITEM,
+  ;; so move it to the pushed down item.
+  (let ((ov (todos-get-overlay 'prefix))
+       (marked (todos-marked-item-p)))
+    (insert item "\n")
+    (when marked (move-overlay ov (point) (point))))
   (todos-backward-item)
   (todos-prefix-overlays))
 
-(defun todos-prefix-overlays ()                ;FIXME: this is a category function
-  "Put before-string overlay in front of this category's items.
+(defun todos-prefix-overlays ()
+  "Update the prefix overlays of the current category's items.
 The overlay's value is the string `todos-prefix' or with non-nil
 `todos-number-priorities' an integer in the sequence from 1 to
 the number of todo or done items in the category indicating the
 item's priority.  Todo and done items are numbered independently
 of each other."
-  (when (or todos-number-priorities
-           (not (string-match "^[[:space:]]*$" todos-prefix)))
-    (let ((prefix (propertize (concat todos-prefix " ")
-                             'face 'todos-prefix-string))
-         (num 0)
-         (cat-tp (or (cdr (assoc-string (todos-current-category)
-                      (nth 2 (assoc-string todos-current-todos-file
-                                           todos-priorities-rules))))
-                     todos-show-priorities))
-         done)
-      (save-excursion
-       (goto-char (point-min))
-       (while (not (eobp))
-         (when (or (todos-date-string-matcher (line-end-position))
-                   (todos-done-string-matcher (line-end-position)))
-           (goto-char (match-beginning 0))
-           (when todos-number-priorities
-             (setq num (1+ num))
-             ;; Reset number to 1 for first done item.
-             (when (and (looking-at todos-done-string-start)
-                        (looking-back (concat "^"
-                                              (regexp-quote todos-category-done)
-                                              "\n")))
-               (setq num 1
-                     done t))
-             (setq prefix (propertize (concat (number-to-string num) " ")
-                                      'face
-                                      (if (and (not done) (<= num cat-tp))
-                                          'todos-top-priority ; make defface
-                                        'todos-prefix-string))))
-           (let ((ovs (overlays-in (point) (point)))
-                 marked ov-pref)
-             (if ovs
-                 (dolist (ov ovs)
-                   (let ((val (overlay-get ov 'before-string)))
-                     (if (equal val "*")
-                         (setq marked t)
-                       (setq ov-pref val)))))
-             ;; Omitting this condition doesn't appear to slow
-             ;; redisplay down, while having it prevents updating
-             ;; display after changing number of top priorities.
-             ;; (unless (equal ov-pref prefix)
-               ;; Why doesn't this work?
-               ;; (remove-overlays (point) (point) 'before-string)
-             (remove-overlays (point) (point))
-             (overlay-put (make-overlay (point) (point))
-                          'before-string prefix)
-             (and marked (overlay-put (make-overlay (point) (point))
-                                      'before-string todos-item-mark))));)
-         (forward-line))))))
+  (let ((num 0)
+       (cat-tp (or (cdr (assoc-string
+                         (todos-current-category)
+                         (nth 2 (assoc-string todos-current-todos-file
+                                              todos-priorities-rules))))
+                   todos-show-priorities))
+       done prefix)
+    (save-excursion
+      (goto-char (point-min))
+      (while (not (eobp))
+       (when (or (todos-date-string-matcher (line-end-position))
+                 (todos-done-string-matcher (line-end-position)))
+         (goto-char (match-beginning 0))
+         (setq num (1+ num))
+         ;; Reset number to 1 for first done item.
+         (when (and (looking-at todos-done-string-start)
+                    (looking-back (concat "^"
+                                          (regexp-quote todos-category-done)
+                                          "\n")))
+           (setq num 1
+                 done t))
+         (setq prefix (concat (propertize
+                               (if todos-number-priorities
+                                   (number-to-string num)
+                                 todos-prefix)
+                               'face
+                               ;; Prefix of top priority items has a
+                               ;; distinct face in Todos mode.
+                               (if (and (not done) (<= num cat-tp)
+                                        (eq major-mode 'todos-mode))
+                                   'todos-top-priority
+                                 'todos-prefix-string))
+                              " "))
+         (let ((ov (todos-get-overlay 'prefix))
+               (marked (todos-marked-item-p)))
+           ;; Prefix overlay must be at a single position so its
+           ;; bounds aren't changed when (re)moving an item.
+           (unless ov (setq ov (make-overlay (point) (point))))
+           (overlay-put ov 'todos 'prefix)
+           (overlay-put ov 'before-string (if marked
+                                              (concat todos-item-mark prefix)
+                                            prefix))))
+       (forward-line)))))
 
 ;; ---------------------------------------------------------------------------
 ;;; Helper functions for user input with prompting and completion
@@ -1765,7 +1794,6 @@ When ARG is `day', non-nil arguments MO and YR determine the
 number of the last the day of the month."
   (let (year monthname month day
             dayname)                   ; Needed by calendar-date-display-form.
-    ;; FIXME: year can be omitted from Diary
     (when (or (not arg) (eq arg 'year))
       (while (if (natnump year) (< year 1) (not (eq year '*)))
        (setq year (read-from-minibuffer
@@ -1791,9 +1819,6 @@ number of the last the day of the month."
              month (1+ (- (length mlist)
                           (length (or (member monthname mlist)
                                       (member monthname mablist))))))
-       ;; FIXME: We follow diary-insert-entry in using abbreviated
-       ;; month name (and no day name) in date string.  Should this
-       ;; be customizable?
        (setq monthname (aref mabarray (1- month)))))
     (when (or (not arg) (eq arg 'day))
       (let ((last (let ((mm (or month mo))
@@ -1893,8 +1918,9 @@ the empty string (i.e., no time string)."
                   :notify (lambda (&rest ignore)
                             (setq todos-multiple-filter-files
                                   (mapcar (lambda (f)
-                                            (concat todos-files-directory
-                                                    f ".todo"))
+                                            (file-truename
+                                             (concat todos-files-directory
+                                                     f ".todo")))
                                           (widget-value
                                            todos-multiple-filter-files-widget)))
                             (quit-window t)
@@ -1905,31 +1931,69 @@ the empty string (i.e., no time string)."
   (message "Click \"Apply\" after selecting files.")
   (recursive-edit))
 
-(defun todos-filter-items (filter &optional multifile)
-  "Build and display a list of items from different categories.
-
-The items are selected according to the value of FILTER, which
-can be `top' for top priority items, `diary' for diary items,
-`regexp' for items matching a regular expresion entered by the
-user, or a cons cell of one of these symbols and a number set by
-the calling command, which overrides `todos-show-priorities'.
-
-With non-nil argument MULTIFILE list top priorities of multiple
-Todos files, by default those in `todos-filter-files'."
+(defun todos-filter-items (filter &optional new multifile)
+  "Internal routine for displaying items that satisfy FILTER.
+The values of FILTER can be `top' for top priority items, a cons
+of `top' and a number passed by the caller, `diary' for diary
+items, or `regexp' for items matching a regular expresion entered
+by the user.  The items can be from any categories in the current
+todo file or, with non-nil MULTIFILE, from several files.  If NEW
+is nil, visit an appropriate file containing the list of filtered
+items; if there is no such file, or with non-nil NEW, build the
+list and display it.
+
+See the document strings of the commands `todos-top-priorities',
+`todos-diary-items', `todos-regexp-items', and those of the
+corresponding multifile commands for further details. "
+  (let* ((top (eq filter 'top))
+        (diary (eq filter 'diary))
+        (regexp (eq filter 'regexp))
+        (buf (cond (top todos-top-priorities-buffer)
+                   (diary todos-diary-items-buffer)
+                   (regexp todos-regexp-items-buffer)))
+        (flist (if multifile
+                   (or todos-filter-files
+                       (progn (todos-multiple-filter-files)
+                              todos-multiple-filter-files))
+                 (list todos-current-todos-file)))
+        (multi (> (length flist) 1))
+        (fname (if (equal flist 'quit)
+                   ;; Pressed `cancel' in t-m-f-f file selection dialog.
+                   (keyboard-quit)
+                 (concat todos-files-directory
+                         (mapconcat 'todos-short-file-name flist "-")
+                         (cond (top ".todt")
+                               (diary ".tody")
+                               (regexp ".todr")))))
+        (rxfiles (when regexp
+                   (directory-files todos-files-directory t ".*\\.todr$" t)))
+        (file-exists (or (file-exists-p fname) rxfiles)))
+    (cond ((and top new (natnump new))
+          (todos-filter-items-1 (cons 'top new) flist))
+         ((and (not new) file-exists)
+          (when (and rxfiles (> (length rxfiles) 1))
+            (let ((rxf (mapcar 'todos-short-file-name rxfiles)))
+              (setq fname (todos-absolute-file-name
+                           (completing-read "Choose a regexp items file: "
+                                            rxf) 'regexp))))
+          (find-file fname)
+          (todos-prefix-overlays)
+          (todos-check-filtered-items-file))
+         (t
+          (todos-filter-items-1 filter flist)))
+    (when (or new (not file-exists))
+      (setq fname (replace-regexp-in-string "-" ", " fname))
+      (rename-buffer (format (concat "%s for file" (if multi "s" "")
+                                  " \"%s\"") buf fname)))))
+
+(defun todos-filter-items-1 (filter file-list)
+  "Internal subroutine called by `todos-filter-items'.
+The values of FILTER and FILE-LIST are passed from the caller."
   (let ((num (if (consp filter) (cdr filter) todos-show-priorities))
        (buf (get-buffer-create todos-filtered-items-buffer))
-       (files (list todos-current-todos-file))
+       (multifile (> (length file-list) 1))
        regexp fname bufstr cat beg end done)
-    (when multifile
-      (setq files (or todos-multiple-filter-files ; Passed from todos-*-multifile.
-                     (if (or (consp filter)
-                             (null todos-filter-files))
-                         (progn (todos-multiple-filter-files)
-                                todos-multiple-filter-files)
-                       todos-filter-files))
-           todos-multiple-filter-files nil))
-    (if (eq files 'quit) (keyboard-quit))
-    (if (null files)
+    (if (null file-list)
        (error "No files have been chosen for filtering")
       (with-current-buffer buf
        (erase-buffer)
@@ -1938,7 +2002,7 @@ Todos files, by default those in `todos-filter-files'."
       (when (eq filter 'regexp)
        (setq regexp (read-string "Enter a regular expression: ")))
       (save-current-buffer
-       (dolist (f files)
+       (dolist (f file-list)
          ;; Before inserting file contents into temp buffer, save a modified
          ;; buffer visiting it.
          (let ((bf (find-buffer-visiting f)))
@@ -1975,21 +2039,23 @@ Todos files, by default those in `todos-filter-files'."
            ;; Delete Todos file categories sexp.
            (delete-region (line-beginning-position) (1+ (line-end-position)))
            (let (fnum)
-             ;; Unless the number of items to show was supplied by prefix
-             ;; argument of caller, the file-wide value from
+             ;; Unless the number of top priorities to show was
+             ;; passed by the caller, the file-wide value from
              ;; `todos-priorities-rules', if non-nil, overrides
              ;; `todos-show-priorities'.
              (unless (consp filter)
-               (setq fnum (nth 1 (assoc f todos-priorities-rules))))
+               (setq fnum (or (nth 1 (assoc f todos-priorities-rules))
+                              todos-show-priorities)))
              (while (re-search-forward
                      (concat "^" (regexp-quote todos-category-beg) "\\(.+\\)\n")
                      nil t)
                (setq cat (match-string 1))
                (let (cnum)
-                 ;; Unless the number of items to show was supplied by prefix
-                 ;; argument of caller, the category-wide value from
-                 ;; `todos-priorities-rules', if non-nil, overrides a non-nil
-                 ;; file-wide value from `todos-priorities-rules' as well as
+                 ;; Unless the number of top priorities to show was
+                 ;; passed by the caller, the category-wide value
+                 ;; from `todos-priorities-rules', if non-nil,
+                 ;; overrides a non-nil file-wide value from
+                 ;; `todos-priorities-rules' as well as
                  ;; `todos-show-priorities'.
                  (unless (consp filter)
                    (let ((cats (nth 2 (assoc f todos-priorities-rules))))
@@ -2117,19 +2183,142 @@ set the user customizable option `todos-priorities-rules'."
     (customize-save-variable 'todos-priorities-rules rules)
     (todos-prefix-overlays)))
 
-(defun todos-filtered-buffer-name (buffer-type file-list)
-  "Rename Todos filtered buffer using BUFFER-TYPE and FILE-LIST.
-
-The new name is constructed from the string BUFFER-TYPE, which
-refers to one of the top priorities, diary or regexp item
-filters, and the names of the filtered files in FILE-LIST.  Used
-in Todos Filter Items mode."
-  (let* ((flist (if (listp file-list) file-list (list file-list)))
-        (multi (> (length flist) 1))
-        (fnames (mapconcat (lambda (f) (todos-short-file-name f))
-                          flist ", ")))
-    (rename-buffer (format (concat "%s for file" (if multi "s" "")
-                                  " \"%s\"") buffer-type fnames))))
+(defconst todos-filtered-items-buffer "Todos filtered items"
+  "Initial name of buffer in Todos Filter Items mode.")
+
+(defconst todos-top-priorities-buffer "Todos top priorities"
+  "Buffer type string for `todos-filter-items'.")
+
+(defconst todos-diary-items-buffer "Todos diary items"
+  "Buffer type string for `todos-filter-items'.")
+
+(defconst todos-regexp-items-buffer "Todos regexp items"
+  "Buffer type string for `todos-filter-items'.")
+
+(defun todos-find-item (str)
+  "Search for filtered item STR in its saved Todos file.
+Return the list (FOUND FILE CAT), where CAT and FILE are the
+item's category and file, and FOUND is a cons cell if the search
+succeeds, whose car is the start of the item in FILE and whose
+cdr is `done', if the item is now a done item, `changed', if its
+text was truncated or augmented or, for a top priority item, if
+its priority has changed, and `same' otherwise."
+  (string-match (concat (if todos-filter-done-items
+                           (concat "\\(?:" todos-done-string-start "\\|"
+                                   todos-date-string-start "\\)")
+                         todos-date-string-start)
+                       todos-date-pattern "\\(?: " diary-time-regexp "\\)?"
+                       (if todos-filter-done-items
+                           "\\]"
+                         (regexp-quote todos-nondiary-end)) "?"
+                       "\\(?4: \\[\\(?3:(archive) \\)?\\(?2:.*:\\)?"
+                       "\\(?1:.*\\)\\]\\).*$") str)
+  (let ((cat (match-string 1 str))
+       (file (match-string 2 str))
+       (archive (string= (match-string 3 str) "(archive) "))
+       (filcat (match-string 4 str))
+       (tpriority 1)
+       (tpbuf (string-match "top" (buffer-name)))
+       found)
+    (setq str (replace-match "" nil nil str 4))
+    (when tpbuf
+      ;; Calculate priority of STR wrt its category.
+      (save-excursion
+       (while (search-backward filcat nil t)
+           (setq tpriority (1+ tpriority)))))
+    (setq file (if file
+                  (concat todos-files-directory (substring file 0 -1)
+                          (if archive ".toda" ".todo"))
+                (if archive
+                    (concat (file-name-sans-extension
+                             todos-global-current-todos-file) ".toda")
+                  todos-global-current-todos-file)))
+    (find-file-noselect file)
+    (with-current-buffer (find-buffer-visiting file)
+      (save-restriction
+       (widen)
+       (goto-char (point-min))
+       (let ((beg (re-search-forward
+                   (concat "^" (regexp-quote (concat todos-category-beg cat))
+                           "$")
+                   nil t))
+             (done (save-excursion
+                     (re-search-forward
+                      (concat "^" (regexp-quote todos-category-done)) nil t)))
+             (end (save-excursion
+                    (or (re-search-forward
+                         (concat "^" (regexp-quote todos-category-beg))
+                         nil t)
+                        (point-max)))))
+         (setq found (when (search-forward str end t)
+                       (goto-char (match-beginning 0))))
+         (when found
+           (setq found
+                 (cons found (if (> (point) done)
+                                 'done
+                               (let ((cpriority 1))
+                                 (when tpbuf
+                                   (save-excursion
+                                     ;; Not top item in category.
+                                     (while (> (point) (1+ beg))
+                                       (let ((opoint (point)))
+                                         (todos-backward-item)
+                                         ;; Can't move backward beyond
+                                         ;; first item in file.
+                                         (unless (= (point) opoint)
+                                           (setq cpriority (1+ cpriority)))))))
+                                 (if (and (= tpriority cpriority)
+                                          ;; Proper substring is not the same.
+                                          (string= (todos-item-string)
+                                                   str))
+                                     'same
+                                   'changed)))))))))
+      (list found file cat)))
+
+(defun todos-check-filtered-items-file ()
+  "Check if filtered items file is up to date and a show suitable message."
+  ;; (catch 'old
+  (let ((count 0))
+    (while (not (eobp))
+      (let* ((item (todos-item-string))
+            (found (car (todos-find-item item))))
+       (unless (eq (cdr found) 'same)
+         (save-excursion
+           (overlay-put (make-overlay (todos-item-start) (todos-item-end))
+                        'face 'todos-search))
+         (setq count (1+ count))))
+         ;; (throw 'old (message "The marked item is not up to date.")))
+      (todos-forward-item))
+    (if (zerop count)
+       (message "Filtered items file is up to date.")
+      (message (concat "The highlighted item" (if (= count 1) " is " "s are ")
+                      "not up to date."
+                      ;; "\nType <return> on item for details."
+                      )))))
+
+(defun todos-filter-items-filename ()
+  "Return absolute file name for saving this Filtered Items buffer."
+  (let ((bufname (buffer-name)))
+    (string-match "\"\\([^\"]+\\)\"" bufname)
+    (let* ((filename-str (substring bufname (match-beginning 1) (match-end 1)))
+          (filename-base (replace-regexp-in-string ", " "-" filename-str))
+          (top-priorities (string-match "top priorities" bufname))
+          (diary-items (string-match "diary items" bufname))
+          (regexp-items (string-match "regexp items" bufname)))
+      (when regexp-items
+       (let ((prompt (concat "Enter a short identifying string"
+                             " to make this file name unique: ")))
+         (setq filename-base (concat filename-base "-" (read-string prompt)))))
+      (concat todos-files-directory filename-base
+             (cond (top-priorities ".todt")
+                   (diary-items ".tody")
+                   (regexp-items ".todr"))))))
+
+(defun todos-save-filtered-items-buffer ()
+  "Save current Filtered Items buffer to a file.
+If the file already exists, overwrite it only on confirmation."
+  (let ((filename (or (buffer-file-name) (todos-filter-items-filename))))
+    (write-file filename t)))
 
 ;; ---------------------------------------------------------------------------
 ;;; Sorting and display routines for Todos Categories mode.
@@ -2140,15 +2329,26 @@ in Todos Filter Items mode."
     (dolist (c categories longest)
       (setq longest (max longest (length c))))))
 
+(defun todos-adjusted-category-label-length ()
+  "Return adjusted length of category label button.
+The adjustment ensures proper tabular alignment in Todos
+Categories mode."
+  (let* ((categories (mapcar 'car todos-categories))
+        (longest (todos-longest-category-name-length categories))
+        (catlablen (length todos-categories-category-label))
+        (lc-diff (- longest catlablen)))
+    (if (and (natnump lc-diff)
+            (eq (logand lc-diff 1) 1)) ; oddp from cl.el
+       (1+ longest)
+      (max longest catlablen))))
+
 (defun todos-padded-string (str)
-  "Return string STR padded with spaces.
+  "Return category name or label string STR padded with spaces.
 The placement of the padding is determined by the value of user
 option `todos-categories-align'."
-  (let* ((categories (mapcar 'car todos-categories))
-        (len (max (todos-longest-category-name-length categories)
-                  (length todos-categories-category-label)))
+  (let* ((len (todos-adjusted-category-label-length))
         (strlen (length str))
-        (strlen-odd (eq (logand strlen 1) 1)) ; oddp from cl.el
+        (strlen-odd (eq (logand strlen 1) 1))
         (padding (max 0 (/ (- len strlen) 2)))
         (padding-left (cond ((eq todos-categories-align 'left) 0)
                             ((eq todos-categories-align 'center) padding)
@@ -2321,7 +2521,6 @@ which is the value of the user option
     (let ((archive (member todos-current-todos-file todos-archives))
          buffer-read-only) 
       (erase-buffer)
-      ;; FIXME: add usage tips?
       (insert (format (concat "Category counts for Todos "
                              (if archive "archive" "file")
                              " \"%s\".")
@@ -2481,8 +2680,8 @@ which is the value of the user option
    "-\\_>" ""
    (replace-regexp-in-string
     "-+" "-"
-    ;; FIXME: "todos-insert-item-"
     (concat "todos-item-insert-"
+    ;; (concat "todos-insert-item-"
            (mapconcat (lambda (e) (if e (symbol-name e))) arglist "-")))))
 
 (defvar todos-insertion-commands-names
@@ -2491,7 +2690,6 @@ which is the value of the user option
          todos-insertion-commands-args)
   "List of names of Todos insertion commands.")
 
-;; FIXME: prefix argument ARG is nil
 (defmacro todos-define-insertion-command (&rest args)
   (let ((name (intern (todos-insertion-command-name args)))
        (arg0 (nth 0 args))
@@ -2504,9 +2702,6 @@ which is the value of the user option
        (interactive (list current-prefix-arg))
        (todos-insert-item arg ',arg0 ',arg1 ',arg2 ',arg3 ',arg4))))
 
-;; FIXME: exclude todos-insert-item (or rather from
-;; todos-insertion-key-bindings?), otherwise its doc string won't be
-;; found with C-h k (but it will with M-x todos-insert-item)
 (defvar todos-insertion-commands
   (mapcar (lambda (c)
            (eval `(todos-define-insertion-command ,@c)))
@@ -2539,6 +2734,7 @@ which is the value of the user option
                    (setq key (concat key key1)))))
            todos-insertion-commands-arg-key-list)
       (if (string-match (concat (regexp-quote "todos-item-insert") "\\_>") cname)
+      ;; (if (string-match (concat (regexp-quote "todos-insert-item") "\\_>") cname)
          (setq key (concat key "i")))
       (define-key map key c))))
 
@@ -2584,6 +2780,7 @@ which is the value of the user option
     ;;               navigation                        
     ("f"            . todos-forward-category)
     ("b"            . todos-backward-category)
+    ("t"             . todos-show)
     ("j"            . todos-jump-to-category)
     ("n"            . todos-forward-item)
     ("p"            . todos-backward-item)
@@ -2636,7 +2833,6 @@ which is the value of the user option
     map)
   "Todos mode keymap.")
 
-;; FIXME
 (easy-menu-define
   todos-menu todos-mode-map "Todos Menu"
   '("Todos"
@@ -2705,9 +2901,12 @@ which is the value of the user option
     (define-key map "n" 'todos-forward-item)
     (define-key map "p" 'todos-backward-item)
     ;; display commands
-    (define-key map "C" 'todos-display-categories)
+    (define-key map "Cd" 'todos-display-categories)
     (define-key map "H" 'todos-highlight-item)
     (define-key map "N" 'todos-hide-show-item-numbering)
+    (define-key map "*"        'todos-mark-unmark-item)
+    (define-key map "C*" 'todos-mark-category)
+    (define-key map "Cu" 'todos-unmark-category)
     ;; (define-key map "" 'todos-hide-show-date-time)
     (define-key map "P" 'todos-print)
     (define-key map "q" 'todos-quit)
@@ -2715,7 +2914,6 @@ which is the value of the user option
     (define-key map "S" 'todos-search)
     (define-key map "t" 'todos-show)
     (define-key map "u" 'todos-unarchive-items)
-    (define-key map "U" 'todos-unarchive-category)
     map)
   "Todos Archive mode keymap.")
 
@@ -2729,17 +2927,19 @@ which is the value of the user option
 (defvar todos-categories-mode-map
   (let ((map (make-sparse-keymap)))
     (suppress-keymap map t)
+    ;; (define-key map "c" 'todos-display-categories-numberically-or-alphabetically)
     (define-key map "c" 'todos-display-categories-alphabetically-or-by-priority)
     (define-key map "t" 'todos-display-categories-sorted-by-todo)
     (define-key map "y" 'todos-display-categories-sorted-by-diary)
     (define-key map "d" 'todos-display-categories-sorted-by-done)
     (define-key map "a" 'todos-display-categories-sorted-by-archived)
+    (define-key map "#" 'todos-set-category-priority)
     (define-key map "l" 'todos-lower-category-priority)
     (define-key map "+" 'todos-lower-category-priority)
     (define-key map "r" 'todos-raise-category-priority)
     (define-key map "-" 'todos-raise-category-priority)
-    (define-key map "n" 'todos-forward-button)
-    (define-key map "p" 'todos-backward-button)
+    (define-key map "n" 'todos-forward-button) ; todos-next-button
+    (define-key map "p" 'todos-backward-button) ; todos-previous-button
     (define-key map [tab] 'todos-forward-button)
     (define-key map [backtab] 'todos-backward-button)
     (define-key map "q" 'todos-quit)
@@ -2791,7 +2991,6 @@ which is the value of the user option
   ""
   (set (make-local-variable 'todos-categories) (todos-set-categories))
   (set (make-local-variable 'todos-category-number) 1)
-  (set (make-local-variable 'todos-first-visit) t)
   (add-hook 'find-file-hook 'todos-display-as-todos-file nil t))
 
 (put 'todos-mode 'mode-class 'special)
@@ -2809,7 +3008,6 @@ which is the value of the user option
                (funcall todos-files-function))
     (set (make-local-variable 'todos-current-todos-file)
         (file-truename (buffer-file-name))))
-  (set (make-local-variable 'todos-first-visit) t)
   (set (make-local-variable 'todos-show-done-only) nil)
   (set (make-local-variable 'todos-categories-with-marks) nil)
   (add-hook 'find-file-hook 'todos-add-to-buffer-list nil t)
@@ -2820,16 +3018,6 @@ which is the value of the user option
            'todos-reset-and-enable-done-separator nil t)
   (add-hook 'kill-buffer-hook 'todos-reset-global-current-todos-file nil t))
 
-(defun todos-unload-hook ()
-  ""
-  (remove-hook 'pre-command-hook 'todos-show-current-file t)
-  (remove-hook 'post-command-hook 'todos-update-buffer-list t)
-  (remove-hook 'find-file-hook 'todos-display-as-todos-file t)
-  (remove-hook 'find-file-hook 'todos-add-to-buffer-list t)
-  (remove-hook 'window-configuration-change-hook
-              'todos-reset-and-enable-done-separator t)
-  (remove-hook 'kill-buffer-hook 'todos-reset-global-current-todos-file t))
-
 (put 'todos-archive-mode 'mode-class 'special)
 
 ;; If todos-mode is parent, all todos-mode key bindings appear to be
@@ -2850,8 +3038,20 @@ which is the value of the user option
   (set (make-local-variable 'todos-current-todos-file)
        todos-global-current-todos-file)
   (let ((cats (with-current-buffer
-                 (find-buffer-visiting todos-current-todos-file)
-               todos-categories)))
+                 ;; Can't use find-buffer-visiting when
+                 ;; `todos-display-categories' is called on first
+                 ;; invocation of `todos-show', since there is then
+                 ;; no buffer visiting the current file.
+                 (find-file-noselect todos-current-todos-file 'nowarn)
+               (or todos-categories
+                   ;; In Todos Edit mode todos-categories is now nil
+                   ;; since it uses same buffer as Todos mode but
+                   ;; doesn't have the latter's local variables.
+                   (save-excursion
+                     (goto-char (point-min))
+                     (read (buffer-substring-no-properties
+                            (line-beginning-position)
+                            (line-end-position))))))))
     (set (make-local-variable 'todos-categories) cats)))
 
 (define-derived-mode todos-edit-mode text-mode "Todos-Ed"
@@ -2859,7 +3059,8 @@ which is the value of the user option
 
 \\{todos-edit-mode-map}"
   (todos-modes-set-1)
-  (todos-mode-external-set))
+  (todos-mode-external-set)
+  (setq buffer-read-only nil))
 
 (put 'todos-categories-mode 'mode-class 'special)
 
@@ -2869,7 +3070,7 @@ which is the value of the user option
 \\{todos-categories-mode-map}"
   (todos-mode-external-set))
 
-(put 'todos-filter-mode 'mode-class 'special)
+(put 'todos-filtered-items-mode 'mode-class 'special)
 
 (define-derived-mode todos-filtered-items-mode special-mode "Todos-Fltr"
   "Mode for displaying and reprioritizing top priority Todos.
@@ -2886,23 +3087,29 @@ which is the value of the user option
 
 ;;;###autoload
 (defun todos-show (&optional solicit-file)
-  "Visit the current Todos file and display one of its categories.
-With non-nil prefix argument SOLICIT-FILE prompt for which todo
-file to visit.
-
-Without a prefix argument, the first invocation of this command
-in a session visits `todos-default-todos-file' (creating it if it
-does not yet exist); subsequent invocations from outside of Todos
-mode revisit this file or, if the user option
-`todos-show-current-file' is non-nil, whichever Todos file
-\(either a todo or an archive file) was visited last.
-
-The category displayed on initial invocation is the first member
-of `todos-categories' for the current Todos file, on subsequent
-invocations whichever category was displayed last.  If
-`todos-display-categories-first' is non-nil, then the first
-invocation of `todos-show' displays a clickable listing of the
-categories in the current Todos file.
+  "Visit a Todos file and display one of its categories.
+
+When invoked in Todos mode, prompt for which todo file to visit.
+When invoked outside of Todos mode with non-nil prefix argument
+SOLICIT-FILE prompt for which todo file to visit; otherwise visit
+`todos-default-todos-file'.  Subsequent invocations from outside
+of Todos mode revisit this file or, with option
+`todos-show-current-file' non-nil (the default), whichever Todos
+file was last visited.
+
+Calling this command before any Todos file exists prompts for a
+file name and an initial category (defaulting to
+`todos-initial-file' and `todos-initial-category'), creates both
+of these, visits the file and displays the category.
+
+The first invocation of this command on an existing Todos file
+interacts with the option `todos-show-first': if its value is
+`first' (the default), show the first category in the file; if
+its value is `table', show the table of categories in the file;
+if its value is one of `top', `diary' or `regexp', show the
+corresponding saved top priorities, diary items, or regexp items
+file, if any.  Subsequent invocations always show the file's
+current (i.e., last displayed) category.
 
 In Todos mode just the category's unfinished todo items are shown
 by default.  The done items are hidden, but typing
@@ -2910,11 +3117,13 @@ by default.  The done items are hidden, but typing
 items.  With non-nil user option `todos-show-with-done' both todo
 and done items are always shown on visiting a category.
 
-If this command is invoked in Todos Archive mode, it visits the
+Invoking this command in Todos Archive mode visits the
 corresponding Todos file, displaying the corresponding category."
   (interactive "P")
   (let* ((cat)
-        (file (cond (solicit-file
+        (show-first todos-show-first)
+        (file (cond ((or (eq major-mode 'todos-mode)
+                         solicit-file)
                      (if (funcall todos-files-function)
                          (todos-read-file-name "Choose a Todos file to visit: "
                                                nil t)
@@ -2932,12 +3141,44 @@ corresponding Todos file, displaying the corresponding category."
                               todos-global-current-todos-file)
                          (todos-absolute-file-name todos-default-todos-file)
                          (todos-add-file))))))
-    (if (and todos-first-visit todos-display-categories-first)
-       (todos-display-categories)
+    (unless (member file todos-visited)
+      ;; Can't setq t-c-t-f here, otherwise wrong file shown when
+      ;; todos-show is called from todos-display-categories.
+      (let ((todos-current-todos-file file))
+       (cond ((eq todos-show-first 'table)
+              (todos-display-categories))
+             ((memq todos-show-first '(top diary regexp))
+              (let* ((shortf (todos-short-file-name file))
+                     (fi-file (todos-absolute-file-name
+                               shortf todos-show-first)))
+                (when (eq todos-show-first 'regexp)
+                  (let ((rxfiles (directory-files todos-files-directory t
+                                                  ".*\\.todr$" t)))
+                    (when (and rxfiles (> (length rxfiles) 1))
+                      (let ((rxf (mapcar 'todos-short-file-name rxfiles)))
+                        (setq fi-file (todos-absolute-file-name
+                                       (completing-read
+                                        "Choose a regexp items file: "
+                                        rxf) 'regexp))))))
+                (if (file-exists-p fi-file)
+                    (set-window-buffer
+                     (selected-window)
+                     (set-buffer (find-file-noselect fi-file 'nowarn)))
+                  (message "There is no %s file for %s"
+                           (cond ((eq todos-show-first 'top)
+                                  "top priorities")
+                                 ((eq todos-show-first 'diary)
+                                  "diary items")
+                                 ((eq todos-show-first 'regexp)
+                                  "regexp items"))
+                           shortf)
+                  (setq todos-show-first 'first)))))))
+    (when (or (member file todos-visited)
+             (eq todos-show-first 'first))
       (set-window-buffer (selected-window)
                         (set-buffer (find-file-noselect file 'nowarn)))
-      ;; If called from archive file, show corresponding category in Todos
-      ;; file, if it exists.
+      ;; If called from archive file, show corresponding
+      ;; category in Todos file, if it exists.
       (when (assoc cat todos-categories)
        (setq todos-category-number (todos-category-number cat)))
       ;; If this is a new Todos file, add its first category.
@@ -2945,7 +3186,8 @@ corresponding Todos file, displaying the corresponding category."
        (setq todos-category-number
              (todos-add-category todos-current-todos-file "")))
       (save-excursion (todos-category-select)))
-    (setq todos-first-visit nil)))
+    (setq todos-show-first show-first)
+    (add-to-list 'todos-visited file)))
 
 (defun todos-display-categories ()
   "Display a table of the current file's categories and item counts.
@@ -3063,11 +3305,14 @@ displayed."
   (interactive)
   (todos-show-archive t))
 
-;; FIXME: need this?
 (defun todos-save ()
   "Save the current Todos file."
   (interactive)
-  (save-buffer))
+  (cond ((eq major-mode 'todos-filtered-items-mode)
+        (todos-check-filtered-items-file)
+        (todos-save-filtered-items-buffer))
+       (t
+        (save-buffer))))
 
 (defun todos-quit ()
   "Exit the current Todos-related buffer.
@@ -3075,12 +3320,19 @@ Depending on the specific mode, this either kills the buffer or
 buries it and restores state as needed."
   (interactive)
   (cond ((eq major-mode 'todos-categories-mode)
-        (kill-buffer)
-        (setq todos-descending-counts nil)
-        (todos-show))
+        ;; Postpone killing buffer till after calling todos-show, to
+        ;; prevent killing todos-mode buffer.
+        (let ((buf (current-buffer)))
+          (setq todos-descending-counts nil)
+          ;; Ensure todos-show calls todos-display-categories only on
+          ;; first invocation per file.
+          (when (eq todos-show-first 'table)
+            (add-to-list 'todos-visited todos-current-todos-file))
+          (todos-show)
+          (kill-buffer buf)))
        ((eq major-mode 'todos-filtered-items-mode)
         (kill-buffer)
-        (todos-show))
+        (unless (eq major-mode 'todos-mode) (todos-show)))
        ((member major-mode (list 'todos-mode 'todos-archive-mode))
         ;; Have to write previously nonexistant archives to file, and might
         ;; as well save Todos file also.
@@ -3368,51 +3620,40 @@ CAT; this is used in Todos Categories mode."
 (defun todos-jump-to-item ()
   "Jump to the file and category of the filtered item at point."
   (interactive)
-  (let ((str (todos-item-string))
-       (buf (current-buffer))
-       cat file archive beg)
-    (string-match (concat (if todos-filter-done-items
-                             (concat "\\(?:" todos-done-string-start "\\|"
-                                     todos-date-string-start "\\)")
-                           todos-date-string-start)
-                         todos-date-pattern "\\(?: " diary-time-regexp "\\)?"
-                         (if todos-filter-done-items
-                             "\\]"
-                           (regexp-quote todos-nondiary-end)) "?"
-                         "\\(?4: \\[\\(?3:(archive) \\)?\\(?2:.*:\\)?"
-                         "\\(?1:.*\\)\\]\\).*$") str)
-    (setq cat (match-string 1 str))
-    (setq file (match-string 2 str))
-    (setq archive (string= (match-string 3 str) "(archive) "))
-    (setq str (replace-match "" nil nil str 4))
-    (setq file (if file
-                  (concat todos-files-directory (substring file 0 -1)
-                          (if archive ".toda" ".todo"))
-                (if archive
-                    (concat (file-name-sans-extension
-                             todos-global-current-todos-file) ".toda")
-                  todos-global-current-todos-file)))
-    (find-file-noselect file)
-    (with-current-buffer (find-buffer-visiting file)
-      (widen)
-      (goto-char (point-min))
-      (re-search-forward
-       (concat "^" (regexp-quote (concat todos-category-beg cat)) "$") nil t)
-      (search-forward str)
-      (setq beg (match-beginning 0)))
-    (kill-buffer buf)
-    (set-window-buffer (selected-window) (set-buffer (find-buffer-visiting file)))
-    (setq todos-current-todos-file file)
-    (setq todos-category-number (todos-category-number cat))
-    (let ((todos-show-with-done (if todos-filter-done-items t
-                                 todos-show-with-done)))
-      (todos-category-select))
-    (goto-char beg)))
+  (let* ((str (todos-item-string))
+        (buf (current-buffer))
+        (res (todos-find-item str))
+        (found (nth 0 res))
+        (file (nth 1 res))
+        (cat (nth 2 res)))
+    (if (not found)
+       (message "Category %s does not contain this item." cat)
+      (kill-buffer buf)
+      (set-window-buffer (selected-window)
+                        (set-buffer (find-buffer-visiting file)))
+      (setq todos-current-todos-file file)
+      (setq todos-category-number (todos-category-number cat))
+      (let ((todos-show-with-done (if (or todos-filter-done-items
+                                         (eq (cdr found) 'done))
+                                     t
+                                   todos-show-with-done)))
+       (todos-category-select))
+      (goto-char (car found)))))
 
 (defun todos-forward-item (&optional count)
   "Move point down to start of item with next lower priority.
 With positive numerical prefix COUNT, move point COUNT items
-downward."
+downward.
+
+If the category's done items are hidden, this command also moves
+point to the empty line below the last todo item from any higher
+item in the category, i.e., when invoked with or without a prefix
+argument.  If the category's done items are visible, this command
+called with a prefix argument only moves point to a lower item,
+e.g., with point on the last todo item and called with prefix 1,
+it moves point to the first done item; but if called with point
+on the last todo item without a prefix argument, it moves point
+the the empty line above the done items separator."
   (interactive "P")
   ;; It's not worth the trouble to allow prefix arg value < 1, since we have
   ;; the corresponding command.
@@ -3428,9 +3669,9 @@ downward."
       ;; space above todos-done-separator, since that is a legitimate place to
       ;; insert an item.  But skip this space if count > 1, since that should
       ;; only stop on an item.
-      (when (and not-done (todos-done-item-p))
-       (if (or (not count) (= count 1))
-           (re-search-backward "^$" start t))))))
+      (when (and not-done (todos-done-item-p) (not count))
+       ;; (if (or (not count) (= count 1))
+           (re-search-backward "^$" start t)))));)
     ;; FIXME: The preceding sexp is insufficient when buffer is not narrowed,
     ;; since there could be no done items in this category, so the search puts
     ;; us on first todo item of next category.  Does this ever happen?  If so:
@@ -3447,7 +3688,14 @@ downward."
 (defun todos-backward-item (&optional count)
   "Move point up to start of item with next higher priority.
 With positive numerical prefix COUNT, move point COUNT items
-upward."
+upward.
+
+If the category's done items are visible, this command called
+with a prefix argument only moves point to a higher item, e.g.,
+with point on the first done item and called with prefix 1, it
+moves to the last todo item; but if called with point on the
+first done item without a prefix argument, it moves point the the
+empty line above the done items separator."
   (interactive "P")
   ;; Avoid moving to bob if on the first item but not at bob.
   (when (> (line-number-at-pos) 1)
@@ -3465,7 +3713,8 @@ upward."
        ;; todos-done-separator, since that is a legitimate place to insert an
        ;; item.  But skip this space if count > 1, since that should only
        ;; stop on an item.
-       (when (and done (not (todos-done-item-p)) (or (not count) (= count 1))
+       (when (and done (not (todos-done-item-p)) (not count)
+                                       ;(or (not count) (= count 1))
                   (not (equal (buffer-name) todos-regexp-items-buffer)))
          (re-search-forward (concat "^" (regexp-quote todos-category-done))
                             nil t)
@@ -3487,9 +3736,6 @@ upward."
        ;; Align with beginning of category label.
        (forward-char (+ 4 (length todos-categories-number-separator)))))
 
-;; FIXME: (i) Extend search to other Todos files. (ii) Allow navigating among
-;; hits. (But these features are effectively available with
-;; todos-regexp-items-multifile, so maybe it's not worth the trouble here.)
 (defun todos-search ()
   "Search for a regular expression in this Todos file.
 The search runs through the whole file and encompasses all and
@@ -3569,11 +3815,21 @@ face."
   (interactive)
   (if (zerop (todos-get-count 'done (todos-current-category)))
       (message "There are no done items in this category.")
-    (save-excursion
+    (let ((opoint (point)))
       (goto-char (point-min))
-      (let ((todos-show-with-done (not (re-search-forward
-                                       todos-done-string-start nil t))))
-       (todos-category-select)))))
+      (let* ((shown (re-search-forward todos-done-string-start nil t))
+            (todos-show-with-done (not shown)))
+       (todos-category-select)
+       (goto-char opoint)
+       ;; If start of done items sections is below the bottom of the
+       ;; window, make it visible.
+       (unless shown
+         (setq shown (progn
+                       (goto-char (point-min))
+                       (re-search-forward todos-done-string-start nil t)))
+         (if (not (pos-visible-in-window-p shown))
+             (recenter)
+           (goto-char opoint)))))))
 
 (defun todos-show-done-only ()
   "Switch between displaying only done or only todo items."
@@ -3595,78 +3851,78 @@ face."
   (save-excursion
     (save-restriction
       (goto-char (point-min))
-      (let ((ovs (overlays-in (point) (1+ (point))))
-           ov hidden)
-       (while ovs
-         (setq ov (pop ovs))
-         (if (equal (overlay-get ov 'display) "")
-             (setq ovs nil hidden t)))
+      (if (todos-get-overlay 'header)
+         (remove-overlays 1 (1+ (buffer-size)) 'todos 'header)
        (widen)
        (goto-char (point-min))
-       (if hidden
-           (remove-overlays (point-min) (point-max) 'display "")
-         (while (not (eobp))
-           (when (re-search-forward
-                  (concat todos-date-string-start todos-date-pattern
-                          "\\( " diary-time-regexp "\\)?"
-                          (regexp-quote todos-nondiary-end) "? ")
-                  nil t)
-             (unless (save-match-data (todos-done-item-p))
-               (setq ov (make-overlay (match-beginning 0) (match-end 0) nil t))
-               (overlay-put ov 'display "")))
-           (todos-forward-item)))))))
-
-(defun todos-mark-unmark-item (&optional n all)
-  "Mark item at point if unmarked, or unmark it if marked.
-
+       (while (not (eobp))
+         (when (re-search-forward
+                (concat todos-date-string-start todos-date-pattern
+                        "\\( " diary-time-regexp "\\)?"
+                        (regexp-quote todos-nondiary-end) "? ")
+                nil t)
+           (unless (save-match-data (todos-done-item-p))
+             (setq ov (make-overlay (match-beginning 0) (match-end 0) nil t))
+             (overlay-put ov 'todos 'header)
+             (overlay-put ov 'display "")))
+         (todos-forward-item))))))
+
+(defun todos-mark-unmark-item (&optional n)
+  "Mark item with `todos-item-mark' if unmarked, otherwise unmark it.
 With a positive numerical prefix argument N, change the
-markedness of the next N items.  With non-nil argument ALL, mark
-all visible items in the category (depending on visibility, all
-todo and done items, or just todo or just done items).
-
-The mark is the character \"*\" inserted in front of the item's
-priority number or the `todos-prefix' string; if `todos-prefix'
-is \"*\", then the mark is \"@\"."
+marking of the next N items."
   (interactive "p")
-  (if all (goto-char (point-min)))
-  (unless (> n 0) (setq n 1))
-  (let ((i 0))
-    (while (or (and all (not (eobp)))
-              (< i n))
-      (let* ((cat (todos-current-category))
-            (ov (todos-marked-item-p))
-            (marked (assoc cat todos-categories-with-marks)))
-       (if (and ov (not all))
-           (progn
-             (delete-overlay ov)
-             (if (= (cdr marked) 1)    ; Deleted last mark in this category.
-                 (setq todos-categories-with-marks
-                       (assq-delete-all cat todos-categories-with-marks))
-               (setcdr marked (1- (cdr marked)))))
-         (when (todos-item-start)
-           (unless (and all (todos-marked-item-p))
-             (setq ov (make-overlay (point) (point)))
-             (overlay-put ov 'before-string todos-item-mark)
-             (if marked
-                 (setcdr marked (1+ (cdr marked)))
-               (push (cons cat 1) todos-categories-with-marks))))))
-      (todos-forward-item)
-      (setq i (1+ i)))))
+  (unless (> n 1) (setq n 1))
+  (dotimes (i n)
+    (let* ((cat (todos-current-category))
+          (marks (assoc cat todos-categories-with-marks))
+          (ov (todos-get-overlay 'prefix))
+          (pref (overlay-get ov 'before-string)))
+      (if (todos-marked-item-p)
+         (progn
+           (overlay-put ov 'before-string (substring pref 1))
+           (if (= (cdr marks) 1)       ; Deleted last mark in this category.
+               (setq todos-categories-with-marks
+                     (assq-delete-all cat todos-categories-with-marks))
+             (setcdr marks (1- (cdr marks)))))
+       (overlay-put ov 'before-string (concat todos-item-mark pref))
+       (if marks
+           (setcdr marks (1+ (cdr marks)))
+         (push (cons cat 1) todos-categories-with-marks))))
+    (todos-forward-item)))
 
 (defun todos-mark-category ()
-  "Put the \"*\" mark on all items in this category.
-\(If `todos-prefix' is \"*\", then the mark is \"@\".)"
+  "Mark all visiblw items in this category with `todos-item-mark'."
   (interactive)
-  (todos-mark-unmark-item 0 t))
+  (let* ((cat (todos-current-category))
+        (marks (assoc cat todos-categories-with-marks)))
+    (save-excursion
+      (goto-char (point-min))
+      (while (not (eobp))
+       (let* ((ov (todos-get-overlay 'prefix))
+              (pref (overlay-get ov 'before-string)))
+         (unless (todos-marked-item-p)
+           (overlay-put ov 'before-string (concat todos-item-mark pref))
+           (if marks
+               (setcdr marks (1+ (cdr marks)))
+             (push (cons cat 1) todos-categories-with-marks))))
+       (todos-forward-item)))))
 
 (defun todos-unmark-category ()
-  "Remove the \"*\" mark from all items in this category.
-\(If `todos-prefix' is \"*\", then the mark is \"@\".)"
+  "Remove `todos-item-mark' from all visible items in this category."
   (interactive)
-  (remove-overlays (point-min) (point-max) 'before-string todos-item-mark)
-  (setq todos-categories-with-marks
-       (delq (assoc (todos-current-category) todos-categories-with-marks)
-             todos-categories-with-marks)))
+  (let* ((cat (todos-current-category))
+        (marks (assoc cat todos-categories-with-marks)))
+    (save-excursion
+      (goto-char (point-min))
+      (while (not (eobp))
+       (let* ((ov (todos-get-overlay 'prefix))
+              ;; No overlay on empty line between todo and done items.
+              (pref (when ov (overlay-get ov 'before-string))))
+         (when (todos-marked-item-p)
+           (overlay-put ov 'before-string (substring pref 1)))
+         (todos-forward-item))))
+    (setq todos-categories-with-marks (delq marks todos-categories-with-marks))))
 
 ;; ---------------------------------------------------------------------------
 ;;; Item filtering commands
@@ -3683,85 +3939,104 @@ See `todos-set-top-priorities' for more details."
   (interactive)
   (todos-set-top-priorities t))
 
-(defun todos-top-priorities (&optional num)
-  "List top priorities of each category in `todos-filter-files'.
-Number of entries for each category is given by NUM, which
-defaults to `todos-show-priorities'."
+(defun todos-top-priorities (&optional arg)
+  "Display a list of top priority items from different categories.
+The categories can be any of those in the current Todos file.
+
+With numerical prefix ARG show at most ARG top priority items
+from each category.  With `C-u' as prefix argument show the
+numbers of top priority items specified by category in
+`todos-priorities-rules', if this has an entry for the file(s);
+otherwise show `todos-show-priorities' items per category in the
+file(s).  With no prefix argument, if a top priorities file for
+the current Todos file has previously been saved (see
+`todos-save-filtered-items-buffer'), visit this file; if there is
+no such file, build the list as with prefix argument `C-u'.
+
+  The prefix ARG regulates how many top priorities from
+each category to show, as described above."
   (interactive "P")
-  (let ((arg (if num (cons 'top num) 'top))
-       (buf todos-top-priorities-buffer)
-       (file todos-current-todos-file))
-    (todos-filter-items arg)
-    (todos-filtered-buffer-name buf file)))
+  (todos-filter-items 'top arg))
 
 (defun todos-top-priorities-multifile (&optional arg)
-  "List top priorities of each category in `todos-filter-files'.
-
-If the prefix argument ARG is a number, this is the maximum
-number of top priorities to list in each category.  If the prefix
-argument is `C-u', prompt for which files to filter and use
-`todos-show-priorities' as the number of top priorities to list
-in each category.  If the prefix argument is `C-uC-u', prompt
-both for which files to filter and for how many top priorities to
-list in each category."
+  "Display a list of top priority items from different categories.
+The categories are a subset of the categories in the files listed
+in `todos-filter-files', or if this nil, in the files chosen from
+a file selection dialog that pops up in this case.
+
+With numerical prefix ARG show at most ARG top priority items
+from each category in each file.  With `C-u' as prefix argument
+show the numbers of top priority items specified in
+`todos-priorities-rules', if this is non-nil; otherwise show
+`todos-show-priorities' items per category.  With no prefix
+argument, if a top priorities file for the chosen Todos files
+exists (see `todos-save-filtered-items-buffer'), visit this file;
+if there is no such file, do the same as with prefix argument
+`C-u'."
   (interactive "P")
-  (let* ((buf todos-top-priorities-buffer)
-        files
-        (pref (if (numberp arg)
-                  (cons 'top arg)
-                (setq files (if (or (consp arg)
-                                    (null todos-filter-files))
-                                (progn (todos-multiple-filter-files)
-                                       todos-multiple-filter-files)
-                              todos-filter-files))
-                (if (equal arg '(16))
-                    (cons 'top (read-number
-                                "Enter number of top priorities to show: "
-                                todos-show-priorities))
-                  'top))))
-    (todos-filter-items pref t)
-    (todos-filtered-buffer-name buf files)))
-
-(defun todos-diary-items ()
-  "Display todo items for diary inclusion in this Todos file."
-  (interactive)
-  (let ((buf todos-diary-items-buffer)
-       (file todos-current-todos-file))
-    (todos-filter-items 'diary)
-    (todos-filtered-buffer-name buf file)))
+  (todos-filter-items 'top arg t))
+
+(defun todos-diary-items (&optional arg)
+  "Display a list of todo diary items from different categories.
+The categories can be any of those in the current Todos file.
+
+Called with no prefix argument, if a diary items file for the
+current Todos file has previously been saved (see
+`todos-save-filtered-items-buffer'), visit this file; if there is
+no such file, build the list of diary items.  Called with a
+prefix argument, build the list even if there is a saved file of
+diary items."
+  (interactive "P")
+  (todos-filter-items 'diary arg))
 
 (defun todos-diary-items-multifile (&optional arg)
-  "Display todo items for diary inclusion in one or more Todos file.
-The files are those listed in `todos-filter-files'."
+  "Display a list of todo diary items from different categories.
+The categories are a subset of the categories in the files listed
+in `todos-filter-files', or if this nil, in the files chosen from
+a file selection dialog that pops up in this case.
+
+Called with no prefix argument, if a diary items file for the
+chosen Todos files has previously been saved (see
+`todos-save-filtered-items-buffer'), visit this file; if there is
+no such file, build the list of diary items.  Called with a
+prefix argument, build the list even if there is a saved file of
+diary items."
   (interactive "P")
-  (let ((buf todos-diary-items-buffer)
-       (files (if (or arg (null todos-filter-files))
-                  (progn (todos-multiple-filter-files)
-                         todos-multiple-filter-files)
-                todos-filter-files)))
-    (todos-filter-items 'diary t)
-    (todos-filtered-buffer-name buf files)))
-
-(defun todos-regexp-items ()
-  "Display todo items matching a user-entered regular expression.
-The items are those in the current Todos file."
-  (interactive)
-  (let ((buf todos-regexp-items-buffer)
-       (file todos-current-todos-file))
-    (todos-filter-items 'regexp)
-    (todos-filtered-buffer-name buf file)))
+  (todos-filter-items 'diary arg t))
+
+(defun todos-regexp-items (&optional arg)
+  "Prompt for a regular expression and display items that match it.
+The matches can be from any categories in the current Todos file
+and with non-nil option `todos-filter-done-items', can include
+not only todo items but also done items, including those in
+Archive files.
+
+Called with no prefix argument, if a regexp items file for the
+current Todos file has previously been saved (see
+`todos-save-filtered-items-buffer'), visit this file; if there is
+no such file, build the list of regexp items.  Called with a
+prefix argument, build the list even if there is a saved file of
+regexp items."
+  (interactive "P")
+  (todos-filter-items 'regexp arg))
 
 (defun todos-regexp-items-multifile (&optional arg)
-  "Display todo items matching a user-entered regular expression.
-The items are those in the files listed in `todos-filter-files'."
+  "Prompt for a regular expression and display items that match it.
+The matches can be from any categories in the files listed in
+`todos-filter-files', or if this nil, in the files chosen from a
+file selection dialog that pops up in this case.  With non-nil
+option `todos-filter-done-items', the matches can include not
+only todo items but also done items, including those in Archive
+files.
+
+Called with no prefix argument, if a regexp items file for the
+current Todos file has previously been saved (see
+`todos-save-filtered-items-buffer'), visit this file; if there is
+no such file, build the list of regexp items.  Called with a
+prefix argument, build the list even if there is a saved file of
+regexp items."
   (interactive "P")
-  (let ((buf todos-regexp-items-buffer)
-       (files (if (or arg (null todos-filter-files))
-                  (progn (todos-multiple-filter-files)
-                         todos-multiple-filter-files)
-                todos-filter-files)))
-    (todos-filter-items 'regexp t)
-    (todos-filtered-buffer-name buf files)))
+  (todos-filter-items 'regexp arg t))
 
 ;; ---------------------------------------------------------------------------
 ;;; Editing Commands
@@ -3815,7 +4090,7 @@ return the new category number."
            file0 (if (called-interactively-p 'any)
                      (cdr catfil)
                    file)))
-    (find-file file0)          ;FIXME:? find-file-noselect, set-buffer etc.
+    (find-file file0)
     (let ((counts (make-vector 4 0))   ; [todo diary done archived]
          (num (1+ (length todos-categories)))
          (buffer-read-only nil))
@@ -3890,7 +4165,7 @@ i.e. including all existing todo and done items."
                                     "the archived category will remain\n"
                                     "after deleting the todo category.  "
                                     "Do you still want to delete it\n"
-                                    "(see 'todos-skip-archived-categories' "
+                                    "(see `todos-skip-archived-categories' "
                                     "for another option)? ")))
                  (t
                   (y-or-n-p (concat "Permanently remove category \"" cat
@@ -4000,10 +4275,9 @@ archive of the file moved to, creating it if it does not exist."
              (setq todos-categories
                    (append todos-categories (list (cons new counts))))
              (todos-update-categories-sexp)
-             ;; If archive was just created, save it to avoid "File <xyz> no
-             ;; longer exists!" message on invoking
-             ;; `todos-view-archived-items'.  FIXME: maybe better to save
-             ;; unconditionally?
+             ;; If archive was just created, save it to avoid "File
+             ;; <xyz> no longer exists!" message on invoking
+             ;; `todos-view-archived-items'.
              (unless (file-exists-p (buffer-file-name))
                (save-buffer))
              (todos-category-number (or new cat))
@@ -4239,7 +4513,6 @@ raise or lower the category's priority by one."
 ;; ---------------------------------------------------------------------------
 ;;; Item editing commands
 
-;; FIXME: make insertion options customizable per category?
 ;;;###autoload
 (defun todos-insert-item (&optional arg diary nonmarking date-type time
                                    region-or-here)
@@ -4301,12 +4574,15 @@ omit the current time string according as
 The argument REGION-OR-HERE determines the source and location of
 the new item:
 - If the REGION-OR-HERE is the symbol `here', prompt for the text
-  of the new item and insert it directly above the todo item at
+  of the new item and, if the command was invoked in the current
+  category, insert it directly above the todo item at
   point (hence lowering the priority of the remaining items), or
   if point is on the empty line below the last todo item, insert
-  the new item there.  An error is signalled if
-  `todos-insert-item' is invoked with `here' outside of the
-  current category.
+  the new item there.  If point is in the done items section of
+  the category, insert the new item as the first todo item in the
+  category.  Likewise, if the command with `here' is invoked
+  outside of the current category, jump to the chosen category
+  and insert the new item as the first item in the category.
 - If REGION-OR-HERE is the symbol `region', use the region of the
   current buffer as the text of the new item, depending on the
   value of user option `todos-use-only-highlighted-region': if
@@ -4346,7 +4622,10 @@ the priority is not given by HERE but by prompting."
        (let (use-empty-active-region)
          (unless (and todos-use-only-highlighted-region (use-region-p))
            (error "There is no active region"))))
-      (let* ((buf (current-buffer))
+      (let* ((obuf (current-buffer))
+            (ocat (todos-current-category))
+            (opoint (point))
+            (todos-mm (eq major-mode 'todos-mode))
             (cat+file (cond ((equal arg '(4))
                              (todos-read-category "Insert in category: "))
                             ((equal arg '(16))
@@ -4355,6 +4634,8 @@ the priority is not given by HERE but by prompting."
                             (t
                              (cons (todos-current-category)
                                    (or todos-current-todos-file
+                                       (and todos-show-current-file
+                                            todos-global-current-todos-file)
                                        (todos-absolute-file-name
                                         todos-default-todos-file))))))
             (cat (car cat+file))
@@ -4379,29 +4660,27 @@ the priority is not given by HERE but by prompting."
                            (setq todos-date-from-calendar date-type)
                            (todos-set-date-from-calendar))
                           (t
-                           ;; FIXME: We follow diary-insert-entry in
-                           ;; hardcoding abbreviated month name and no
-                           ;; day name in date string.  Should this be
-                           ;; customizable?
                            (calendar-date-string (calendar-current-date) t t))))
             (time-string (or (and time (todos-read-time))
                              (and todos-always-add-time-string
                                   (substring (current-time-string) 11 16)))))
        (setq todos-date-from-calendar nil)
        (find-file-noselect file 'nowarn)
-       (setq todos-current-todos-file file)
        (set-window-buffer (selected-window)
-                          ;; If current category was nil till now, on
-                          ;; entering Todos mode here it will be set to
-                          ;; file's first category.
                           (set-buffer (find-buffer-visiting file)))
+       ;; If this command was invoked outside of a Todos buffer, the
+       ;; call to todos-current-category above returned nil.  If we
+       ;; just entered Todos mode now, then cat was set to the file's
+       ;; first category, but if todos-mode was already enabled, cat
+       ;; did not get set, so we have to set it explicitly.
+       (unless cat
+         (setq cat (todos-current-category)))
+       (setq todos-current-todos-file file)
        (unless todos-global-current-todos-file
          (setq todos-global-current-todos-file todos-current-todos-file))
-       ;; These are not needed here, since they are called in
-       ;; todos-set-item-priority.
-       ;; (todos-category-number cat)
-       ;; (todos-category-select)
-       (let (buffer-read-only)
+       (let ((buffer-read-only nil)
+             (called-from-outside (not (and todos-mm (equal cat ocat))))
+             done-only item-added)
          (setq new-item
                ;; Add date, time and diary marking as required.
                (concat (if (not (and diary (not todos-include-in-diary)))
@@ -4419,24 +4698,58 @@ the priority is not given by HERE but by prompting."
                          "\\(\n\\)[^[:blank:]]"
                          (concat "\n" (make-string todos-indent-to-here 32))
                          new-item nil nil 1))
-         ;; FIXME: after jumping to another category due to `C-u i h',
-         ;; item is inserted as first item -- ok?
-         (if here
-             (cond ((not (eq major-mode 'todos-mode))
-                    (error "Cannot insert a todo item here outside of Todos mode"))
-                   ((not (eq buf (current-buffer)))
-                    (error "Cannot insert an item here after changing buffer"))
-                   ((or (todos-done-item-p)
-                        ;; Point on last blank line.
-                        (save-excursion (forward-line -1) (todos-done-item-p)))
-                    (error "Cannot insert a new item in the done item section"))
-                   (t
-                    (todos-insert-with-overlays new-item)))
-           ;; (todos-set-item-priority new-item (todos-current-category) t))
-           (todos-set-item-priority new-item cat t)
-           ;; If item is inserted at end of category, make sure the
-           ;; items above it are displayed in the window.
-           (recenter))
+         (unwind-protect
+             (progn
+               ;; Make sure the correct category is selected.  There
+               ;; are two cases: (i) we just visited the file, so no
+               ;; category is selected yet, or (ii) we invoked
+               ;; insertion "here" from outside the category we want
+               ;; to insert in (with priority insertion, category
+               ;; selection is done by todos-set-item-priority).
+               (when (or (= (- (point-max) (point-min)) (buffer-size))
+                         (and here called-from-outside))
+                 (todos-category-number cat)
+                 (todos-category-select))
+               ;; If only done items are displayed in category,
+               ;; toggle to todo items before inserting new item.
+               (when (save-excursion
+                       (goto-char (point-min))
+                       (looking-at todos-done-string-start))
+                 (setq done-only t)
+                 (todos-show-done-only))
+               (if here
+                   (progn
+                     ;; If command was invoked with point in done
+                     ;; items section or outside of the current
+                     ;; category, can't insert "here", so to be
+                     ;; useful give new item top priority.
+                     (when (or (todos-done-item-section-p)
+                               called-from-outside
+                               done-only)
+                       (goto-char (point-min)))
+                     (todos-insert-with-overlays new-item))
+                 (todos-set-item-priority new-item cat t))
+               (setq item-added t))
+           ;; If user cancels before setting priority, restore
+           ;; display.
+           (unless item-added
+             (if ocat
+                 (progn
+                   (unless (equal cat ocat)
+                     (todos-category-number ocat)
+                     (todos-category-select))
+                   (and done-only (todos-show-done-only)))
+               (set-window-buffer (selected-window) (set-buffer obuf)))
+             (goto-char opoint))
+           ;; If the todo items section is not visible when the
+           ;; insertion command is called (either because only done
+           ;; items were shown or because the category was not in the
+           ;; current buffer), then if the item is inserted at the
+           ;; end of the category, point is at eob and eob at
+           ;; window-start, so that higher priority todo items are
+           ;; out of view.  So we recenter to make sure the todo
+           ;; items are displayed in the window.
+           (when item-added (recenter)))
          (todos-update-count 'todo 1)
          (if (or diary todos-include-in-diary) (todos-update-count 'diary 1))
          (todos-update-categories-sexp))))))
@@ -4472,7 +4785,6 @@ the priority is not given by HERE but by prompting."
                                      (calendar-exit)
                                      (exit-recursive-edit))))
         (message "Put cursor on a date and type <return> to set it.")
-        ;; FIXME: is there a better way than recursive-edit?
         (recursive-edit)
         (unwind-protect
             (when (equal (buffer-name) calendar-buffer)
@@ -4497,7 +4809,6 @@ the item at point."
        (let* ((cat (todos-current-category))
               (marked (assoc cat todos-categories-with-marks))
               (item (unless marked (todos-item-string)))
-              ;; FIXME: make confirmation an option?
               (answer (if marked
                           (y-or-n-p "Permanently delete all marked items? ")
                         (when item
@@ -4506,7 +4817,6 @@ the item at point."
                                     (save-excursion (todos-item-end))))
                           (overlay-put ov 'face 'todos-search)
                           (y-or-n-p (concat "Permanently delete this item? ")))))
-              (opoint (point))
               buffer-read-only)
          (when answer
            (and marked (goto-char (point-min)))
@@ -4527,11 +4837,8 @@ the item at point."
                        (throw 'done (setq item nil))))
                  (todos-forward-item))))
            (when marked
-             (remove-overlays (point-min) (point-max)
-                              'before-string todos-item-mark)
              (setq todos-categories-with-marks
-                   (assq-delete-all cat todos-categories-with-marks))
-             (goto-char opoint))
+                   (assq-delete-all cat todos-categories-with-marks)))
            (todos-update-categories-sexp)
            (todos-prefix-overlays)))
       (if ov (delete-overlay ov)))))
@@ -4572,7 +4879,7 @@ minibuffer; otherwise, edit it in Todos Edit mode."
                                              todos-date-pattern) new))
              (setq new (read-from-minibuffer
                         "Item must start with a date: " new))))
-         ;; Indent newlines inserted by C-q C-j if nonspace char follows.
+         ;; Ensure lines following hard newlines are indented.
          (setq new (replace-regexp-in-string
                     "\\(\n\\)[^[:blank:]]"
                     (concat "\n" (make-string todos-indent-to-here 32)) new
@@ -4588,45 +4895,71 @@ minibuffer; otherwise, edit it in Todos Edit mode."
 Use of newlines invokes `todos-indent' to insure compliance with
 the format of Diary entries."
   (interactive)
-  (todos-edit-multiline t))
-
-(defun todos-edit-multiline (&optional item)
-  ""
-  (interactive)
-  ;; FIXME: should there be only one live Todos Edit buffer?
-  ;; (let ((buffer-name todos-edit-buffer))
-  (let ((buffer-name (generate-new-buffer-name todos-edit-buffer)))
-    (set-window-buffer
-     (selected-window)
-     (set-buffer (make-indirect-buffer
-                 (file-name-nondirectory todos-current-todos-file)
-                 buffer-name)))
-    (if item
-       (narrow-to-region (todos-item-start) (todos-item-end))
-      (widen))
+  (let ((buf todos-edit-buffer))
+    (set-window-buffer (selected-window)
+                      (set-buffer (make-indirect-buffer (buffer-name) buf)))
+    (narrow-to-region (todos-item-start) (todos-item-end))
     (todos-edit-mode)
     (message "%s" (substitute-command-keys
-                  (concat "Type \\[todos-edit-quit] to check file format "
-                          "validity and return to Todos mode.\n")))))
+                  (concat "Type \\[todos-edit-quit] "
+                          "to return to Todos mode.\n")))))
+
+(defun todos-edit-multiline (&optional item) ;FIXME: not item editing command
+  ""                                   ;FIXME
+  (interactive)
+  (widen)
+  (todos-edit-mode)
+  (remove-overlays)
+  (message "%s" (substitute-command-keys
+                (concat "Type \\[todos-edit-quit] to check file format "
+                        "validity and return to Todos mode.\n"))))
 
 (defun todos-edit-quit ()
   "Return from Todos Edit mode to Todos mode.
+If the item contains hard line breaks, make sure the following
+lines are indented by `todos-indent-to-here' to conform to diary
+format.
 
 If the whole file was in Todos Edit mode, check before returning
 whether the file is still a valid Todos file and if so, also
 recalculate the Todos categories sexp, in case changes were made
 in the number or names of categories."
   (interactive)
-  ;; FIXME: Should do todos-check-format only if file was actually changed --
-  ;; but how to tell?
-  (when (eq (buffer-size) (- (point-max) (point-min)))
-    (when (todos-check-format) (todos-repair-categories-sexp)))
-  (kill-buffer)
-  ;; In case next buffer is not the one holding todos-current-todos-file.
-  (todos-show))
+  (if (> (buffer-size) (- (point-max) (point-min)))
+      (let ((item (buffer-string))
+           (regex "\\(\n\\)[^[:blank:]]"))
+       ;; Ensure lines following hard newlines are indented.
+       (when (string-match regex (buffer-string))
+         (replace-regexp-in-string
+          regex (concat "\n" (make-string todos-indent-to-here 32))
+          nil nil 1)
+         (delete-region (point-min) (point-max))
+         (insert item))
+       (kill-buffer))
+      (when (todos-check-format)
+       ;; FIXME: separate out sexp check?
+       ;; If manual editing makes e.g. item counts change, have to
+       ;; call this to update todos-categories, but it restores
+       ;; category order to list order.
+       ;; (todos-repair-categories-sexp)
+       ;; Compare (todos-make-categories-list t) with sexp and if
+       ;; different ask (todos-update-categories-sexp) ?
+       (todos-mode)
+       (let* ((cat-beg (concat "^" (regexp-quote todos-category-beg)
+                               "\\(.*\\)$"))
+              (curline (buffer-substring-no-properties
+                        (line-beginning-position) (line-end-position)))
+              (cat (cond ((string-match cat-beg curline)
+                          (match-string-no-properties 1 curline))
+                         ((or (re-search-backward cat-beg nil t)
+                              (re-search-forward cat-beg nil t))
+                          (match-string-no-properties 1)))))
+         (todos-category-number cat)
+         (todos-category-select)
+         (goto-char (point-min))))))
 
 (defun todos-edit-item-header-1 (what &optional inc)
-  "Underlying function to edit items' date/time headers.
+  "Function underlying commands to edit item date/time header.
 
 The argument WHAT (passed by invoking commands) specifies what
 part of the header to edit; possible values are these symbols:
@@ -4717,7 +5050,7 @@ otherwise, edit just the item at point."
                                  (todos-read-date 'year))
                                 ((string= oyear "*")
                                  (error "Cannot increment *"))
-                                (t     ; FIXME: handle negative years
+                                (t
                                  (number-to-string (+ yy inc))))))
               ((eq what 'month)
                (setf day oday
@@ -4986,112 +5319,130 @@ items in this category."
                (insert diary-nonmarking-symbol))))
        (todos-forward-item)))))))
 
-;; FIXME: Make NOP if point isn't on a todo item (cf. todos-copy-item,
-;; todos-move-item
 (defun todos-set-item-priority (&optional item cat new arg)
-  "Set todo ITEM's priority in CATegory and move item accordingly.
+  "Prompt for and set ITEM's priority in CATegory.
 
-Interactively, ITEM defaults to the item at point, CAT to the
-current category in Todos mode, and the priority is a number
-between 1 and the number of items in the category.
-Non-interactively, non-nil NEW means ITEM is a new item and the
-lowest priority is one more than the number of items in CAT.
+Interactively, ITEM is the todo item at point, CAT is the current
+category, and the priority is a number between 1 and the number
+of items in the category.  Non-interactively, non-nil NEW means
+ITEM is a new item and the lowest priority is one more than the
+number of items in CAT.
 
 The new priority is set either interactively by prompt or by a
 numerical prefix argument, or noninteractively by argument ARG,
 whose value can be either of the symbols `raise' or `lower',
 meaning to raise or lower the item's priority by one."
-  (interactive)
-  (let* ((item (or item (todos-item-string)))
-        (marked (todos-marked-item-p))
-        (cat (or cat (cond ((eq major-mode 'todos-mode)
-                            (todos-current-category))
-                           ((eq major-mode 'todos-filtered-items-mode)
-                            (let* ((regexp1
-                                    (concat todos-date-string-start
-                                            todos-date-pattern
-                                            "\\( " diary-time-regexp "\\)?"
-                                            (regexp-quote todos-nondiary-end)
-                                            "?\\(?1: \\[\\(.+:\\)?.+\\]\\)")))
-                              (save-excursion
-                                (re-search-forward regexp1 nil t)
-                                (match-string-no-properties 1)))))))
-        curnum
-        (todo (cond ((or (eq arg 'raise) (eq arg 'lower)
-                         (eq major-mode 'todos-filtered-items-mode))
-                     (save-excursion
-                       (let ((curstart (todos-item-start))
-                             (count 0))
-                         (goto-char (point-min))
-                         (while (looking-at todos-item-start)
-                           (setq count (1+ count))
-                           (when (= (point) curstart) (setq curnum count))
-                           (todos-forward-item))
-                         count)))
-                    ((eq major-mode 'todos-mode)
-                     (todos-get-count 'todo cat))))
-        (maxnum (if new (1+ todo) todo))
-        (prompt (format "Set item priority (1-%d): " maxnum))
-        (priority (cond ((numberp current-prefix-arg)
-                         current-prefix-arg)
-                        ((and (eq arg 'raise) (>= curnum 1))
-                         (1- curnum))
-                        ((and (eq arg 'lower) (<= curnum maxnum))
-                         (1+ curnum))))
-        candidate
-        buffer-read-only)
-    (unless (and priority
-                (or (and (eq arg 'raise) (zerop priority))
-                    (and (eq arg 'lower) (> priority maxnum))))
-      ;; When moving item to another category, show the category before
-      ;; prompting for its priority.
-      (unless (or arg (called-interactively-p t))
-       (todos-category-number cat)
-       (todos-category-select))
-      ;; Prompt for priority only when the category has at least one todo item.
-      (when (> maxnum 1)
-       (while (not priority)
-         (setq candidate (read-number prompt))
-         (setq prompt (when (or (< candidate 1) (> candidate maxnum))
-                        (format "Priority must be an integer between 1 and %d.\n"
-                                maxnum)))
-         (unless prompt (setq priority candidate))))
-      ;; In Top Priorities buffer, an item's priority can be changed
-      ;; wrt items in another category, but not wrt items in the same
-      ;; category.
-      (when (eq major-mode 'todos-filtered-items-mode)
-       (let* ((regexp2 (concat todos-date-string-start todos-date-pattern
-                               "\\( " diary-time-regexp "\\)?"
-                               (regexp-quote todos-nondiary-end)
-                               "?\\(?1:" (regexp-quote cat) "\\)"))
-              (end (cond ((< curnum priority)
-                          (save-excursion (todos-item-end)))
-                         ((> curnum priority)
-                          (save-excursion (todos-item-start)))))
-              (match (save-excursion
-                       (cond ((< curnum priority)
-                              (todos-forward-item (1+ (- priority curnum)))
-                              (when (re-search-backward regexp2 end t)
-                                (match-string-no-properties 1)))
-                             ((> curnum priority)
-                              (todos-backward-item (- curnum priority))
-                              (when (re-search-forward regexp2 end t)
-                                (match-string-no-properties 1)))))))
-         (when match
-           (error (concat "Cannot reprioritize items from the same "
-                          "category in this mode, only in Todos mode")))))
-      ;; Interactively or with non-nil ARG, relocate the item within its
-      ;; category.
-      (when (or arg (called-interactively-p))
-       (todos-remove-item))
-      (goto-char (point-min))
-      (when priority
-       (unless (= priority 1)
-         (todos-forward-item (1- priority))))
-      (todos-insert-with-overlays item)
-      ;; If item was marked, restore the mark.
-      (and marked (overlay-put (make-overlay (point) (point))
-                              'before-string todos-item-mark)))))
+  (interactive)                                ;FIXME: Prefix arg?
+  (unless (and (called-interactively-p 'any)
+              (or (todos-done-item-p) (looking-at "^$")))
+    (let* ((item (or item (todos-item-string)))
+          (marked (todos-marked-item-p))
+          (cat (or cat (cond ((eq major-mode 'todos-mode)
+                              (todos-current-category))
+                             ((eq major-mode 'todos-filtered-items-mode)
+                              (let* ((regexp1
+                                      (concat todos-date-string-start
+                                              todos-date-pattern
+                                              "\\( " diary-time-regexp "\\)?"
+                                              (regexp-quote todos-nondiary-end)
+                                              "?\\(?1: \\[\\(.+:\\)?.+\\]\\)")))
+                                (save-excursion
+                                  (re-search-forward regexp1 nil t)
+                                  (match-string-no-properties 1)))))))
+          curnum
+          (todo (cond ((or (eq arg 'raise) (eq arg 'lower)
+                           (eq major-mode 'todos-filtered-items-mode))
+                       (save-excursion
+                         (let ((curstart (todos-item-start))
+                               (count 0))
+                           (goto-char (point-min))
+                           (while (looking-at todos-item-start)
+                             (setq count (1+ count))
+                             (when (= (point) curstart) (setq curnum count))
+                             (todos-forward-item))
+                           count)))
+                      ((eq major-mode 'todos-mode)
+                       (todos-get-count 'todo cat))))
+          (maxnum (if new (1+ todo) todo))
+          (prompt (format "Set item priority (1-%d): " maxnum))
+          (priority (cond ((and (not arg) (numberp current-prefix-arg))
+                           current-prefix-arg)
+                          ((and (eq arg 'raise) (>= curnum 1))
+                           (1- curnum))
+                          ((and (eq arg 'lower) (<= curnum maxnum))
+                           (1+ curnum))))
+          candidate
+          buffer-read-only)
+      (unless (and priority
+                  (or (and (eq arg 'raise) (zerop priority))
+                      (and (eq arg 'lower) (> priority maxnum))))
+       ;; When moving item to another category, show the category before
+       ;; prompting for its priority.
+       (unless (or arg (called-interactively-p 'any))
+         (todos-category-number cat)
+         ;; If done items in category are visible, keep them visible.
+         (let ((done todos-show-with-done))
+           (when (> (buffer-size) (- (point-max) (point-min)))
+             (save-excursion
+               (goto-char (point-min))
+               (setq done (re-search-forward todos-done-string-start nil t))))
+           (let ((todos-show-with-done done))
+             (todos-category-select)
+             ;; Keep top of category in view while setting priority.
+             (goto-char (point-min)))))
+       ;; Prompt for priority only when the category has at least one todo item.
+       (when (> maxnum 1)
+         (while (not priority)
+           (setq candidate (read-number prompt))
+           (setq prompt (when (or (< candidate 1) (> candidate maxnum))
+                          (format "Priority must be an integer between 1 and %d.\n"
+                                  maxnum)))
+           (unless prompt (setq priority candidate))))
+       ;; In Top Priorities buffer, an item's priority can be changed
+       ;; wrt items in another category, but not wrt items in the same
+       ;; category.
+       (when (eq major-mode 'todos-filtered-items-mode)
+         (let* ((regexp2 (concat todos-date-string-start todos-date-pattern
+                                 "\\( " diary-time-regexp "\\)?"
+                                 (regexp-quote todos-nondiary-end)
+                                 "?\\(?1:" (regexp-quote cat) "\\)"))
+                (end (cond ((< curnum priority)
+                            (save-excursion (todos-item-end)))
+                           ((> curnum priority)
+                            (save-excursion (todos-item-start)))))
+                (match (save-excursion
+                         (cond ((< curnum priority)
+                                (todos-forward-item (1+ (- priority curnum)))
+                                (when (re-search-backward regexp2 end t)
+                                  (match-string-no-properties 1)))
+                               ((> curnum priority)
+                                (todos-backward-item (- curnum priority))
+                                (when (re-search-forward regexp2 end t)
+                                  (match-string-no-properties 1)))))))
+           (when match
+             (error (concat "Cannot reprioritize items from the same "
+                            "category in this mode, only in Todos mode")))))
+       ;; Interactively or with non-nil ARG, relocate the item within its
+       ;; category.
+       (when (or arg (called-interactively-p 'any))
+         (todos-remove-item))
+       (goto-char (point-min))
+       (when priority
+         (unless (= priority 1)
+           (todos-forward-item (1- priority))
+           ;; When called from todos-item-undo and the highest priority
+           ;; is chosen, this advances point to the first done item, so
+           ;; move it up to the empty line above the done items
+           ;; separator.
+           (when (looking-back (concat "^"
+                                       (regexp-quote todos-category-done) "\n"))
+             (todos-backward-item))))
+       (todos-insert-with-overlays item)
+       ;; If item was marked, restore the mark.
+       (and marked
+            (let* ((ov (todos-get-overlay 'prefix))
+                   (pref (overlay-get ov 'before-string)))
+              (overlay-put ov 'before-string (concat todos-item-mark pref))))))))
 
 (defun todos-raise-item-priority ()
   "Raise priority of current item by moving it up by one item."
@@ -5104,69 +5455,103 @@ meaning to raise or lower the item's priority by one."
   (todos-set-item-priority nil nil nil 'lower))
 
 (defun todos-move-item (&optional file)
-  "Move at least one todo item to another category.
-
+  "Move at least one todo or done item to another category.
 If there are marked items, move all of these; otherwise, move
 the item at point.
 
 With prefix argument FILE, prompt for a specific Todos file and
 choose (with TAB completion) a category in it to move the item or
 items to; otherwise, choose and move to any category in either
-the current Todos file or a file in `todos-category-completions-files'.
-
-If the chosen category is not one of the existing categories,
-then it is created and the item(s) become(s) the first
-entry/entries in that category."
+the current Todos file or one of the files in
+`todos-category-completions-files'.  If the chosen category is
+not an existing categories, then it is created and the item(s)
+become(s) the first entry/entries in that category.
+
+With moved Todo items, prompt to set the priority in the category
+moved to (with multiple todos items, the one that had the highest
+priority in the category moved from gets the new priority and the
+rest of the moved todo items are inserted in sequence below it).
+Moved done items are appended to the top of the done items
+section in the category moved to."
   (interactive "P")
   (let* ((cat1 (todos-current-category))
         (marked (assoc cat1 todos-categories-with-marks)))
-    (unless (or (todos-done-item-p)
-               ;; Point is between todo and done items.
-               (and (looking-at "^$") (not marked)))
+    ;; Noop if point is not on an item and there are no marked items.
+    (unless (and (looking-at "^$")
+                (not marked))
       (let* ((buffer-read-only)
             (file1 todos-current-todos-file)
             (num todos-category-number)
             (item (todos-item-string))
             (diary-item (todos-diary-item-p))
+            (done-item (and (todos-done-item-p) (concat item "\n")))
             (omark (save-excursion (todos-item-start) (point-marker)))
-            (count 0)
-            (count-diary 0)
-            ov cat+file cat2 file2 moved nmark)
+            (todo 0)
+            (diary 0)
+            (done 0)
+            ov cat2 file2 moved nmark todo-items done-items)
        (unwind-protect
            (progn
              (unless marked
                (setq ov (make-overlay (save-excursion (todos-item-start))
                                       (save-excursion (todos-item-end))))
                (overlay-put ov 'face 'todos-search))
-             (setq cat+file (let ((pl (if (and marked (> (cdr marked) 1))
-                                          "s" "")))
-                              (todos-read-category (concat "Move item" pl
+             (let* ((pl (if (and marked (> (cdr marked) 1)) "s" ""))
+                    (cat+file (todos-read-category (concat "Move item" pl
                                                            " to category: ")
-                                                   nil file))
-                   cat2 (car cat+file)
-                   file2 (cdr cat+file)))
+                                                   nil file)))
+               (while (and (equal (car cat+file) cat1)
+                           (equal (cdr cat+file) file1))
+                 (setq cat+file (todos-read-category
+                                 "Choose a different category: ")))
+               (setq cat2 (car cat+file)
+                     file2 (cdr cat+file))))
          (if ov (delete-overlay ov)))
        (set-buffer (find-buffer-visiting file1))
        (if marked
            (progn
-             (setq item nil)
              (goto-char (point-min))
              (while (not (eobp))
                (when (todos-marked-item-p)
-                 (setq item (concat item (todos-item-string) "\n"))
-                 (setq count (1+ count))
-                 (when (todos-diary-item-p)
-                   (setq count-diary (1+ count-diary))))
+                 (if (todos-done-item-p)
+                     (setq done-items (concat done-items
+                                              (todos-item-string) "\n")
+                           done (1+ done))
+                   (setq todo-items (concat todo-items
+                                            (todos-item-string) "\n")
+                         todo (1+ todo))
+                   (when (todos-diary-item-p)
+                     (setq diary (1+ diary)))))
                (todos-forward-item))
-             ;; Chop off last newline.
-             (setq item (substring item 0 -1)))
-         (setq count 1)
-         (when (todos-diary-item-p) (setq count-diary 1)))
+             ;; Chop off last newline of multiple todo item string,
+             ;; since it will be reinserted when setting priority
+             ;; (but with done items priority is not set, so keep
+             ;; last newline).
+             (and todo-items
+                  (setq todo-items (substring todo-items 0 -1))))
+         (if (todos-done-item-p)
+             (setq done 1)
+           (setq todo 1)
+           (when (todos-diary-item-p) (setq diary 1))))
        (set-window-buffer (selected-window)
                           (set-buffer (find-file-noselect file2 'nowarn)))
        (unwind-protect
            (progn
-             (todos-set-item-priority item cat2 t)
+             (when (or todo-items (and item (not done-item)))
+               (todos-set-item-priority (or todo-items item) cat2 t))
+             ;; Move done items en bloc to top of done items section.
+             (when (or done-items done-item)
+               (todos-category-number cat2)
+               (widen)
+               (goto-char (point-min))
+               (re-search-forward (concat "^" (regexp-quote
+                                               (concat todos-category-beg cat2))
+                                          "$")
+                                  nil t)
+               (re-search-forward
+                (concat "^" (regexp-quote todos-category-done)) nil t)
+               (forward-line)
+               (insert (or done-items done-item)))
              (setq moved t))
          (cond
           ;; Move succeeded, so remove item from starting category,
@@ -5174,8 +5559,9 @@ entry/entries in that category."
           ;; the moved item.
           (moved
            (setq nmark (point-marker))
-           (todos-update-count 'todo count)
-           (todos-update-count 'diary count-diary)
+           (when todo (todos-update-count 'todo todo))
+           (when diary (todos-update-count 'diary diary))
+           (when done (todos-update-count 'done done))
            (todos-update-categories-sexp)
            (with-current-buffer (find-buffer-visiting file1)
              (save-excursion
@@ -5189,51 +5575,44 @@ entry/entries in that category."
                         (concat "^" (regexp-quote todos-category-beg)) nil t)
                        (forward-line)
                        (setq beg (point))
-                       (re-search-forward
-                        (concat "^" (regexp-quote todos-category-done)) nil t)
-                       (setq end (match-beginning 0))
+                       (setq end (if (re-search-forward
+                                      (concat "^" (regexp-quote
+                                                   todos-category-beg)) nil t)
+                                     (match-beginning 0)
+                                   (point-max)))
                        (goto-char beg)
                        (while (< (point) end)
                          (if (todos-marked-item-p)
                              (todos-remove-item)
                            (todos-forward-item)))
-                       ;; FIXME: does this work?
-                       (remove-overlays (point-min) (point-max)
-                                        'before-string todos-item-mark)
                        (setq todos-categories-with-marks
                              (assq-delete-all cat1 todos-categories-with-marks)))
                    (if ov (delete-overlay ov))
                    (todos-remove-item))))
-             (todos-update-count 'todo (- count) cat1)
-             (todos-update-count 'diary (- count-diary) cat1)
+             (when todo (todos-update-count 'todo (- todo) cat1))
+             (when diary (todos-update-count 'diary (- diary) cat1))
+             (when done (todos-update-count 'done (- done) cat1))
              (todos-update-categories-sexp))
            (set-window-buffer (selected-window)
                               (set-buffer (find-file-noselect file2 'nowarn)))
            (setq todos-category-number (todos-category-number cat2))
-           (todos-category-select)
+           (let ((todos-show-with-done (or done-items done-item)))
+             (todos-category-select))
            (goto-char nmark)
-           ;; If item is moved to end of category, make sure the
-           ;; items above it are displayed in the window.
+           ;; If item is moved to end of (just first?) category, make
+           ;; sure the items above it are displayed in the window.
            (recenter))
-          ;; User quit before moving, so return to starting category.
+          ;; User quit before setting priority of todo item(s), so
+          ;; return to starting category.
           (t
+           (set-window-buffer (selected-window)
+                              (set-buffer (find-file-noselect file1 'nowarn)))
            (todos-category-number cat1)
            (todos-category-select)
            (goto-char omark))))))))
 
-;; (defun todos-move-item-to-diary ()
-;;   "Move one or more items in current category to the diary file.
-;;
-;; If there are marked items, move all of these; otherwise, move
-;; the item at point."
-;;   (interactive)
-;;   ;; FIXME
-;;   )
-
-;; FIXME: make adding date customizable, and make this and time customization
-;; overridable via double prefix arg ??
 (defun todos-item-done (&optional arg)
-  "Tag at least one item in this category as done and hide it.
+  "Tag a todo item in this category as done and relocate it.
 
 With prefix argument ARG prompt for a comment and append it to
 the done item; this is only possible if there are no marked
@@ -5241,7 +5620,9 @@ items.  If there are marked items, tag all of these with
 `todos-done-string' plus the current date and, if
 `todos-always-add-time-string' is non-nil, the current time;
 otherwise, just tag the item at point.  Items tagged as done are
-relocated to the category's (by default hidden) done section."
+relocated to the category's (by default hidden) done section.  If
+done items are visible on invoking this command, they remain
+visible."
   (interactive "P")
   (let* ((cat (todos-current-category))
         (marked (assoc cat todos-categories-with-marks)))
@@ -5254,25 +5635,27 @@ relocated to the category's (by default hidden) done section."
                            ""))
             (done-prefix (concat "[" todos-done-string date-string time-string
                                  "] "))
-            (comment (and arg (not marked) (read-string "Enter a comment: ")))
+            (comment (and arg (read-string "Enter a comment: ")))
             (item-count 0)
             (diary-count 0)
-            item done-item
-            (buffer-read-only))
+            (show-done (save-excursion
+                         (goto-char (point-min))
+                         (re-search-forward todos-done-string-start nil t)))
+            (buffer-read-only nil)
+            item done-item opoint)
+       ;; Don't add empty comment to done item.
+       (setq comment (unless (zerop (length comment))
+                       (concat " [" todos-comment-string ": " comment "]")))
        (and marked (goto-char (point-min)))
        (catch 'done
-         (while (not (eobp))
+         ;; Stop looping when we hit the empty line below the last
+         ;; todo item (this is eobp if only done items are hidden).
+         (while (not (looking-at "^$")) ;(not (eobp))
            (if (or (not marked) (and marked (todos-marked-item-p)))
                (progn
                  (setq item (todos-item-string))
-                 (setq done-item (cond (marked
-                                        (concat done-item done-prefix item "\n"))
-                                       (comment
-                                        (concat done-prefix item " ["
-                                                todos-comment-string
-                                                ": " comment "]"))
-                                       (t
-                                        (concat done-prefix item))))
+                 (setq done-item (concat done-item done-prefix item
+                                         comment (and marked "\n")))
                  (setq item-count (1+ item-count))
                  (when (todos-diary-item-p)
                    (setq diary-count (1+ diary-count)))
@@ -5282,7 +5665,6 @@ relocated to the category's (by default hidden) done section."
        (when marked
          ;; Chop off last newline of done item string.
          (setq done-item (substring done-item 0 -1))
-         (remove-overlays (point-min) (point-max) 'before-string todos-item-mark)
          (setq todos-categories-with-marks
                (assq-delete-all cat todos-categories-with-marks)))
        (save-excursion
@@ -5290,12 +5672,16 @@ relocated to the category's (by default hidden) done section."
          (re-search-forward
           (concat "^" (regexp-quote todos-category-done)) nil t)
          (forward-char)
+         (when show-done (setq opoint (point)))
          (insert done-item "\n"))
        (todos-update-count 'todo (- item-count))
        (todos-update-count 'done item-count)
        (todos-update-count 'diary (- diary-count))
        (todos-update-categories-sexp)
-       (save-excursion (todos-category-select))))))
+       (let ((todos-show-with-done show-done))
+         (todos-category-select)
+         ;; When done items are shown, put cursor on first just done item.
+         (when opoint (goto-char opoint)))))))
 
 (defun todos-done-item-add-edit-or-delete-comment (&optional arg)
   "Add a comment to this done item or edit an existing comment.
@@ -5303,6 +5689,7 @@ With prefix ARG delete an existing comment."
   (interactive "P")
   (when (todos-done-item-p)
     (let ((item (todos-item-string))
+         (opoint (point))
          (end (save-excursion (todos-item-end)))
          comment buffer-read-only)
       (save-excursion
@@ -5317,26 +5704,34 @@ With prefix ARG delete an existing comment."
                                         (cons (match-string 1) 1)))
              (replace-match comment nil nil nil 1))
          (setq comment (read-string "Enter a comment: "))
+         ;; If user moved point during editing, make sure it moves back.
+         (goto-char opoint)
          (todos-item-end)
          (insert " [" todos-comment-string ": " comment "]"))))))
 
 (defun todos-item-undo ()
-  "Restore this done item to the todo section of this category.
-If done item has a comment, ask whether to omit the comment from
-the restored item."
+  "Restore at least one done item to this category's todo section.
+Prompt for the new priority.  If there are marked items, undo all
+of these, giving the first undone item the new priority and the
+rest following directly in sequence; otherwise, undo just the
+item at point.
+
+If the done item has a comment, ask whether to omit the comment
+from the restored item.  With multiple marked done items with
+comments, only ask once, and if affirmed, omit subsequent
+comments without asking."
   (interactive)
   (let* ((cat (todos-current-category))
-        (marked (assoc cat todos-categories-with-marks)))
+        (marked (assoc cat todos-categories-with-marks))
+        (pl (if (and marked (> (cdr marked) 1)) "s" "")))
     (when (or marked (todos-done-item-p))
       (let ((buffer-read-only)
-           (bufmod (buffer-modified-p))
            (opoint (point))
-           (orig-mrk (progn (todos-item-start) (point-marker)))
-           (orig-item (todos-item-string))
+           (omark (point-marker))
            (first 'first)
            (item-count 0)
            (diary-count 0)
-           start end item undone)
+           start end item ov npoint undone)
        (and marked (goto-char (point-min)))
        (catch 'done
          (while (not (eobp))
@@ -5344,6 +5739,10 @@ the restored item."
                (if (not (todos-done-item-p))
                    (error "Only done items can be undone")
                  (todos-item-start)
+                 (unless marked
+                   (setq ov (make-overlay (save-excursion (todos-item-start))
+                                          (save-excursion (todos-item-end))))
+                   (overlay-put ov 'face 'todos-search))
                  ;; Find the end of the date string added upon tagging item as
                  ;; done.
                  (setq start (search-forward "] "))
@@ -5359,60 +5758,52 @@ the restored item."
                    (if (eq first 'first)
                        (setq first
                              (if (eq todos-undo-item-omit-comment 'ask)
-                                 (when (y-or-n-p
-                                        "Omit comment from restored item? ")
+                                 (when (y-or-n-p (concat "Omit comment" pl
+                                                         " from restored item"
+                                                         pl "? "))
                                    'omit)
                                (when todos-undo-item-omit-comment 'omit)))
                      t)
                    (when (eq first 'omit)
-                     (delete-region (match-beginning 0) (match-end 0))
-                     (setq end (point))))
+                     (setq end (match-beginning 0)))
                  (setq item (concat item
                                     (buffer-substring-no-properties start end)
                                     (when marked "\n")))
-                 (todos-remove-item)
-                 (unless marked (throw 'done nil)))
-             (todos-forward-item))))
-       (if marked
+                 (unless marked (throw 'done nil)))))
+           (todos-forward-item)))
+       (unwind-protect
            (progn
-             ;; (remove-overlays (point-min) (point-max)
-             ;;                       'before-string todos-item-mark)
-             (setq todos-categories-with-marks
-                   (assq-delete-all cat todos-categories-with-marks))
-             ;; Insert undone items that were marked at end of todo item list.
-             (goto-char (point-min))
-             (re-search-forward (concat "^" (regexp-quote todos-category-done))
-                                nil t)
-             (forward-line -1)
-             (insert item)
-             (todos-update-count 'todo item-count)
-             (todos-update-count 'done (- item-count))
-             (when diary-count (todos-update-count 'diary diary-count))
-             (todos-update-categories-sexp)
-             (let ((todos-show-with-done (> (todos-get-count 'done) 0)))
-               (todos-category-select)))
-         ;; With an unmarked undone item, prompt for its priority.  If user
-         ;; cancels before setting new priority, then leave the done item
-         ;; unchanged.
-         (unwind-protect
-             (progn
-               (todos-set-item-priority item (todos-current-category) t)
-               (setq undone t)
-               (todos-update-count 'todo 1)
-               (todos-update-count 'done -1)
-               (and (todos-diary-item-p) (todos-update-count 'diary 1))
-               (todos-update-categories-sexp)
-               (let ((todos-show-with-done (> (todos-get-count 'done) 0)))
-                 (todos-category-select)))
-           (unless undone
-             (let ((todos-show-with-done t))
-               (widen)
-               (goto-char orig-mrk)
-               (todos-insert-with-overlays orig-item)
-               (set-buffer-modified-p bufmod)
-               (todos-category-select))
-               (goto-char opoint))))
-       (set-marker orig-mrk nil)))))
+             ;; Chop off last newline of multiple items string, since
+             ;; it will be reinserted on setting priority.
+             (and marked (setq item (substring item 0 -1)))
+             (todos-set-item-priority item cat t)
+             (setq npoint (point))
+             (setq undone t))
+         (if ov (delete-overlay ov))
+         (if (not undone)
+             (goto-char opoint)
+           (if marked
+               (progn
+                 (setq item nil)
+                 (re-search-forward
+                  (concat "^" (regexp-quote todos-category-done)) nil t)
+                 (while (not (eobp))
+                   (if (todos-marked-item-p)
+                       (todos-remove-item)
+                     (todos-forward-item)))
+                 (setq todos-categories-with-marks
+                       (assq-delete-all cat todos-categories-with-marks)))
+             (goto-char omark)
+             (todos-remove-item))
+           (todos-update-count 'todo item-count)
+           (todos-update-count 'done (- item-count))
+           (when diary-count (todos-update-count 'diary diary-count))
+           (todos-update-categories-sexp)
+           (let ((todos-show-with-done (> (todos-get-count 'done) 0)))
+             (todos-category-select))
+           ;; Put cursor on undone item.
+           (goto-char npoint)))
+       (set-marker omark nil)))))
 
 (defun todos-archive-done-item (&optional all)
   "Archive at least one done item in this category.
@@ -5533,8 +5924,6 @@ this category does not exist in the archive, it is created."
                         (todos-update-count 'done (- count))
                         (todos-update-count 'archived count)))))
              (when marked
-               (remove-overlays (point-min) (point-max)
-                                'before-string todos-item-mark)
                (setq todos-categories-with-marks
                      (assq-delete-all cat todos-categories-with-marks)))
              (todos-update-categories-sexp)
@@ -5555,120 +5944,114 @@ this category does not exist in the archive, it is created."
   (interactive)
   (todos-archive-done-item t))
 
-(defun todos-unarchive-items (&optional all)
+(defun todos-unarchive-items ()
   "Unarchive at least one item in this archive category.
-
 If there are marked items, unarchive all of these; otherwise,
-with non-nil argument ALL, unarchive all items in this category;
-otherwise, unarchive the item at point.
+unarchive the item at point.
 
 Unarchived items are restored as done items to the corresponding
-category in the Todos file, inserted at the end of done section.
-If all items in the archive category were restored, the category
-is deleted from the archive.  If this was the only category in the
-archive, the archive file is deleted."
+category in the Todos file, inserted at the top of done items
+section.  If all items in the archive category have been
+restored, the category is deleted from the archive.  If this was
+the only category in the archive, the archive file is deleted."
   (interactive)
   (when (eq major-mode 'todos-archive-mode)
-    (catch 'end
-      (let* ((cat (todos-current-category))
-            (tbuf (find-file-noselect
-                   (concat (file-name-sans-extension todos-current-todos-file)
-                           ".todo") t))
-            (marked (assoc cat todos-categories-with-marks))
-            (item (concat (todos-item-string) "\n"))
-            (all-items (when all (buffer-substring-no-properties
-                                  (point-min) (point-max))))
-            (all-count (when all (todos-get-count 'done)))
-            marked-items marked-count
-            buffer-read-only)
-       (when marked
-         (save-excursion
-           (goto-char (point-min))
-           (while (not (eobp))
-             (when (todos-marked-item-p)
-               (concat marked-items (todos-item-string) "\n")
-               (setq marked-count (1+ marked-count)))
-             (todos-forward-item))))
-       ;; Restore items to end of category's done section and update counts.
-       (with-current-buffer tbuf
-         (let (buffer-read-only)
-           (widen)
-           (goto-char (point-min))
-           (re-search-forward (concat "^" (regexp-quote
-                                           (concat todos-category-beg cat)) "$")
-                              nil t)
-           ;; Go to end of category's done section.
-           (if (re-search-forward (concat "^" (regexp-quote todos-category-beg))
-                                  nil t)
-               (goto-char (match-beginning 0))
-             (goto-char (point-max)))
-           (cond (marked
-                  (insert marked-items)
-                  (todos-update-count 'done marked-count cat)
-                  (todos-update-count 'archived (- marked-count) cat))
-                 (all
-                  (insert all-items)
-                  (todos-update-count 'done all-count cat)
-                  (todos-update-count 'archived (- all-count) cat))
-                 (t
-                  (insert item)
-                  (todos-update-count 'done 1 cat)
-                  (todos-update-count 'archived -1 cat)))
-           (todos-update-categories-sexp)))
-       ;; Delete restored items from archive.
-       (cond ((or marked item)
-              (and marked (goto-char (point-min)))
-              (catch 'done
-                (while (not (eobp))
-                  (if (or (and marked (todos-marked-item-p)) item)
-                      (progn
-                        (todos-remove-item)
-                        ;; Don't leave point below last item.
-                        (and item (bolp) (eolp) (< (point-min) (point-max))
-                             (todos-backward-item))
-                        (when item 
-                          (throw 'done (setq item nil))))
-                    (todos-forward-item))))
-              (todos-update-count 'done (if marked (- marked-count) -1) cat))
-             (all
-              (remove-overlays (point-min) (point-max))
-              (delete-region (point-min) (point-max))))
-       ;; If that was the last category in the archive, delete the whole file.
-       (if (= (length todos-categories) 1)
-           (progn
-             (delete-file todos-current-todos-file)
-             ;; Don't bother confirming killing the archive buffer.
-             (set-buffer-modified-p nil)
-             (kill-buffer))
-         ;; Otherwise, if the archive category is now empty, delete it.
-         (when (eq (point-min) (point-max))
-           (widen)
-           (let ((beg (re-search-backward
-                       (concat "^" (regexp-quote todos-category-beg) cat "$")
-                       nil t))
-                 (end (if (re-search-forward
-                           (concat "^" (regexp-quote todos-category-beg))
-                           nil t 2)
-                          (match-beginning 0)
-                        (point-max))))
-             (remove-overlays beg end)
-             (delete-region beg end)
-             (setq todos-categories (delete (assoc cat todos-categories)
-                                            todos-categories))
-             (todos-update-categories-sexp))))
-       ;; Visit category in Todos file and show restored done items.
-       (let ((tfile (buffer-file-name tbuf))
-             (todos-show-with-done t))
-         (set-window-buffer (selected-window)
-                            (set-buffer (find-file-noselect tfile)))
-         (todos-category-number cat)
-         (todos-show)
-         (message "Items unarchived."))))))
-
-(defun todos-unarchive-category ()
-  "Unarchive all items in this category.  See `todos-unarchive-items'."
-  (interactive)
-  (todos-unarchive-items t))
+    (let* ((cat (todos-current-category))
+          (tbuf (find-file-noselect
+                 (concat (file-name-sans-extension todos-current-todos-file)
+                         ".todo") t))
+          (marked (assoc cat todos-categories-with-marks))
+          (item (concat (todos-item-string) "\n"))
+          (marked-count 0)
+          marked-items
+          buffer-read-only)
+      (when marked
+       (save-excursion
+         (goto-char (point-min))
+         (while (not (eobp))
+           (when (todos-marked-item-p)
+             (setq marked-items (concat marked-items (todos-item-string) "\n"))
+             (setq marked-count (1+ marked-count)))
+           (todos-forward-item))))
+      ;; Restore items to top of category's done section and update counts.
+      (with-current-buffer tbuf
+       (let (buffer-read-only newcat)
+         (widen)
+         (goto-char (point-min))
+         ;; Find the corresponding todo category, or if there isn't
+         ;; one, add it.
+         (unless (re-search-forward
+                  (concat "^" (regexp-quote (concat todos-category-beg cat))
+                          "$") nil t)
+           (todos-add-category nil cat)
+           (setq newcat t))
+         ;; Go to top of category's done section.
+         (re-search-forward
+          (concat "^" (regexp-quote todos-category-done)) nil t)
+         (forward-line)
+         ;; FIXME: delete after checking
+         ;; ;; Put point below newly added category beginning,
+         ;; ;; otherwise the following search wrongly succeeds.
+         ;; (forward-line))
+         ;; (if (re-search-forward (concat "^" (regexp-quote todos-category-beg))
+         ;;                     nil t)
+         ;;     (goto-char (match-beginning 0))
+         ;;   (goto-char (point-max)))
+         (cond (marked
+                (insert marked-items)
+                (todos-update-count 'done marked-count cat)
+                (unless newcat         ; Newly added category has no archive.
+                  (todos-update-count 'archived (- marked-count) cat)))
+               (t
+                (insert item)
+                (todos-update-count 'done 1 cat)
+                (unless newcat         ; Newly added category has no archive.
+                  (todos-update-count 'archived -1 cat))))
+         (todos-update-categories-sexp)))
+      ;; Delete restored items from archive.
+      (when marked
+       (setq item nil)
+       (goto-char (point-min)))
+      (catch 'done
+       (while (not (eobp))
+         (if (or (todos-marked-item-p) item)
+             (progn
+               (todos-remove-item)
+               (when item
+                 (throw 'done (setq item nil))))
+           (todos-forward-item))))
+      (todos-update-count 'done (if marked (- marked-count) -1) cat)
+      ;; If that was the last category in the archive, delete the whole file.
+      (if (= (length todos-categories) 1)
+         (progn
+           (delete-file todos-current-todos-file)
+           ;; Kill the archive buffer silently.
+           (set-buffer-modified-p nil)
+           (kill-buffer))
+       ;; Otherwise, if the archive category is now empty, delete it.
+       (when (eq (point-min) (point-max))
+         (widen)
+         (let ((beg (re-search-backward
+                     (concat "^" (regexp-quote todos-category-beg) cat "$")
+                     nil t))
+               (end (if (re-search-forward
+                         (concat "^" (regexp-quote todos-category-beg))
+                         nil t 2)
+                        (match-beginning 0)
+                      (point-max))))
+           (remove-overlays beg end)
+           (delete-region beg end)
+           (setq todos-categories (delete (assoc cat todos-categories)
+                                          todos-categories))
+           (todos-update-categories-sexp))))
+      ;; Visit category in Todos file and show restored done items.
+      (let ((tfile (buffer-file-name tbuf))
+           (todos-show-with-done t))
+       (set-window-buffer (selected-window)
+                          (set-buffer (find-file-noselect tfile)))
+       (todos-category-number cat)
+       (todos-category-select)
+       (message "Items unarchived.")))))
 
 (provide 'todos)
 
@@ -5678,6 +6061,7 @@ archive, the archive file is deleted."
 ;; ---------------------------------------------------------------------------
 (add-to-list 'auto-mode-alist '("\\.todo\\'" . todos-mode))
 (add-to-list 'auto-mode-alist '("\\.toda\\'" . todos-archive-mode))
+(add-to-list 'auto-mode-alist '("\\.tod[tyr]\\'" . todos-filtered-items-mode))
 
 ;;; Addition to calendar.el
 ;; FIXME: autoload when key-binding is defined in calendar.el