(symbol-file): Remove unused variable `functions'.
[bpt/emacs.git] / lisp / facemenu.el
index 812f5a2..37399e0 100644 (file)
@@ -1,6 +1,6 @@
 ;;; facemenu.el --- create a face menu for interactively adding fonts to text
 
-;; Copyright (c) 1994, 1995, 1996, 2001 Free Software Foundation, Inc.
+;; Copyright (c) 1994, 1995, 1996, 2001, 2002 Free Software Foundation, Inc.
 
 ;; Author: Boris Goldowsky <boris@gnu.org>
 ;; Keywords: faces
 
 ;; This file defines a menu of faces (bold, italic, etc) which allows you to
 ;; set the face used for a region of the buffer.  Some faces also have
-;; keybindings, which are shown in the menu.  Faces with names beginning with
-;; "fg:" or "bg:", as in "fg:red", are treated specially.
-;; Such faces are assumed to consist only of a foreground (if "fg:") or
-;; background (if "bg:") color.  They are thus put into the color submenus
-;; rather than the general Face submenu.  These faces can also be
-;; automatically created by selecting the "Other..." menu items in the
-;; "Foreground" and "Background" submenus.
+;; keybindings, which are shown in the menu.
 ;;
 ;; The menu also contains submenus for indentation and justification-changing
 ;; commands.
@@ -92,7 +86,9 @@
 
 ;;; Code:
 
-(provide 'facemenu)
+(eval-when-compile 
+  (require 'help)
+  (require 'button))
 
 ;;; Provide some binding for startup:
 ;;;###autoload (define-key global-map "\M-g" 'facemenu-keymap)
     (italic      . "i")
     (bold-italic . "l") ; {bold} intersect {italic} = {l}
     (underline   . "u"))
-  "Alist of interesting faces and keybindings. 
+  "Alist of interesting faces and keybindings.
 Each element is itself a list: the car is the name of the face,
 the next element is the key to use as a keyboard equivalent of the menu item;
 the binding is made in `facemenu-keymap'.
