Bounds check returns value.

heathhenley · heathhenley · commit b7923c95db09 · 2017-02-23T14:14:59.000-05:00
diff --git a/lib/matplotlib/widgets.py b/lib/matplotlib/widgets.py
@@ -334,13 +334,17 @@ def __init__(self, ax, label, valmin, valmax, valinit=0.5, valfmt='%1.2f',
         if slidermax is not None and not hasattr(slidermax, 'val'):
             raise ValueError("Argument slidermax ({}) has no 'val'"
                              .format(type(slidermax)))
-
+        self.closedmin = closedmin
+        self.closedmax = closedmax
+        self.slidermin = slidermin
+        self.slidermax = slidermax
+        self.drag_active = False
         self.valmin = valmin
         self.valmax = valmax
+        valinit = self._value_in_bounds(valinit)
         self.val = valinit
         self.valinit = valinit
         self.poly = ax.axvspan(valmin, valinit, 0, 1, **kwargs)
-
         self.vline = ax.axvline(valinit, 0, 1, color='r', lw=1)
 
         self.valfmt = valfmt
@@ -365,12 +369,7 @@ def __init__(self, ax, label, valmin, valmax, valinit=0.5, valfmt='%1.2f',
         self.cnt = 0
         self.observers = {}
 
-        self.closedmin = closedmin
-        self.closedmax = closedmax
-        self.slidermin = slidermin
-        self.slidermax = slidermax
-        self.drag_active = False
-        self._value_in_bounds(valinit)
+        self.set_val(valinit)
 
     def _value_in_bounds(self, val):
         """ Makes sure self.val is with given bounds."""
@@ -392,7 +391,7 @@ def _value_in_bounds(self, val):
             if not self.closedmax:
                 return
             val = self.slidermax.val
-        self.set_val(val)
+	return val
 
     def _update(self, event):
         """update the slider position"""
@@ -416,7 +415,7 @@ def _update(self, event):
             event.canvas.release_mouse(self.ax)
             return
         val = event.xdata
-        self._value_in_bounds(val)
+        self.set_val(self._value_in_bounds(val))
 
     def set_val(self, val):
         xy = self.poly.xy
