(XTset_vertical_scroll_bar): Don't clear a zero height

or width area.
(XTset_vertical_scroll_bar, x_scroll_bar_create): Don't configure
a widget to zero height.

2 files changed, 23 insertions, 14 deletions

diff --git a/src/ChangeLog b/src/ChangeLog
index 7f3285bdae2..135969d183f 100644
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,10 @@
+2001-03-02  Gerd Moellmann  <gerd@gnu.org>
+
+        * xterm.c (XTset_vertical_scroll_bar): Don't clear a zero height
+        or width area.
+        (XTset_vertical_scroll_bar, x_scroll_bar_create): Don't configure
+        a widget to zero height.
+        
 2001-03-01  Gerd Moellmann  <gerd@gnu.org>
 
         * window.c (Fwindow_end): Call move_it_past_eol only if
diff --git a/src/xterm.c b/src/xterm.c
index 9c75606b53b..6db999e241e 100644
--- a/src/xterm.c
+++ b/src/xterm.c
@@ -8420,15 +8420,15 @@ x_scroll_bar_create (w, top, left, width, height)
 
   /* Map the window/widget.  */
 #ifdef USE_TOOLKIT_SCROLL_BARS
- {
-   Widget scroll_bar = SCROLL_BAR_X_WIDGET (FRAME_X_DISPLAY (f), bar);
-   XtConfigureWidget (scroll_bar,
-                     left + VERTICAL_SCROLL_BAR_WIDTH_TRIM,
-                     top,
-                     width - VERTICAL_SCROLL_BAR_WIDTH_TRIM * 2,
-                     height, 0);
-   XtMapWidget (scroll_bar);
- }
+  {
+    Widget scroll_bar = SCROLL_BAR_X_WIDGET (FRAME_X_DISPLAY (f), bar);
+    XtConfigureWidget (scroll_bar,
+                       left + VERTICAL_SCROLL_BAR_WIDTH_TRIM,
+                       top,
+                       width - VERTICAL_SCROLL_BAR_WIDTH_TRIM * 2,
+                       max (height, 1), 0);
+    XtMapWidget (scroll_bar);
+    }
 #else /* not USE_TOOLKIT_SCROLL_BARS */
   XMapRaised (FRAME_X_DISPLAY (f), SCROLL_BAR_X_WINDOW (bar));
 #endif /* not USE_TOOLKIT_SCROLL_BARS */
@@ -8628,8 +8628,9 @@ XTset_vertical_scroll_bar (w, portion, whole, position)
   if (NILP (w->vertical_scroll_bar))
     {
       BLOCK_INPUT;
-      x_clear_area (FRAME_X_DISPLAY (f), FRAME_X_WINDOW (f),
-                    left, top, width, height, False);
+      if (width && height)
+        x_clear_area (FRAME_X_DISPLAY (f), FRAME_X_WINDOW (f),
+                      left, top, width, height, False);
       UNBLOCK_INPUT;
       bar = x_scroll_bar_create (w, top, sb_left, sb_width, height);
     }
@@ -8655,8 +8656,9 @@ XTset_vertical_scroll_bar (w, portion, whole, position)
 
       /* Since toolkit scroll bars are smaller than the space reserved
          for them on the frame, we have to clear "under" them.  */
-      x_clear_area (FRAME_X_DISPLAY (f), FRAME_X_WINDOW (f),
-                    left, top, width, height, False);
+      if (width && height)
+        x_clear_area (FRAME_X_DISPLAY (f), FRAME_X_WINDOW (f),
+                      left, top, width, height, False);
 
       /* Move/size the scroll bar widget.  */
       if (mask)
@@ -8664,7 +8666,7 @@ XTset_vertical_scroll_bar (w, portion, whole, position)
                            sb_left + VERTICAL_SCROLL_BAR_WIDTH_TRIM,
                            top,
                            sb_width - VERTICAL_SCROLL_BAR_WIDTH_TRIM * 2,
-                           height, 0);
+                           max (height, 1), 0);
 
 #else /* not USE_TOOLKIT_SCROLL_BARS */