@@ -240,8 +236,8 @@ when they are created."
 (let ((map facemenu-menu))
   (define-key map [dc] (cons (purecopy "Display Colors") 'list-colors-display))
   (define-key map [df] (cons (purecopy "Display Faces") 'list-faces-display))
-  (define-key map [dp] (cons (purecopy "List Properties")
-                            'list-text-properties-at))
+  (define-key map [dp] (cons (purecopy "Describe Properties")
+                            'describe-text-properties))
   (define-key map [ra] (cons (purecopy "Remove Text Properties")
                             'facemenu-remove-all))
   (define-key map [rm] (cons (purecopy "Remove Face Properties")
@@ -346,7 +342,7 @@ typing a character to insert cancels the specification."
 ;;;###autoload
 (defun facemenu-set-foreground (color &optional start end)
   "Set the foreground COLOR of the region or next character typed.
-The color is prompted for.  A face named `fg:color' is used \(or created).
+This command reads the color in the minibuffer.
 
 If the region is active (normally true except in Transient Mark mode)
 and there is no prefix argument, this command sets the region to the
@@ -364,13 +360,13 @@ typing a character to insert cancels the specification."
                         (region-end))))
   (unless (color-defined-p color)
     (message "Color `%s' undefined" color))
-  (facemenu-add-new-face color 'facemenu-foreground-menu)
+  (facemenu-add-new-color color 'facemenu-foreground-menu)
   (facemenu-add-face (list (list :foreground color)) start end))
 
 ;;;###autoload
 (defun facemenu-set-background (color &optional start end)
   "Set the background COLOR of the region or next character typed.
-Reads the color in the minibuffer.
+This command reads the color in the minibuffer.
 
 If the region is active (normally true except in Transient Mark mode)
 and there is no prefix argument, this command sets the region to the
@@ -388,7 +384,7 @@ typing a character to insert cancels the specification."
                         (region-end))))
   (unless (color-defined-p color)
     (message "Color `%s' undefined" color))
-  (facemenu-add-new-face color 'facemenu-background-menu)
+  (facemenu-add-new-color color 'facemenu-background-menu)
   (facemenu-add-face (list (list :background color)) start end))
 
 ;;;###autoload
@@ -462,48 +458,13 @@ These special properties include `invisible', `intangible' and `read-only'."
   (let ((inhibit-read-only t))
     (remove-text-properties 
      start end '(invisible nil intangible nil read-only nil))))
-
-;;;###autoload
-(defun list-text-properties-at (p)
-  "Pop up a buffer listing text-properties at LOCATION."
-  (interactive "d")
-  (let ((props (text-properties-at p))
-       category
-       str)
-    (if (null props)
-       (message "None")
-      (if (and (not (cdr (cdr props)))
-              (not (eq (car props) 'category))
-              (< (length (setq str (format "Text property at %d:  %s  %S"
-                                           p (car props) (car (cdr props)))))
-                 (frame-width)))
-         (message "%s" str)
-       (with-output-to-temp-buffer "*Text Properties*"
-         (princ (format "Text properties at %d:\n\n" p))
-         (setq help-xref-stack nil)
-         (while props
-           (if (eq (car props) 'category)
-               (setq category (car (cdr props))))
-           (princ (format "%-20s %S\n"
-                          (car props) (car (cdr props))))
-           (setq props (cdr (cdr props))))
-         (if category
-             (progn
-               (setq props (symbol-plist category))
-               (princ (format "\nCategory %s:\n\n" category))
-               (while props
-                 (princ (format "%-20s %S\n"
-                                (car props) (car (cdr props))))
-                 (if (eq (car props) 'category)
-                     (setq category (car (cdr props))))
-                 (setq props (cdr (cdr props)))))))))))
-
+\f
 ;;;###autoload
 (defun facemenu-read-color (&optional prompt)
   "Read a color using the minibuffer."
   (let ((col (completing-read (or prompt "Color: ") 
                              (or facemenu-color-alist
-                                 (mapcar 'list (defined-colors)))
+                                 (defined-colors))
                              nil t)))
     (if (equal "" col)
        nil
@@ -516,14 +477,19 @@ If the optional argument LIST is non-nil, it should be a list of
 colors to display.  Otherwise, this command computes a list
 of colors that the current display can handle."
   (interactive)
-  (when (null list)
+  (when (and (null list) (> (display-color-cells) 0))
     (setq list (defined-colors))
     ;; Delete duplicate colors.
     (let ((l list))
       (while (cdr l)
        (if (facemenu-color-equal (car l) (car (cdr l)))
            (setcdr l (cdr (cdr l)))
-         (setq l (cdr l))))))
+         (setq l (cdr l)))))
+    (when (memq (display-visual-class) '(gray-scale pseudo-color direct-color))
+      ;; Don't show more than what the display can handle.
+      (let ((lc (nthcdr (1- (display-color-cells)) list)))
+       (if lc
+           (setcdr lc nil)))))
   (with-output-to-temp-buffer "*Colors*"
     (save-excursion
       (set-buffer standard-output)
@@ -555,7 +521,7 @@ If START is nil or START to END is empty, add FACE to next typed character
 instead.  For each section of that region that has a different face property,
 FACE will be consed onto it, and other faces that are completely hidden by
 that will be removed from the list.
-If `facemenu-add-face-function' and maybe `facemenu-end-add-face' are non-`nil'
+If `facemenu-add-face-function' and maybe `facemenu-end-add-face' are non-nil,
 they are used to set the face information.
 
 As a special case, if FACE is `default', then the region is left with NO face
@@ -611,7 +577,7 @@ If the optional argument FRAME is given, use the faces in that frame; otherwise
 use the selected frame.  If t, then the global, non-frame faces are used."
   (let* ((mask-atts (copy-sequence
                     (if (consp (car face-list))
-                        (face-attribute-vector (car face-list))
+                        (face-attributes-as-vector (car face-list))
                       (or (internal-lisp-face-p (car face-list) frame)
                           (check-face (car face-list))))))
         (active-list (list (car face-list)))
@@ -620,7 +586,7 @@ use the selected frame.  If t, then the global, non-frame faces are used."
     (while face-list
       (if (let ((face-atts
                 (if (consp (car face-list))
-                    (face-attribute-vector (car face-list))
+                    (face-attributes-as-vector (car face-list))
                   (or (internal-lisp-face-p (car face-list) frame)
                       (check-face (car face-list)))))
                (i mask-len)
@@ -637,43 +603,29 @@ use the selected frame.  If t, then the global, non-frame faces are used."
 (defun facemenu-get-face (symbol)
   "Make sure FACE exists.
 If not, create it and add it to the appropriate menu.  Return the SYMBOL."
-  (let ((name (symbol-name symbol))
-       foreground)
+  (let ((name (symbol-name symbol)))
     (cond ((facep symbol))
          (t (make-face symbol))))
   symbol)
 
-(defun facemenu-add-new-face (face-or-color &optional menu)
-  "Add FACE-OR-COLOR (a face or a color) to the appropriate Face menu.
-If MENU is nil, then FACE-OR-COLOR is a face to be added
-to `facemenu-face-menu'.  If MENU is `facemenu-foreground-menu'
-or `facemenu-background-menu', FACE-OR-COLOR is a color
-to be added to the specified menu.
+(defun facemenu-add-new-face (face)
+  "Add FACE (a face) to the Face menu.
 
 This is called whenever you create a new face."
   (let* (name
         symbol
-        docstring
-        (key (cdr (assoc face-or-color facemenu-keybindings)))
+        menu docstring
+        (key (cdr (assoc face facemenu-keybindings)))
         function menu-val)
-    (if (symbolp face-or-color)
-       (setq name (symbol-name face-or-color)
-             symbol face-or-color)
-      (setq name face-or-color
+    (if (symbolp face)
+       (setq name (symbol-name face)
+             symbol face)
+      (setq name face
            symbol (intern name)))
-    (cond ((eq menu 'facemenu-foreground-menu)
-          (setq docstring
-                (format "Select foreground color %s for subsequent insertion."
-                        name)))
-         ((eq menu 'facemenu-background-menu)
-          (setq docstring
-                (format "Select background color %s for subsequent insertion."
-                        name)))
-         (t
-          (setq menu 'facemenu-face-menu)
-          (setq docstring
-                (format "Select face `%s' for subsequent insertion."
-                        name))))
+    (setq menu 'facemenu-face-menu)
+    (setq docstring
+         (format "Select face `%s' for subsequent insertion."
+                 name))
     (cond ((eq t facemenu-unlisted-faces))
          ((memq symbol facemenu-unlisted-faces))
          ;; test against regexps in facemenu-unlisted-faces
@@ -691,7 +643,12 @@ This is called whenever you create a new face."
                 `(lambda ()
                    ,docstring
                    (interactive)
-                   (facemenu-set-face (quote ,symbol))))
+                   (facemenu-set-face
+                    (quote ,symbol)
+                    (if (and mark-active (not current-prefix-arg))
+                        (region-beginning))
+                    (if (and mark-active (not current-prefix-arg))
+                        (region-end)))))
           (define-key 'facemenu-keymap key (cons name function))
           (define-key menu key (cons name function)))
          ((facemenu-iterate ; check if equivalent face is already in the menu
@@ -710,6 +667,48 @@ This is called whenever you create a new face."
             (define-key menu key (cons name function))))))
   nil) ; Return nil for facemenu-iterate
 
+(defun facemenu-add-new-color (color &optional menu)
+  "Add COLOR (a color name string) to the appropriate Face menu.
+MENU should be `facemenu-foreground-menu' or
+`facemenu-background-menu'.
+
+This is called whenever you use a new color."
+  (let* (name
+        symbol
+        docstring
+        function menu-val key
+        (color-p (memq menu '(facemenu-foreground-menu
+                              facemenu-background-menu))))
+    (unless (stringp color)
+      (error "%s is not a color" color))
+    (setq name color
+         symbol (intern name))
+
+    (cond ((eq menu 'facemenu-foreground-menu)
+          (setq docstring
+                (format "Select foreground color %s for subsequent insertion."
+                        name)))
+         ((eq menu 'facemenu-background-menu)
+          (setq docstring
+                (format "Select background color %s for subsequent insertion."
+                        name))))
+    (cond ((facemenu-iterate ; check if equivalent face is already in the menu
+           (lambda (m) (and (listp m) 
+                            (symbolp (car m))
+                            (stringp (cadr m))
+                            (string-equal (cadr m) color)))
+           (cdr (symbol-function menu))))
+         (t   ; No keyboard equivalent.  Figure out where to put it:
+          (setq key (vector symbol)
+                function 'facemenu-set-face-from-menu
+                menu-val (symbol-function menu))
+          (if (and facemenu-new-faces-at-end
+                  (> (length menu-val) 3))
+              (define-key-after menu-val key (cons name function)
+                (car (nth (- (length menu-val) 3) menu-val)))
+            (define-key menu key (cons name function))))))
+  nil) ; Return nil for facemenu-iterate
+
 (defun facemenu-complete-face-list (&optional oldlist)
   "Return list of all faces that look different.
 Starts with given ALIST of faces, and adds elements only if they display 
@@ -734,4 +733,5 @@ Returns the non-nil value it found, or nil if all were nil."
 
 (facemenu-update)
 
+(provide 'facemenu)
 ;;; facemenu.el ends here