]> code.delx.au - gnu-emacs/blobdiff - lisp/cus-edit.el
(dabbrev-case-replace, dabbrev-case-fold-search):
[gnu-emacs] / lisp / cus-edit.el
index 655b3c83bf676fb37485bf04572c1f7590719449..023592a88a9c729c7543d134470cb071c11c3d4c 100644 (file)
@@ -4,7 +4,7 @@
 ;;
 ;; Author: Per Abrahamsen <abraham@dina.kvl.dk>
 ;; Keywords: help, faces
-;; Version: 1.84
+;; Version: 1.90
 ;; X-URL: http://www.dina.kvl.dk/~abraham/custom/
 
 ;; This file is part of GNU Emacs.
@@ -26,6 +26,8 @@
 
 ;;; Commentary:
 ;;
+;; This file implements the code to create and edit customize buffers.
+;; 
 ;; See `custom.el'.
 
 ;;; Code:
 (require 'cus-face)
 (require 'wid-edit)
 (require 'easymenu)
+(eval-when-compile (require 'cl))
+
+(condition-case nil
+    (require 'cus-load)
+  (error nil))
 
 (defun custom-face-display-set (face spec &optional frame)
   (face-spec-set face spec frame))
@@ -355,10 +362,30 @@ Return a list suitable for use in `interactive'."
                (if v
                    (format "Customize variable (default %s): " v)
                  "Customize variable: ")
-               obarray 'boundp t))
+               obarray (lambda (symbol)
+                         (and (boundp symbol)
+                              (or (get symbol 'custom-type)
+                                  (user-variable-p symbol))))))
      (list (if (equal val "")
               v (intern val)))))
 
