Merge pull request #10107 from lkjell/fix_selector_widget_stay_span

jklymak · web-flow · commit 5e22976345ac · 2018-03-05T17:02:32.000-08:00
Fix stay_span to reset onclick in SpanSelector.
diff --git a/lib/matplotlib/widgets.py b/lib/matplotlib/widgets.py
@@ -1802,6 +1802,8 @@ def _press(self, event):
             self.pressv = xdata
         else:
             self.pressv = ydata
+
+        self._set_span_xy(event)
         return False
 
     def _release(self, event):
@@ -1840,6 +1842,26 @@ def _onmove(self, event):
         """on motion notify event"""
         if self.pressv is None:
             return
+
+        self._set_span_xy(event)
+
+        if self.onmove_callback is not None:
+            vmin = self.pressv
+            xdata, ydata = self._get_data(event)
+            if self.direction == 'horizontal':
+                vmax = xdata or self.prev[0]
+            else:
+                vmax = ydata or self.prev[1]
+
+            if vmin > vmax:
+                vmin, vmax = vmax, vmin
+            self.onmove_callback(vmin, vmax)
+
+        self.update()
+        return False
+
+    def _set_span_xy(self, event):
+        """Setting the span coordinates"""
         x, y = self._get_data(event)
         if x is None:
             return
@@ -1860,21 +1882,6 @@ def _onmove(self, event):
             self.rect.set_y(minv)
             self.rect.set_height(maxv - minv)
 
-        if self.onmove_callback is not None:
-            vmin = self.pressv
-            xdata, ydata = self._get_data(event)
-            if self.direction == 'horizontal':
-                vmax = xdata or self.prev[0]
-            else:
-                vmax = ydata or self.prev[1]
-
-            if vmin > vmax:
-                vmin, vmax = vmax, vmin
-            self.onmove_callback(vmin, vmax)
-
-        self.update()
-        return False
-
 
 class ToolHandles(object):
     """Control handles for canvas tools.
