]> code.delx.au - gnu-emacs/blobdiff - lisp/wid-edit.el
Merge from emacs--rel--22
[gnu-emacs] / lisp / wid-edit.el
index 659c562ea659dfc3e6f331eedc468325f30ab6e3..1ce6deda89efcf05a6e031cadcc117ed03978d10 100644 (file)
@@ -1,7 +1,7 @@
 ;;; wid-edit.el --- Functions for creating and using widgets -*-byte-compile-dynamic: t;-*-
 ;;
 ;; Copyright (C) 1996, 1997, 1999, 2000, 2001, 2002, 2003,
-;;   2004, 2005, 2006 Free Software Foundation, Inc.
+;;   2004, 2005, 2006, 2007 Free Software Foundation, Inc.
 ;;
 ;; Author: Per Abrahamsen <abraham@dina.kvl.dk>
 ;; Maintainer: FSF
@@ -11,7 +11,7 @@
 
 ;; GNU Emacs is free software; you can redistribute it and/or modify
 ;; it under the terms of the GNU General Public License as published by
-;; the Free Software Foundation; either version 2, or (at your option)
+;; the Free Software Foundation; either version 3, or (at your option)
 ;; any later version.
 
 ;; GNU Emacs is distributed in the hope that it will be useful,
@@ -84,7 +84,7 @@
   :group 'hypermedia)
 
 (defgroup widget-documentation nil
-  "Options controling the display of documentation strings."
+  "Options controlling the display of documentation strings."
   :group 'widgets)
 
 (defgroup widget-faces nil
@@ -275,14 +275,15 @@ minibuffer."
                     keys
                     (char 0)
                     (arg 1))
-                (while (not (or (and (>= char ?0) (< char next-digit))
+                (while (not (or (and (integerp char)
+                                     (>= char ?0) (< char next-digit))
                                 (eq value 'keyboard-quit)))
                   ;; Unread a SPC to lead to our new menu.
                   (setq unread-command-events (cons ?\s unread-command-events))
                   (setq keys (read-key-sequence title))
                   (setq value
                         (lookup-key overriding-terminal-local-map keys t)
-                        char (string-to-char (substring keys 1)))
+                        char (aref keys 1))
                   (cond ((eq value 'scroll-other-window)
                          (let ((minibuffer-scroll-window
                                 (get-buffer-window buf)))
@@ -403,8 +404,18 @@ new value.")
     ;; We want to avoid the face with image buttons.
     (unless (widget-get widget :suppress-face)
       (overlay-put overlay 'face (widget-apply widget :button-face-get))
-      (overlay-put overlay 'mouse-face 
-                  (widget-apply widget :mouse-face-get)))
+      (overlay-put overlay 'mouse-face
+                  ;; Make new list structure for the mouse-face value
+                  ;; so that different widgets will have
+                  ;; different `mouse-face' property values
+                  ;; and will highlight separately.
+                  (let ((mouse-face-value
+                         (widget-apply widget :mouse-face-get)))
+                    ;; If it's a list, copy it.
+                    (if (listp mouse-face-value)
+                        (copy-sequence mouse-face-value)
+                      ;; If it's a symbol, put it in a list.
+                      (list mouse-face-value)))))
     (overlay-put overlay 'pointer 'hand)
     (overlay-put overlay 'follow-link follow-link)
     (overlay-put overlay 'help-echo help-echo)))
@@ -477,12 +488,12 @@ new value.")
 ;;; Widget Properties.
 
 (defsubst widget-type (widget)
-  "Return the type of WIDGET, a symbol."
+  "Return the type of WIDGET.  The type is a symbol."
   (car widget))
 
 ;;;###autoload
 (defun widgetp (widget)
-  "Return non-nil iff WIDGET is a widget."
+  "Return non-nil if WIDGET is a widget."
   (if (symbolp widget)
       (get widget 'widget-type)
     (and (consp widget)
@@ -499,7 +510,7 @@ Otherwise, just return the value."
       value)))
 
 (defun widget-member (widget property)
-  "Non-nil iff there is a definition in WIDGET for PROPERTY."
+  "Non-nil if there is a definition in WIDGET for PROPERTY."
   (cond ((plist-member (cdr widget) property)
         t)
        ((car widget)
@@ -598,7 +609,7 @@ automatically."
   :type 'directory)
 
 (defcustom widget-image-enable t
-  "If non nil, use image buttons in widgets when available."
+  "If non-nil, use image buttons in widgets when available."
   :version "21.1"
   :group 'widgets
   :type 'boolean)
@@ -655,7 +666,9 @@ button is pressed or inactive, respectively.  These are currently ignored."
       (progn (widget-put widget :suppress-face t)
             (insert-image image
                           (propertize
-                           tag 'mouse-face widget-button-pressed-face)))
+                            ;; Use a `list' so it's unique and won't get
+                            ;; accidentally merged with neighbouring images.
+                           tag 'mouse-face (list widget-button-pressed-face))))
     (insert tag)))
 
 (defun widget-move-and-invoke (event)
@@ -912,71 +925,94 @@ Recommended as a parent keymap for modes using widgets.")
 ;; backward-compatibility alias
 (put 'widget-button-pressed-face 'face-alias 'widget-button-pressed)
 
+(defvar widget-button-click-moves-point nil
+  "If non-nil, `widget-button-click' moves point to a button after invoking it.
+If nil, point returns to its original position after invoking a button.")
+
 (defun widget-button-click (event)
   "Invoke the button that the mouse is pointing at."
   (interactive "e")
   (if (widget-event-point event)
-      (let* ((pos (widget-event-point event))
+      (let* ((oevent event)
+            (mouse-1 (memq (event-basic-type event) '(mouse-1 down-mouse-1)))
+            (pos (widget-event-point event))
             (start (event-start event))
             (button (get-char-property
                      pos 'button (and (windowp (posn-window start))
-                                      (window-buffer (posn-window start))))))
-       (if button
-           ;; Mouse click on a widget button.  Do the following
-           ;; in a save-excursion so that the click on the button
-           ;; doesn't change point.
-           (save-selected-window
-             (select-window (posn-window (event-start event)))
-             (save-excursion
-               (goto-char (posn-point (event-start event)))
-               (let* ((overlay (widget-get button :button-overlay))
-                      (pressed-face (or (widget-get button :pressed-face)
-                                        widget-button-pressed-face))
-                      (face (overlay-get overlay 'face))
-                      (mouse-face (overlay-get overlay 'mouse-face)))
-                 (unwind-protect
-                     ;; Read events, including mouse-movement events
-                     ;; until we receive a release event.  Highlight/
-                     ;; unhighlight the button the mouse was initially
-                     ;; on when we move over it.
-                     (save-excursion
-                       (when face      ; avoid changing around image
-                         (overlay-put overlay 'face pressed-face)
-                         (overlay-put overlay 'mouse-face pressed-face))
-                       (unless (widget-apply button :mouse-down-action event)
-                         (let ((track-mouse t))
-                           (while (not (widget-button-release-event-p event))
-                             (setq event (read-event)
-                                   pos (widget-event-point event))
-                             (if (and pos
-                                      (eq (get-char-property pos 'button)
-                                          button))
-                                 (when face
-                                   (overlay-put overlay 'face pressed-face)
-                                   (overlay-put overlay 'mouse-face pressed-face))
-                               (overlay-put overlay 'face face)
-                               (overlay-put overlay 'mouse-face mouse-face)))))
-
-                       ;; When mouse is released over the button, run
-                       ;; its action function.
-                       (when (and pos
-                                  (eq (get-char-property pos 'button) button))
-                         (widget-apply-action button event)))
-                   (overlay-put overlay 'face face)
-                   (overlay-put overlay 'mouse-face mouse-face))))
-
-             (unless (pos-visible-in-window-p (widget-event-point event))
-               (mouse-set-point event)
-               (beginning-of-line)
-               (recenter))
-             )
-
+                                      (window-buffer (posn-window start)))))
+            newpoint)
+       (when (or (null button)
+                 (catch 'button-press-cancelled
+             ;; Mouse click on a widget button.  Do the following
+             ;; in a save-excursion so that the click on the button
+             ;; doesn't change point.
+             (save-selected-window
+               (select-window (posn-window (event-start event)))
+               (save-excursion
+                 (goto-char (posn-point (event-start event)))
+                 (let* ((overlay (widget-get button :button-overlay))
+                        (pressed-face (or (widget-get button :pressed-face)
+                                          widget-button-pressed-face))
+                        (face (overlay-get overlay 'face))
+                        (mouse-face (overlay-get overlay 'mouse-face)))
+                   (unwind-protect
+                       ;; Read events, including mouse-movement
+                       ;; events, waiting for a release event.  If we
+                       ;; began with a mouse-1 event and receive a
+                       ;; movement event, that means the user wants
+                       ;; to perform drag-selection, so cancel the
+                       ;; button press and do the default mouse-1
+                       ;; action.  For mouse-2, just highlight/
+                       ;; unhighlight the button the mouse was
+                       ;; initially on when we move over it.
+                       (save-excursion
+                         (when face    ; avoid changing around image
+                           (overlay-put overlay 'face pressed-face)
+                           (overlay-put overlay 'mouse-face pressed-face))
+                         (unless (widget-apply button :mouse-down-action event)
+                           (let ((track-mouse t))
+                             (while (not (widget-button-release-event-p event))
+                               (setq event (read-event))
+                               (when (and mouse-1 (mouse-movement-p event))
+                                 (push event unread-command-events)
+                                 (setq event oevent)
+                                 (throw 'button-press-cancelled t))
+                               (unless (or (integerp event)
+                                           (memq (car event) '(switch-frame select-window))
+                                           (eq (car event) 'scroll-bar-movement))
+                                 (setq pos (widget-event-point event))
+                                 (if (and pos
+                                          (eq (get-char-property pos 'button)
+                                              button))
+                                     (when face
+                                       (overlay-put overlay 'face pressed-face)
+                                       (overlay-put overlay 'mouse-face pressed-face))
+                                   (overlay-put overlay 'face face)
+                                   (overlay-put overlay 'mouse-face mouse-face))))))
+
+                         ;; When mouse is released over the button, run
+                         ;; its action function.
+                         (when (and pos (eq (get-char-property pos 'button) button))
+                           (goto-char pos)
+                           (widget-apply-action button event)
+                           (if widget-button-click-moves-point
+                               (setq newpoint (point)))))
+                     (overlay-put overlay 'face face)
+                     (overlay-put overlay 'mouse-face mouse-face))))
+
+               (if newpoint (goto-char newpoint))
+               ;; This loses if the widget action switches windows. -- cyd
+               ;; (unless (pos-visible-in-window-p (widget-event-point event))
+               ;;   (mouse-set-point event)
+               ;;   (beginning-of-line)
+               ;;   (recenter))
+               )
+             nil))
          (let ((up t) command)
            ;; Mouse click not on a widget button.  Find the global
            ;; command to run, and check whether it is bound to an
            ;; up event.
-           (mouse-set-point event)
-           (if (memq (event-basic-type event) '(mouse-1 down-mouse-1))
+           (if mouse-1
                (cond ((setq command    ;down event
                             (lookup-key widget-global-map [down-mouse-1]))
                       (setq up nil))
@@ -1414,7 +1450,7 @@ The value of the :type attribute should be an unconverted widget type."
 
 (defun widget-default-complete (widget)
   "Call the value of the :complete-function property of WIDGET.
-If that does not exists, call the value of `widget-complete-field'."
+If that does not exist, call the value of `widget-complete-field'."
   (call-interactively (or (widget-get widget :complete-function)
                          widget-complete-field)))
 
@@ -1467,6 +1503,8 @@ If that does not exists, call the value of `widget-complete-field'."
                      (delete-backward-char 1))
                    (insert ?\n)
                    (setq doc-end (point)))))
+              ((eq escape ?h)
+               (widget-add-documentation-string-button widget))
               ((eq escape ?v)
                (if (and button-begin (not button-end))
                    (widget-apply widget :value-create)
@@ -1492,44 +1530,7 @@ If that does not exists, call the value of `widget-complete-field'."
   (widget-clear-undo))
 
 (defun widget-default-format-handler (widget escape)
-  ;; We recognize the %h escape by default.
-  (let* ((buttons (widget-get widget :buttons)))
-    (cond ((eq escape ?h)
-          (let* ((doc-property (widget-get widget :documentation-property))
-                 (doc-try (cond ((widget-get widget :doc))
-                                ((functionp doc-property)
-                                 (funcall doc-property
-                                          (widget-get widget :value)))
-                                ((symbolp doc-property)
-                                 (documentation-property
-                                  (widget-get widget :value)
-                                  doc-property))))
-                 (doc-text (and (stringp doc-try)
-                                (> (length doc-try) 1)
-                                doc-try))
-                 (doc-indent (widget-get widget :documentation-indent)))
-            (when doc-text
-              (and (eq (preceding-char) ?\n)
-                   (widget-get widget :indent)
-                   (insert-char ?\s (widget-get widget :indent)))
-              ;; The `*' in the beginning is redundant.
-              (when (eq (aref doc-text  0) ?*)
-                (setq doc-text (substring doc-text 1)))
-              ;; Get rid of trailing newlines.
-              (when (string-match "\n+\\'" doc-text)
-                (setq doc-text (substring doc-text 0 (match-beginning 0))))
-              (push (widget-create-child-and-convert
-                     widget 'documentation-string
-                     :indent (cond ((numberp doc-indent )
-                                    doc-indent)
-                                   ((null doc-indent)
-                                    nil)
-                                   (t 0))
-                     doc-text)
-                    buttons))))
-         (t
-          (error "Unknown escape `%c'" escape)))
-    (widget-put widget :buttons buttons)))
+  (error "Unknown escape `%c'" escape))
 
 (defun widget-default-button-face-get (widget)
   ;; Use :button-face or widget-button-face
@@ -1617,7 +1618,7 @@ If that does not exists, call the value of `widget-complete-field'."
       (widget-princ-to-string (widget-get widget :value))))
 
 (defun widget-default-active (widget)
-  "Return t iff this widget active (user modifiable)."
+  "Return t if this widget is active (user modifiable)."
   (or (widget-get widget :always-active)
       (and (not (widget-get widget :inactive))
           (let ((parent (widget-get widget :parent)))
@@ -1641,13 +1642,32 @@ If that does not exists, call the value of `widget-complete-field'."
   (widget-default-action widget event))
 
 (defun widget-default-prompt-value (widget prompt value unbound)
-  "Read an arbitrary value.  Stolen from `set-variable'."
-;; (let ((initial (if unbound
-;; nil
-;; It would be nice if we could do a `(cons val 1)' here.
-;; (prin1-to-string (custom-quote value))))))
+  "Read an arbitrary value."
   (eval-minibuffer prompt))
 
+(defun widget-docstring (widget)
+  "Return the documentation string specificied by WIDGET, or nil if none.
+If WIDGET has a `:doc' property, that specifies the documentation string.
+Otherwise, try the `:documentation-property' property.  If this
+is a function, call it with the widget's value as an argument; if
+it is a symbol, use this symbol together with the widget's value
+as the argument to `documentation-property'."
+  (let ((doc (or (widget-get widget :doc)
+                (let ((doc-prop (widget-get widget :documentation-property))
+                      (value (widget-get widget :value)))
+                  (cond ((functionp doc-prop)
+                         (funcall doc-prop value))
+                        ((symbolp doc-prop)
+                         (documentation-property value doc-prop)))))))
+    (when (and (stringp doc) (> (length doc) 0))
+      ;; Remove any redundant `*' in the beginning.
+      (when (eq (aref doc 0) ?*)
+       (setq doc (substring doc 1)))
+      ;; Remove trailing newlines.
+      (when (string-match "\n+\\'" doc)
+       (setq doc (substring doc 0 (match-beginning 0))))
+      doc)))
+
 ;;; The `item' Widget.
 
 (define-widget 'item 'default
@@ -1696,7 +1716,7 @@ If END is omitted, it defaults to the length of LIST."
 ;;; The `push-button' Widget.
 
 ;; (defcustom widget-push-button-gui t
-;;   "If non nil, use GUI push buttons when available."
+;;   "If non-nil, use GUI push buttons when available."
 ;;   :group 'widgets
 ;;   :type 'boolean)
 
@@ -1828,7 +1848,9 @@ If END is omitted, it defaults to the length of LIST."
 ;;; The `editable-field' Widget.
 
 (define-widget 'editable-field 'default
-  "An editable text field."
+  "An editable text field.
+Note: In an `editable-field' widget, the `%v' escape must be preceded
+by some other text in the `:format' string (if specified)."
   :convert-widget 'widget-value-convert-widget
   :keymap widget-field-keymap
   :format "%v"
@@ -1850,7 +1872,7 @@ If END is omitted, it defaults to the length of LIST."
   "History of field minibuffer edits.")
 
 (defun widget-field-prompt-internal (widget prompt initial history)
-  "Read string for WIDGET promptinhg with PROMPT.
+  "Read string for WIDGET prompting with PROMPT.
 INITIAL is the initial input and HISTORY is a symbol containing
 the earlier input."
   (read-string prompt initial history))
@@ -2539,7 +2561,7 @@ Return an alist of (TYPE MATCH)."
 ;;; The `editable-list' Widget.
 
 ;; (defcustom widget-editable-list-gui nil
-;;   "If non nil, use GUI push-buttons in editable list when available."
+;;   "If non-nil, use GUI push-buttons in editable list when available."
 ;;   :type 'boolean
 ;;   :group 'widgets)
 
@@ -2841,7 +2863,7 @@ The first group should be the link itself."
 
 (defcustom widget-documentation-link-p 'intern-soft
   "Predicate used to test if a string is useful as a link.
-The value should be a function.  The function will be called one
+The value should be a function.  The function will be called with one
 argument, a string, and should return non-nil if there should be a
 link for that string."
   :type 'function
@@ -2887,7 +2909,8 @@ link for that string."
   "A documentation string."
   :format "%v"
   :action 'widget-documentation-string-action
-  :value-create 'widget-documentation-string-value-create)
+  :value-create 'widget-documentation-string-value-create
+  :visibility-widget 'visibility)
 
 (defun widget-documentation-string-value-create (widget)
   ;; Insert documentation string.
@@ -2899,11 +2922,13 @@ link for that string."
        (let ((before (substring doc 0 (match-beginning 0)))
              (after (substring doc (match-beginning 0)))
              button)
+         (when (and indent (not (zerop indent)))
+           (insert-char ?\s indent))
          (insert before ?\s)
          (widget-documentation-link-add widget start (point))
          (setq button
                (widget-create-child-and-convert
-                widget 'visibility
+                widget (widget-get widget :visibility-widget)
                 :help-echo "Show or hide rest of the documentation."
                 :on "Hide Rest"
                 :off "More"
@@ -2917,6 +2942,8 @@ link for that string."
            (insert after)
            (widget-documentation-link-add widget start (point)))
          (widget-put widget :buttons (list button)))
+      (when (and indent (not (zerop indent)))
+       (insert-char ?\s indent))
       (insert doc)
       (widget-documentation-link-add widget start (point))))
   (insert ?\n))
@@ -2928,6 +2955,29 @@ link for that string."
                (not (widget-get parent :documentation-shown))))
   ;; Redraw.
   (widget-value-set widget (widget-value widget)))
+
+(defun widget-add-documentation-string-button (widget &rest args)
+  "Insert a new `documentation-string' widget based on WIDGET.
+The new widget becomes a child of WIDGET, and is also added to
+its `:buttons' list.  The documentation string is found from
+WIDGET using the function `widget-docstring'.
+Optional ARGS specifies additional keyword arguments for the
+`documentation-string' widget."
+  (let ((doc (widget-docstring widget))
+       (indent (widget-get widget :indent))
+       (doc-indent (widget-get widget :documentation-indent)))
+    (when doc
+      (and (eq (preceding-char) ?\n)
+          indent
+          (insert-char ?\s indent))
+      (unless (or (numberp doc-indent) (null doc-indent))
+       (setq doc-indent 0))
+      (widget-put widget :buttons
+                 (cons (apply 'widget-create-child-and-convert
+                              widget 'documentation-string
+                              :indent doc-indent
+                              (nconc args (list doc)))
+                       (widget-get widget :buttons))))))
 \f
 ;;; The Sexp Widgets.