+(defun custom-menu-filter (menu widget)
+  "Convert MENU to the form used by `widget-choose'.
+MENU should be in the same format as `custom-variable-menu'.
+WIDGET is the widget to apply the filter entries of MENU on."
+  (let ((result nil)
+       current name action filter)
+    (while menu 
+      (setq current (car menu)
+           name (nth 0 current)
+           action (nth 1 current)
+           filter (nth 2 current)
+           menu (cdr menu))
+      (if (or (null filter) (funcall filter widget))
+         (push (cons name action) result)
+       (push name result)))
+    (nreverse result)))
+
 ;;; Unlispify.
 
 (defvar custom-prefix-list nil
@@ -542,7 +569,7 @@ when the action is chosen.")
            children)))
 
 (defun custom-reset-factory ()
-  "Reset all modified, set, or saved group members to their factory settings."
+  "Reset all modified, set, or saved group members to their standard settings."
   (interactive)
   (let ((children custom-options))
     (mapcar (lambda (child)
@@ -552,8 +579,100 @@ when the action is chosen.")
 
 ;;; The Customize Commands
 
+(defun custom-prompt-variable (prompt-var prompt-val)
+  "Prompt for a variable and a value and return them as a list.
+PROMPT-VAR is the prompt for the variable, and PROMPT-VAL is the
+prompt for the value.  The %s escape in PROMPT-VAL is replaced with
+the name of the variable.
+
+If the variable has a `variable-interactive' property, that is used as if
+it were the arg to `interactive' (which see) to interactively read the value.
+
+If the variable has a `custom-type' property, it must be a widget and the
+`:prompt-value' property of that widget will be used for reading the value."
+  (let* ((var (read-variable prompt-var))
+        (minibuffer-help-form '(describe-variable var)))
+    (list var
+         (let ((prop (get var 'variable-interactive))
+               (type (get var 'custom-type))
+               (prompt (format prompt-val var)))
+           (unless (listp type)
+             (setq type (list type)))
+           (cond (prop
+                  ;; Use VAR's `variable-interactive' property
+                  ;; as an interactive spec for prompting.
+                  (call-interactively (list 'lambda '(arg)
+                                            (list 'interactive prop)
+                                            'arg)))
+                 (type
+                  (widget-prompt-value type
+                                       prompt
+                                       (if (boundp var)
+                                           (symbol-value var))
+                                       (not (boundp var))))
+                 (t
+                  (eval-minibuffer prompt)))))))
+
+;;;###autoload
+(defun custom-set-value (var val)
+  "Set VARIABLE to VALUE.  VALUE is a Lisp object.
+
+If VARIABLE has a `variable-interactive' property, that is used as if
+it were the arg to `interactive' (which see) to interactively read the value.
+
+If VARIABLE has a `custom-type' property, it must be a widget and the
+`:prompt-value' property of that widget will be used for reading the value." 
+  (interactive (custom-prompt-variable "Set variable: "
+                                      "Set %s to value: "))
+   
+  (set var val))
+
+;;;###autoload
+(defun custom-set-variable (var val)
+  "Set the default for VARIABLE to VALUE.  VALUE is a Lisp object.
+
+If VARIABLE has a `custom-set' property, that is used for setting
+VARIABLE, otherwise `set-default' is used.
+
+The `customized-value' property of the VARIABLE will be set to a list
+with a quoted VALUE as its sole list member.
+
+If VARIABLE has a `variable-interactive' property, that is used as if
+it were the arg to `interactive' (which see) to interactively read the value.
+
+If VARIABLE has a `custom-type' property, it must be a widget and the
+`:prompt-value' property of that widget will be used for reading the value. " 
+  (interactive (custom-prompt-variable "Set variable: "
+                                      "Set customized value for %s to: "))
+  (funcall (or (get var 'custom-set) 'set-default) var val)
+  (put var 'customized-value (list (custom-quote val))))
+
+;;;###autoload
+(defun customize ()
+  "Select a customization buffer which you can use to set user options.
+User options are structured into \"groups\".
+Initially the top-level group `Emacs' and its immediate subgroups
+are shown; the contents of those subgroups are initially hidden."
+  (interactive)
+  (customize-group 'emacs))
+
+;;;###autoload
+(defun customize-group (group)
+  "Customize GROUP, which must be a customization group."
+  (interactive (list (completing-read "Customize group: (default emacs) "
+                                     obarray 
+                                     (lambda (symbol)
+                                       (get symbol 'custom-group))
+                                     t)))
+
+  (when (stringp group)
+    (if (string-equal "" group)
+       (setq group 'emacs)
+      (setq group (intern group))))
+  (custom-buffer-create (list (list group 'custom-group))))
+
 ;;;###autoload
-(defun customize (symbol)
+(defun customize-other-window (symbol)
   "Customize SYMBOL, which must be a customization group."
   (interactive (list (completing-read "Customize group: (default emacs) "
                                      obarray 
@@ -565,7 +684,7 @@ when the action is chosen.")
     (if (string-equal "" symbol)
        (setq symbol 'emacs)
       (setq symbol (intern symbol))))
-  (custom-buffer-create (list (list symbol 'custom-group))))
+  (custom-buffer-create-other-window (list (list symbol 'custom-group))))
 
 ;;;###autoload
 (defun customize-variable (symbol)
@@ -617,7 +736,24 @@ If SYMBOL is nil, customize all faces."
 
 ;;;###autoload
 (defun customize-customized ()
-  "Customize all already customized user options."
+  "Customize all user options set since the last save in this session."
+  (interactive)
+  (let ((found nil))
+    (mapatoms (lambda (symbol)
+               (and (get symbol 'customized-face)
+                    (custom-facep symbol)
+                    (setq found (cons (list symbol 'custom-face) found)))
+               (and (get symbol 'customized-value)
+                    (boundp symbol)
+                    (setq found
+                          (cons (list symbol 'custom-variable) found)))))
+    (if found 
+       (custom-buffer-create found)
+      (error "No customized user options"))))
+
+;;;###autoload
+(defun customize-saved ()
+  "Customize all already saved user options."
   (interactive)
   (let ((found nil))
     (mapatoms (lambda (symbol)
@@ -630,7 +766,7 @@ If SYMBOL is nil, customize all faces."
                           (cons (list symbol 'custom-variable) found)))))
     (if found 
        (custom-buffer-create found)
-      (error "No customized user options"))))
+      (error "No saved user options"))))
 
 ;;;###autoload
 (defun customize-apropos (regexp &optional all)
@@ -657,6 +793,8 @@ user-settable."
        (custom-buffer-create found)
       (error "No matches"))))
 
+;;; Buffer.
+
 ;;;###autoload
 (defun custom-buffer-create (options)
   "Create a buffer containing OPTIONS.
@@ -667,6 +805,7 @@ that option."
   (switch-to-buffer (get-buffer-create "*Customization*"))
   (custom-buffer-create-internal options))
 
+;;;###autoload
 (defun custom-buffer-create-other-window (options)
   "Create a buffer containing OPTIONS.
 OPTIONS should be an alist of the form ((SYMBOL WIDGET)...), where
@@ -758,6 +897,7 @@ Make the modifications default for future sessions."
   (message "Creating customization setup...")
   (widget-setup)
   (goto-char (point-min))
+  (forward-line 3)                     ;Kludge: bob is writable in XEmacs.
   (message "Creating customization buffer...done"))
 
 ;;; Modification of Basic Widgets.
@@ -840,7 +980,7 @@ this item has been saved.")
                                (rogue "@" custom-rogue-face "\
 this item is not prepared for customization.")
                                (factory " " nil "\
-this item is unchanged from its factory setting."))
+this item is unchanged from its standard setting."))
   "Alist of customize option states.
 Each entry is of the form (STATE MAGIC FACE DESCRIPTION), where 
 
@@ -865,7 +1005,109 @@ STATE is one of the following symbols:
 `rogue'
    This item has no customization information.
 `factory'
-   This item is unchanged from the factory default.
+   This item is unchanged from the standard setting.
+
+MAGIC is a string used to present that state.
+
+FACE is a face used to present the state.
+
+DESCRIPTION is a string describing the state.
+
+The list should be sorted most significant first."
+  :type '(list (checklist :inline t
+                         (group (const nil)
+                                (string :tag "Magic")
+                                face 
+                                (string :tag "Description"))
+                         (group (const unknown)
+                                (string :tag "Magic")
+                                face 
+                                (string :tag "Description"))
+                         (group (const hidden)
+                                (string :tag "Magic")
+                                face 
+                                (string :tag "Description"))
+                         (group (const invalid)
+                                (string :tag "Magic")
+                                face 
+                                (string :tag "Description"))
+                         (group (const modified)
+                                (string :tag "Magic")
+                                face 
+                                (string :tag "Description"))
+                         (group (const set)
+                                (string :tag "Magic")
+                                face 
+                                (string :tag "Description"))
+                         (group (const changed)
+                                (string :tag "Magic")
+                                face 
+                                (string :tag "Description"))
+                         (group (const saved)
+                                (string :tag "Magic")
+                                face 
+                                (string :tag "Description"))
+                         (group (const rogue)
+                                (string :tag "Magic")
+                                face 
+                                (string :tag "Description"))
+                         (group (const factory)
+                                (string :tag "Magic")
+                                face 
+                                (string :tag "Description")))
+              (editable-list :inline t
+                             (group symbol
+                                    (string :tag "Magic")
+                                    face
+                                    (string :tag "Description"))))
+  :group 'customize
+  :group 'custom-faces)
+
+(defcustom custom-group-magic-alist '((nil "#" underline "\
+uninitialized, you should not see this.")
+                               (unknown "?" italic "\
+unknown, you should not see this.")
+                               (hidden "-" default "\
+group now hidden; click on the asterisks above to show contents.")
+                               (invalid "x" custom-invalid-face "\
+the value displayed for this item is invalid and cannot be set.")
+                               (modified "*" custom-modified-face "\
+you have edited something in this group, and can now set it.")
+                               (set "+" custom-set-face "\
+something in this group has been set, but not yet saved.")
+                               (changed ":" custom-changed-face "\
+this item has been changed outside customize.")
+                               (saved "!" custom-saved-face "\
+something in this group has been set and saved.")
+                               (rogue "@" custom-rogue-face "\
+this item is not prepared for customization.")
+                               (factory " " nil "\
+nothing in this group has been changed."))
+  "Alist of customize option states.
+Each entry is of the form (STATE MAGIC FACE DESCRIPTION), where 
+
+STATE is one of the following symbols:
+
+`nil'
+   For internal use, should never occur.
+`unknown'
+   For internal use, should never occur.
+`hidden'
+   This item is not being displayed. 
+`invalid'
+   This item is modified, but has an invalid form.
+`modified'
+   This item is modified, and has a valid form.
+`set'
+   This item has been set but not saved.
+`changed'
+   The current value of this item has been changed temporarily.
+`saved'
+   This item is marked for saving.
+`rogue'
+   This item has no customization information.
+`factory'
+   This item is unchanged from the standard setting.
 
 MAGIC is a string used to present that state.
 
@@ -939,6 +1181,7 @@ The list should be sorted most significant first."
   "Show and manipulate state for a customization option."
   :format "%v"
   :action 'widget-choice-item-action
+  :notify 'ignore
   :value-get 'ignore
   :value-create 'custom-magic-value-create
   :value-delete 'widget-children-value-delete)
@@ -947,7 +1190,9 @@ The list should be sorted most significant first."
   ;; Create compact status report for WIDGET.
   (let* ((parent (widget-get widget :parent))
         (state (widget-get parent :custom-state))
-        (entry (assq state custom-magic-alist))
+        (entry (assq state (if (eq (car parent) 'custom-group)
+                               custom-group-magic-alist
+                             custom-magic-alist)))
         (magic (nth 1 entry))
         (face (nth 2 entry))
         (text (nth 3 entry))
@@ -998,15 +1243,7 @@ Change the state of this item."
 
 (defun custom-level-action (widget &optional event)
   "Toggle visibility for parent to WIDGET."
-  (let* ((parent (widget-get widget :parent))
-        (state (widget-get parent :custom-state)))
-    (cond ((memq state '(invalid modified))
-          (error "There are unset changes"))
-         ((eq state 'hidden)
-          (widget-put parent :custom-state 'unknown))
-         (t
-          (widget-put parent :custom-state 'hidden)))
-    (custom-redraw parent)))
+  (custom-toggle-hide (widget-get widget :parent)))
 
 ;;; The `custom' Widget.
 
@@ -1094,14 +1331,20 @@ Change the state of this item."
 
 (defun custom-redraw (widget)
   "Redraw WIDGET with current settings."
-  (let ((pos (point))
+  (let ((line (count-lines (point-min) (point)))
+       (column (current-column))
+       (pos (point))
        (from (marker-position (widget-get widget :from)))
        (to (marker-position (widget-get widget :to))))
     (save-excursion
       (widget-value-set widget (widget-value widget))
       (custom-redraw-magic widget))
     (when (and (>= pos from) (<= pos to))
-      (goto-char pos))))
+      (condition-case nil
+         (progn 
+           (goto-line line)
+           (move-to-column column))
+       (error nil)))))
 
 (defun custom-redraw-magic (widget)
   "Redraw WIDGET state with current settings."
@@ -1150,6 +1393,17 @@ Change the state of this item."
   "Load all dependencies for WIDGET."
   (custom-load-symbol (widget-value widget)))
 
+(defun custom-toggle-hide (widget)
+  "Toggle visibility of WIDGET."
+  (let ((state (widget-get widget :custom-state)))
+    (cond ((memq state '(invalid modified))
+          (error "There are unset changes"))
+         ((eq state 'hidden)
+          (widget-put widget :custom-state 'unknown))
+         (t 
+          (widget-put widget :custom-state 'hidden)))
+    (custom-redraw widget)))
+
 ;;; The `custom-variable' Widget.
 
 (defface custom-variable-sample-face '((t (:underline t)))
@@ -1186,7 +1440,7 @@ Otherwise, look up symbol in `custom-guess-type-alist'."
                   'sexp))
         (options (get symbol 'custom-options))
         (tmp (if (listp type)
-                 (copy-list type)
+                 (copy-sequence type)
                (list type))))
     (when options
       (widget-put tmp :options options))
@@ -1203,8 +1457,10 @@ Otherwise, look up symbol in `custom-guess-type-alist'."
         (tag (widget-get widget :tag))
         (type (custom-variable-type symbol))
         (conv (widget-convert type))
+        (get (or (get symbol 'custom-get) 'default-value))
+        (set (or (get symbol 'custom-set) 'set-default))
         (value (if (default-boundp symbol)
-                   (default-value symbol)
+                   (funcall get symbol)
                  (widget-get conv :value))))
     ;; If the widget is new, the child determine whether it is hidden.
     (cond (state)
@@ -1234,7 +1490,7 @@ Otherwise, look up symbol in `custom-guess-type-alist'."
                               ((get symbol 'factory-value)
                                (car (get symbol 'factory-value)))
                               ((default-boundp symbol)
-                               (custom-quote (default-value symbol)))
+                               (custom-quote (funcall get symbol)))
                               (t
                                (custom-quote (widget-get conv :value))))))
             (push (widget-create-child-and-convert 
@@ -1266,8 +1522,9 @@ Otherwise, look up symbol in `custom-guess-type-alist'."
 (defun custom-variable-state-set (widget)
   "Set the state of WIDGET."
   (let* ((symbol (widget-value widget))
+        (get (or (get symbol 'custom-get) 'default-value))
         (value (if (default-boundp symbol)
-                   (default-value symbol)
+                   (funcall get symbol)
                  (widget-get widget :value)))
         tmp
         (state (cond ((setq tmp (get symbol 'customized-value))
@@ -1292,29 +1549,52 @@ Otherwise, look up symbol in `custom-guess-type-alist'."
     (widget-put widget :custom-state state)))
 
 (defvar custom-variable-menu 
-  '(("Edit" . custom-variable-edit)
-    ("Edit Lisp" . custom-variable-edit-lisp)
-    ("Set" . custom-variable-set)
-    ("Save" . custom-variable-save)
-    ("Reset to Current" . custom-redraw)
-    ("Reset to Saved" . custom-variable-reset-saved)
-    ("Reset to Factory Settings" . custom-variable-reset-factory))
+  '(("Hide" custom-toggle-hide
+     (lambda (widget)
+       (not (memq (widget-get widget :custom-state) '(modified invalid)))))
+     ("Edit" custom-variable-edit 
+     (lambda (widget)
+       (not (eq (widget-get widget :custom-form) 'edit))))
+    ("Edit Lisp" custom-variable-edit-lisp
+     (lambda (widget)
+       (not (eq (widget-get widget :custom-form) 'lisp))))
+    ("Set" custom-variable-set
+     (lambda (widget)
+       (eq (widget-get widget :custom-state) 'modified)))
+    ("Save" custom-variable-save
+     (lambda (widget)
+       (memq (widget-get widget :custom-state) '(modified set changed rogue))))
+    ("Reset to Current" custom-redraw
+     (lambda (widget)
+       (and (default-boundp (widget-value widget))
+           (memq (widget-get widget :custom-state) '(modified)))))
+    ("Reset to Saved" custom-variable-reset-saved
+     (lambda (widget)
+       (and (get (widget-value widget) 'saved-value)
+           (memq (widget-get widget :custom-state)
+                 '(modified set changed rogue)))))
+    ("Reset to Standard Settings" custom-variable-reset-factory
+     (lambda (widget)
+       (and (get (widget-value widget) 'factory-value)
+           (memq (widget-get widget :custom-state)
+                 '(modified set changed saved rogue))))))
   "Alist of actions for the `custom-variable' widget.
-The key is a string containing the name of the action, the value is a
-lisp function taking the widget as an element which will be called
-when the action is chosen.")
+Each entry has the form (NAME ACTION FILTER) where NAME is the name of
+the menu entry, ACTION is the function to call on the widget when the
+menu is selected, and FILTER is a predicate which takes a `custom-variable'
+widget as an argument, and returns non-nil if ACTION is valid on that
+widget. If FILTER is nil, ACTION is always valid.")
 
 (defun custom-variable-action (widget &optional event)
   "Show the menu for `custom-variable' WIDGET.
 Optional EVENT is the location for the menu."
   (if (eq (widget-get widget :custom-state) 'hidden)
-      (progn 
-       (widget-put widget :custom-state 'unknown)
-       (custom-redraw widget))
+      (custom-toggle-hide widget)
     (let* ((completion-ignore-case t)
           (answer (widget-choose (custom-unlispify-tag-name
                                   (widget-get widget :value))
-                                 custom-variable-menu
+                                 (custom-menu-filter custom-variable-menu
+                                                     widget)
                                  event)))
       (if answer
          (funcall answer widget)))))
@@ -1333,32 +1613,34 @@ Optional EVENT is the location for the menu."
 
 (defun custom-variable-set (widget)
   "Set the current value for the variable being edited by WIDGET."
-  (let ((form (widget-get widget :custom-form))
-       (state (widget-get widget :custom-state))
-       (child (car (widget-get widget :children)))
-       (symbol (widget-value widget))
-       val)
+  (let* ((form (widget-get widget :custom-form))
+        (state (widget-get widget :custom-state))
+        (child (car (widget-get widget :children)))
+        (symbol (widget-value widget))
+        (set (or (get symbol 'custom-set) 'set-default))
+         val)
     (cond ((eq state 'hidden)
           (error "Cannot set hidden variable."))
          ((setq val (widget-apply child :validate))
           (goto-char (widget-get val :from))
           (error "%s" (widget-get val :error)))
          ((eq form 'lisp)
-          (set-default symbol (eval (setq val (widget-value child))))
+          (funcall set symbol (eval (setq val (widget-value child))))
           (put symbol 'customized-value (list val)))
          (t
-          (set-default symbol (setq val (widget-value child)))
+          (funcall set symbol (setq val (widget-value child)))
           (put symbol 'customized-value (list (custom-quote val)))))
     (custom-variable-state-set widget)
     (custom-redraw-magic widget)))
 
 (defun custom-variable-save (widget)
   "Set the default value for the variable being edited by WIDGET."
-  (let ((form (widget-get widget :custom-form))
-       (state (widget-get widget :custom-state))
-       (child (car (widget-get widget :children)))
-       (symbol (widget-value widget))
-       val)
+  (let* ((form (widget-get widget :custom-form))
+        (state (widget-get widget :custom-state))
+        (child (car (widget-get widget :children)))
+        (symbol (widget-value widget))
+        (set (or (get symbol 'custom-set) 'set-default))
+        val)
     (cond ((eq state 'hidden)
           (error "Cannot set hidden variable."))
          ((setq val (widget-apply child :validate))
@@ -1366,12 +1648,12 @@ Optional EVENT is the location for the menu."
           (error "%s" (widget-get val :error)))
          ((eq form 'lisp)
           (put symbol 'saved-value (list (widget-value child)))
-          (set-default symbol (eval (widget-value child))))
+          (funcall set symbol (eval (widget-value child))))
          (t
           (put symbol
                'saved-value (list (custom-quote (widget-value
                                                  child))))
-          (set-default symbol (widget-value child))))
+          (funcall set symbol (widget-value child))))
     (put symbol 'customized-value nil)
     (custom-save-all)
     (custom-variable-state-set widget)
@@ -1379,10 +1661,11 @@ Optional EVENT is the location for the menu."
 
 (defun custom-variable-reset-saved (widget)
   "Restore the saved value for the variable being edited by WIDGET."
-  (let ((symbol (widget-value widget)))
+  (let* ((symbol (widget-value widget))
+        (set (or (get symbol 'custom-set) 'set-default)))
     (if (get symbol 'saved-value)
        (condition-case nil
-           (set-default symbol (eval (car (get symbol 'saved-value))))
+           (funcall set symbol (eval (car (get symbol 'saved-value))))
          (error nil))
       (error "No saved value for %s" symbol))
     (put symbol 'customized-value nil)
@@ -1390,11 +1673,12 @@ Optional EVENT is the location for the menu."
     (custom-redraw widget)))
 
 (defun custom-variable-reset-factory (widget)
-  "Restore the factory setting for the variable being edited by WIDGET."
-  (let ((symbol (widget-value widget)))
+  "Restore the standard setting for the variable being edited by WIDGET."
+  (let* ((symbol (widget-value widget))
+        (set (or (get symbol 'custom-set) 'set-default)))
     (if (get symbol 'factory-value)
-       (set-default symbol (eval (car (get symbol 'factory-value))))
-      (error "No factory default for %S" symbol))
+       (funcall set symbol (eval (car (get symbol 'factory-value))))
+      (error "No standard setting known for %S" symbol))
     (put symbol 'customized-value nil)
     (when (get symbol 'saved-value)
       (put symbol 'saved-value nil)
@@ -1550,9 +1834,7 @@ Match frames with dark backgrounds.")
 
 (defun custom-display-unselected-match (widget value)
   "Non-nil if VALUE is an unselected display specification."
-  (and (listp value)
-       (eq (length value) 2)
-       (not (custom-display-match-frame value (selected-frame)))))
+  (not (custom-display-match-frame value (selected-frame))))
 
 (define-widget 'custom-face-selected 'group 
   "Edit the attributes of the selected display in a face specification."
@@ -1600,17 +1882,32 @@ Match frames with dark backgrounds.")
     (message "Creating face editor...done")))
 
 (defvar custom-face-menu 
-  '(("Edit Selected" . custom-face-edit-selected)
-    ("Edit All" . custom-face-edit-all)
-    ("Edit Lisp" . custom-face-edit-lisp)
-    ("Set" . custom-face-set)
-    ("Save" . custom-face-save)
-    ("Reset to Saved" . custom-face-reset-saved)
-    ("Reset to Factory Setting" . custom-face-reset-factory))
+  '(("Hide" custom-toggle-hide
+     (lambda (widget)
+       (not (memq (widget-get widget :custom-state) '(modified invalid)))))
+    ("Edit Selected" custom-face-edit-selected
+     (lambda (widget)
+       (not (eq (widget-get widget :custom-form) 'selected))))
+    ("Edit All" custom-face-edit-all
+     (lambda (widget)
+       (not (eq (widget-get widget :custom-form) 'all))))
+    ("Edit Lisp" custom-face-edit-lisp
+     (lambda (widget)
+       (not (eq (widget-get widget :custom-form) 'lisp))))
+    ("Set" custom-face-set)
+    ("Save" custom-face-save)
+    ("Reset to Saved" custom-face-reset-saved
+     (lambda (widget)
+       (get (widget-value widget) 'saved-face)))
+    ("Reset to Standard Setting" custom-face-reset-factory
+     (lambda (widget)
+       (get (widget-value widget) 'factory-face))))
   "Alist of actions for the `custom-face' widget.
-The key is a string containing the name of the action, the value is a
-lisp function taking the widget as an element which will be called
-when the action is chosen.")
+Each entry has the form (NAME ACTION FILTER) where NAME is the name of
+the menu entry, ACTION is the function to call on the widget when the
+menu is selected, and FILTER is a predicate which takes a `custom-face'
+widget as an argument, and returns non-nil if ACTION is valid on that
+widget. If FILTER is nil, ACTION is always valid.")
 
 (defun custom-face-edit-selected (widget)
   "Edit selected attributes of the value of WIDGET."
@@ -1646,13 +1943,13 @@ when the action is chosen.")
   "Show the menu for `custom-face' WIDGET.
 Optional EVENT is the location for the menu."
   (if (eq (widget-get widget :custom-state) 'hidden)
-      (progn 
-       (widget-put widget :custom-state 'unknown)
-       (custom-redraw widget))
+      (custom-toggle-hide widget)
     (let* ((completion-ignore-case t)
           (symbol (widget-get widget :value))
           (answer (widget-choose (custom-unlispify-tag-name symbol)
-                                 custom-face-menu event)))
+                                 (custom-menu-filter custom-face-menu
+                                                     widget)
+                                 event)))
       (if answer
          (funcall answer widget)))))
 
@@ -1691,12 +1988,12 @@ Optional EVENT is the location for the menu."
     (custom-redraw-magic widget)))
 
 (defun custom-face-reset-factory (widget)
-  "Restore WIDGET to the face's factory settings."
+  "Restore WIDGET to the face's standard settings."
   (let* ((symbol (widget-value widget))
         (child (car (widget-get widget :children)))
         (value (get symbol 'factory-face)))
     (unless value
-      (error "No factory default for this face"))
+      (error "No standard setting for this face"))
     (put symbol 'customized-face nil)
     (when (get symbol 'saved-face)
       (put symbol 'saved-face nil)
@@ -1865,27 +2162,44 @@ and so forth.  The remaining group tags are shown with
        (message "Creating group... done")))))
 
 (defvar custom-group-menu 
-  '(("Set" . custom-group-set)
-    ("Save" . custom-group-save)
-    ("Reset to Current" . custom-group-reset-current)
-    ("Reset to Saved" . custom-group-reset-saved)
-    ("Reset to Factory" . custom-group-reset-factory))
+  '(("Hide" custom-toggle-hide
+     (lambda (widget)
+       (not (memq (widget-get widget :custom-state) '(modified invalid)))))
+    ("Set" custom-group-set
+     (lambda (widget)
+       (eq (widget-get widget :custom-state) 'modified)))
+    ("Save" custom-group-save
+     (lambda (widget)
+       (memq (widget-get widget :custom-state) '(modified set))))
+    ("Reset to Current" custom-group-reset-current
+     (lambda (widget)
+       (and (default-boundp (widget-value widget))
+           (memq (widget-get widget :custom-state) '(modified)))))
+    ("Reset to Saved" custom-group-reset-saved
+     (lambda (widget)
+       (and (get (widget-value widget) 'saved-value)
+           (memq (widget-get widget :custom-state) '(modified set)))))
+    ("Reset to Standard Settings" custom-group-reset-factory
+     (lambda (widget)
+       (and (get (widget-value widget) 'factory-value)
+           (memq (widget-get widget :custom-state) '(modified set saved))))))
   "Alist of actions for the `custom-group' widget.
-The key is a string containing the name of the action, the value is a
-lisp function taking the widget as an element which will be called
-when the action is chosen.")
+Each entry has the form (NAME ACTION FILTER) where NAME is the name of
+the menu entry, ACTION is the function to call on the widget when the
+menu is selected, and FILTER is a predicate which takes a `custom-group'
+widget as an argument, and returns non-nil if ACTION is valid on that
+widget. If FILTER is nil, ACTION is always valid.")
 
 (defun custom-group-action (widget &optional event)
   "Show the menu for `custom-group' WIDGET.
 Optional EVENT is the location for the menu."
   (if (eq (widget-get widget :custom-state) 'hidden)
-      (progn 
-       (widget-put widget :custom-state 'unknown)
-       (custom-redraw widget))
+      (custom-toggle-hide widget)
     (let* ((completion-ignore-case t)
           (answer (widget-choose (custom-unlispify-tag-name
                                   (widget-get widget :value))
-                                 custom-group-menu
+                                 (custom-menu-filter custom-group-menu
+                                                     widget)
                                  event)))
       (if answer
          (funcall answer widget)))))
@@ -1938,7 +2252,7 @@ Optional EVENT is the location for the menu."
           (states (mapcar (lambda (child)
                             (widget-get child :custom-state))
                           children))
-          (magics custom-magic-alist)
+          (magics custom-group-magic-alist)
           (found 'factory))
       (while magics
        (let ((magic (car (car magics))))
@@ -1986,17 +2300,26 @@ Leave point at the location of the call, or after the last expression."
        (princ "\n"))
       (princ "(custom-set-variables")
       (mapatoms (lambda (symbol)
-                 (let ((value (get symbol 'saved-value)))
+                 (let ((value (get symbol 'saved-value))
+                       (requests (get symbol 'custom-requests))
+                       (now (not (or (get symbol 'factory-value)
+                                     (and (not (boundp symbol))
+                                          (not (get symbol 'force-value)))))))
                    (when value
                      (princ "\n '(")
                      (princ symbol)
                      (princ " ")
                      (prin1 (car value))
-                     (if (or (get symbol 'factory-value)
-                             (and (not (boundp symbol))
-                                  (not (get symbol 'force-value))))
-                         (princ ")")
-                       (princ " t)"))))))
+                     (cond (requests
+                            (if now
+                                (princ " t ")
+                              (princ " nil "))
+                            (prin1 requests)
+                            (princ ")"))
+                           (now
+                            (princ " t)"))
+                           (t
+                            (princ ")")))))))
       (princ ")")
       (unless (looking-at "\n")
        (princ "\n")))))
@@ -2037,6 +2360,22 @@ Leave point at the location of the call, or after the last expression."
       (unless (looking-at "\n")
        (princ "\n")))))
 
+;;;###autoload
+(defun custom-save-customized ()
+  "Save all user options which have been set in this session."
+  (interactive)
+  (mapatoms (lambda (symbol)
+             (let ((face (get symbol 'customized-face))
+                   (value (get symbol 'customized-value)))
+               (when face 
+                 (put symbol 'saved-face face)
+                 (put symbol 'customized-face nil))
+               (when value 
+                 (put symbol 'saved-value value)
+                 (put symbol 'customized-value nil)))))
+  ;; We really should update all custom buffers here.
+  (custom-save-all))
+
 ;;;###autoload
 (defun custom-save-all ()
   "Save all customizations in `custom-file'."
@@ -2178,7 +2517,7 @@ The format is suitable for use with `easy-menu-define'."
 
 (easy-menu-define custom-mode-customize-menu 
     custom-mode-map
-  "Menu used in customization buffers."
+  "Menu used to customize customization buffers."
   (customize-menu-create 'customize))
 
 (easy-menu-define custom-mode-menu 
@@ -2189,7 +2528,7 @@ The format is suitable for use with `easy-menu-define'."
     ["Save" custom-save t]
     ["Reset to Current" custom-reset-current t]
     ["Reset to Saved" custom-reset-saved t]
-    ["Reset to Factory Settings" custom-reset-factory t]
+    ["Reset to Standard Settings" custom-reset-factory t]
     ["Info" (Info-goto-node "(custom)The Customization Buffer") t]))
 
 (defcustom custom-mode-hook nil