(undigestify-rmail-message): Better error messages.
[bpt/emacs.git] / lisp / scroll-bar.el
index 86f6ea0..ca7d145 100644 (file)
@@ -1,6 +1,6 @@
 ;;; scroll-bar.el --- window system-independent scroll bar support.
 
-;;; Copyright (C) 1993 Free Software Foundation, Inc.
+;;; Copyright (C) 1993, 1994, 1995 Free Software Foundation, Inc.
 
 ;; Maintainer: FSF
 ;; Keywords: hardware
 \f
 ;;;; Utilities.
 
+(defun scroll-bar-event-ratio (event)
+  "Given a scroll bar event EVENT, return the scroll bar position as a ratio.
+The value is a cons cell (PORTION . WHOLE) containing two integers
+whose ratio gives the event's vertical position in the scroll bar, with 0
+referring to the top and 1 to the bottom."
+  (nth 2 event))
+
 (defun scroll-bar-scale (num-denom whole)
   "Given a pair (NUM . DENOM) and WHOLE, return (/ (* NUM WHOLE) DENOM).
 This is handy for scaling a position on a scroll bar into real units,
@@ -48,7 +55,6 @@ that scroll bar position."
 
 \f
 ;;;; Helpful functions for enabling and disabling scroll bars.
-(defvar scroll-bar-mode nil)
 
 (defun scroll-bar-mode (flag)
   "Toggle display of vertical scroll bars on each frame.
@@ -57,25 +63,37 @@ created in the future.
 With a numeric argument, if the argument is negative,
 turn off scroll bars; otherwise, turn on scroll bars."
   (interactive "P")
-  (setq scroll-bar-mode (if (null flag) (not scroll-bar-mode)
-                         (or (not (numberp flag)) (>= flag 0))))
-  (mapcar
-   (function
-    (lambda (param-name)
-      (let ((parameter (assq param-name default-frame-alist)))
-       (if (consp parameter)
-           (setcdr parameter scroll-bar-mode)
-         (setq default-frame-alist
-               (cons (cons param-name scroll-bar-mode)
-                     default-frame-alist))))))
-   '(vertical-scroll-bars horizontal-scroll-bars))
-  (let ((frames (frame-list)))
-    (while frames
-      (modify-frame-parameters
-       (car frames)
-       (list (cons 'vertical-scroll-bars scroll-bar-mode)
-            (cons 'horizontal-scroll-bars scroll-bar-mode)))
-      (setq frames (cdr frames)))))
+  (if flag (setq flag (prefix-numeric-value flag)))
+
+  ;; Obtain the current setting by looking at default-frame-alist.
+  (let ((scroll-bar-mode
+        (let ((assq (assq 'vertical-scroll-bars default-frame-alist)))
+          (if assq (cdr assq) t))))
+
+    ;; Tweedle it according to the argument.
+    (setq scroll-bar-mode (if (null flag) (not scroll-bar-mode)
+                           (or (not (numberp flag)) (>= flag 0))))
+
+    ;; Apply it to default-frame-alist.
+    (mapcar
+     (function
+      (lambda (param-name)
+       (let ((parameter (assq param-name default-frame-alist)))
+         (if (consp parameter)
+             (setcdr parameter scroll-bar-mode)
+           (setq default-frame-alist
+                 (cons (cons param-name scroll-bar-mode)
+                       default-frame-alist))))))
+     '(vertical-scroll-bars horizontal-scroll-bars))
+
+    ;; Apply it to existing frames.
+    (let ((frames (frame-list)))
+      (while frames
+       (modify-frame-parameters
+        (car frames)
+        (list (cons 'vertical-scroll-bars scroll-bar-mode)
+              (cons 'horizontal-scroll-bars scroll-bar-mode)))
+       (setq frames (cdr frames))))))
 \f
 ;;;; Buffer navigation using the scroll bar.
 
@@ -90,7 +108,9 @@ EVENT should be a scroll bar click or drag event."
     (save-excursion
       (set-buffer (window-buffer window))
       (save-excursion
-       (goto-char (scroll-bar-scale portion-whole (buffer-size)))
+       (goto-char (+ (point-min)
+                     (scroll-bar-scale portion-whole
+                                       (- (point-max) (point-min)))))
        (beginning-of-line)
        (set-window-start window (point))))))
 
@@ -101,7 +121,10 @@ EVENT should be a scroll bar click or drag event."
         (portion-whole (nth 2 start-position)))
     (save-excursion
       (set-buffer (window-buffer window))
-      (goto-char (scroll-bar-scale portion-whole (buffer-size)))
+      ;; Calculate position relative to the accessible part of the buffer.
+      (goto-char (+ (point-min)
+                   (scroll-bar-scale portion-whole
+                                     (- (point-max) (point-min)))))
       (beginning-of-line)
       (set-window-start window (point)))))
 
@@ -109,18 +132,24 @@ EVENT should be a scroll bar click or drag event."
   "Scroll the window by dragging the scroll bar slider.
 If you click outside the slider, the window scrolls to bring the slider there."
   (interactive "e")
-  (let* (done)
-    (scroll-bar-drag-1 event)
-    (track-mouse
-      (while (not done)
-       (setq event (read-event))
-       (if (eq (car-safe event) 'mouse-movement)
-           (setq event (read-event)))
-       (cond ((eq (car-safe event) 'scroll-bar-movement)
-              (scroll-bar-drag-1 event))
-             (t
-              ;; Exit when we get the drag event; ignore that event.
-              (setq done t)))))))
+  (let* (done
+        (echo-keystrokes 0))
+    (or point-before-scroll
+       (setq point-before-scroll (point)))
+    ;; Our scrolling can move point; don't let that clear point-before-scroll.
+    (let (point-before-scroll)
+      (scroll-bar-drag-1 event)
+      (track-mouse
+       (while (not done)
+         (setq event (read-event))
+         (if (eq (car-safe event) 'mouse-movement)
+             (setq event (read-event)))
+         (cond ((eq (car-safe event) 'scroll-bar-movement)
+                (scroll-bar-drag-1 event))
+               (t
+                ;; Exit when we get the drag event; ignore that event.
+                (setq done t)))))
+      (sit-for 0))))
 
 (defun scroll-bar-scroll-down (event)
   "Scroll the window's top line down to the location of the scroll bar click.
@@ -132,9 +161,13 @@ EVENT should be a scroll bar click."
          (let* ((end-position (event-end event))
                 (window (nth 0 end-position))
                 (portion-whole (nth 2 end-position)))
-           (select-window window)
-           (scroll-down
-            (scroll-bar-scale portion-whole (1- (window-height))))))
+           (let (point-before-scroll)
+             (select-window window))
+           (or point-before-scroll
+               (setq point-before-scroll (point)))
+           (let (point-before-scroll)
+             (scroll-down
+              (scroll-bar-scale portion-whole (1- (window-height)))))))
       (select-window old-selected-window))))
 
 (defun scroll-bar-scroll-up (event)
@@ -147,9 +180,13 @@ EVENT should be a scroll bar click."
          (let* ((end-position (event-end event))
                 (window (nth 0 end-position))
                 (portion-whole (nth 2 end-position)))
-           (select-window window)
-           (scroll-up
-            (scroll-bar-scale portion-whole (1- (window-height))))))
+           (let (point-before-scroll)
+             (select-window window))
+           (or point-before-scroll
+               (setq point-before-scroll (point)))
+           (let (point-before-scroll)
+             (scroll-up
+              (scroll-bar-scale portion-whole (1- (window-height)))))))
       (select-window old-selected-window))))
 
 \f