Bounds check returns value.

heathhenley · heathhenley · commit dc55f80f3131 · 2017-02-24T18:59:07.000-05:00
diff --git a/lib/matplotlib/widgets.py b/lib/matplotlib/widgets.py
@@ -330,13 +330,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
@@ -361,12 +365,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."""
@@ -388,7 +387,7 @@ def _value_in_bounds(self, val):
             if not self.closedmax:
                 return
             val = self.slidermax.val
-        return val
+	return val
 
     def _update(self, event):
         """update the slider position"""
@@ -412,7 +411,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
