]> code.delx.au - gnu-emacs/blobdiff - lisp/avoid.el
(describe-mode): Test mini-mode symbol for being
[gnu-emacs] / lisp / avoid.el
index 1c7ece39272cdd052ed84f6776610817e162ec01..f5712427964bff363d2040d295c8f9c375d85744 100644 (file)
@@ -2,7 +2,7 @@
 
 ;;; Copyright (C) 1993, 1994 Free Software Foundation, Inc.
 
-;; Author: Boris Goldowsky <boris@gnu.ai.mit.edu>
+;; Author: Boris Goldowsky <boris@gnu.org>
 ;; Keywords: mouse
 
 ;; This file is part of GNU Emacs.
@@ -30,7 +30,7 @@
 ;; To use, load or evaluate this file and type M-x mouse-avoidance-mode .
 ;; To set up permanently, put the following in your .emacs: 
 ;;
-;; (if window-system (mouse-avoidance-mode 'animate))
+;; (if (display-mouse-p) (mouse-avoidance-mode 'animate))
 ;;
 ;; Other legitimate alternatives include
 ;; `banish', `exile', `jump', `cat-and-mouse', and `proteus'.
@@ -41,7 +41,7 @@
 ;; For added silliness, make the animatee animate...
 ;; put something similar to the following into your .emacs:
 ;;
-;; (if window-system
+;; (if (eq window-system 'x)
 ;;     (mouse-avoidance-set-pointer-shape
 ;;          (eval (nth (random 4)
 ;;                     '(x-pointer-man x-pointer-spider
   :prefix "mouse-avoidance-"
   :group 'mouse)
 
-
+;;;###autoload
 (defcustom mouse-avoidance-mode nil
   "Activate mouse avoidance mode.  
 See function `mouse-avoidance-mode' for possible values.
-You must modify via \\[customize] for this variable to have an effect."
+Setting this variable directly does not take effect;
+use either \\[customize] or the function `mouse-avoidance-mode'."
   :set (lambda (symbol value)
         ;; 'none below prevents toggling when value is nil.
         (mouse-avoidance-mode (or value 'none))) 
@@ -89,7 +90,8 @@ You must modify via \\[customize] for this variable to have an effect."
                 (const animate) (const exile) (const proteus)
                 )
   :group 'avoid
-  :require 'avoid)
+  :require 'avoid
+  :version "20.3")
 
 
 (defcustom mouse-avoidance-nudge-dist 15
@@ -255,9 +257,9 @@ redefine this function to suit your own tastes."
   (if (null mouse-avoidance-pointer-shapes)
       (progn
        (setq mouse-avoidance-pointer-shapes
-             (mapcar '(lambda (x) (symbol-value (intern x)))
+             (mapcar (lambda (x) (symbol-value (intern x)))
                      (all-completions "x-pointer-" obarray
-                                      '(lambda (x) 
+                                      (lambda (x) 
                                          (and (boundp x)
                                               (integerp (symbol-value x)))))))
        (setq mouse-avoidance-n-pointer-shapes 
@@ -267,14 +269,31 @@ redefine this function to suit your own tastes."
 
 (defun mouse-avoidance-banish-hook ()
   (if (and (not executing-kbd-macro)   ; don't check inside macro
-          (mouse-avoidance-kbd-command (this-command-keys)))
+          (cadr (mouse-position))      ; don't move unless in an Emacs frame
+          ;; Don't do anything if last event was a mouse event.
+          (not (and (consp last-input-event)
+                    (symbolp (car last-input-event))
+                    (let ((modifiers (event-modifiers (car last-input-event))))
+                      (or (memq (car last-input-event)
+                                '(mouse-movement scroll-bar-movement))
+                          (memq 'click modifiers)
+                          (memq 'drag modifiers)
+                          (memq 'down modifiers))))))
       (mouse-avoidance-banish-mouse)))
 
 (defun mouse-avoidance-exile-hook ()
   ;; For exile mode, the state is nil when the mouse is in its normal
   ;; position, and set to the old mouse-position when the mouse is in exile.
   (if (and (not executing-kbd-macro)
-          (mouse-avoidance-kbd-command (this-command-keys)))
+          ;; Don't do anything if last event was a mouse event.
+          (not (and (consp last-input-event)
+                    (symbolp (car last-input-event))
+                    (let ((modifiers (event-modifiers (car last-input-event))))
+                      (or (memq (car last-input-event)
+                                '(mouse-movement scroll-bar-movement))
+                          (memq 'click modifiers)
+                          (memq 'drag modifiers)
+                          (memq 'down modifiers))))))
       (let ((mp (mouse-position)))
        (cond ((and (not mouse-avoidance-state)
                    (mouse-avoidance-too-close-p mp))
@@ -293,7 +312,15 @@ redefine this function to suit your own tastes."
 (defun mouse-avoidance-fancy-hook ()
   ;; Used for the "fancy" modes, ie jump et al.
   (if (and (not executing-kbd-macro)   ; don't check inside macro
-          (mouse-avoidance-kbd-command (this-command-keys))
+          ;; Don't do anything if last event was a mouse event.
+          (not (and (consp last-input-event)
+                    (symbolp (car last-input-event))
+                    (let ((modifiers (event-modifiers (car last-input-event))))
+                      (or (memq (car last-input-event)
+                                '(mouse-movement scroll-bar-movement))
+                          (memq 'click modifiers)
+                          (memq 'drag modifiers)
+                          (memq 'down modifiers)))))
           (mouse-avoidance-too-close-p (mouse-position)))
       (let ((old-pos (mouse-position)))
        (mouse-avoidance-nudge-mouse)
@@ -301,21 +328,6 @@ redefine this function to suit your own tastes."
            ;; This should never happen.
            (apply 'set-mouse-position old-pos)))))
 
-(defun mouse-avoidance-kbd-command (key)
-  "Return t if the KEYSEQENCE is composed of keyboard events only.
-Return nil if there are any lists in the key sequence."
-  (cond ((null key) nil)               ; Null event seems to be
-                                       ; returned occasionally.
-       ((not (vectorp key)) t)         ; Strings are keyboard events.
-       ((catch 'done
-          (let ((i 0)
-                (l (length key)))
-            (while (< i l)
-              (if (listp (aref key i))
-                  (throw 'done nil))
-              (setq i (1+ i))))
-          t))))
-
 ;;;###autoload
 (defun mouse-avoidance-mode (&optional mode)
   "Set cursor avoidance mode to MODE.