Improve signature of set_span_cursor

Author: dstansby

lib/matplotlib/widgets.py

@@ -2648,7 +2648,7 @@ def _handles_artists(self):
         else:
             return ()
 
-    def _set_span_cursor(self, enabled):
+    def _set_span_cursor(self, *, enabled):
         """Update the canvas cursor based on direction of the selector."""
         if enabled:
             cursor = (backend_tools.Cursors.RESIZE_HORIZONTAL
@@ -2667,7 +2667,7 @@ def connect_default_events(self):
 
     def _press(self, event):
         """Button press event handler."""
-        self._set_span_cursor(True)
+        self._set_span_cursor(enabled=True)
         if self._interactive and self._selection_artist.get_visible():
             self._set_active_handle(event)
         else:
@@ -2717,7 +2717,7 @@ def direction(self, direction):
 
     def _release(self, event):
         """Button release event handler."""
-        self._set_span_cursor(False)
+        self._set_span_cursor(enabled=False)
 
         if not self._interactive:
             self._selection_artist.set_visible(False)
@@ -2759,7 +2759,7 @@ def _hover(self, event):
             return
 
         _, e_dist = self._edge_handles.closest(event.x, event.y)
-        self._set_span_cursor(e_dist <= self.grab_range)
+        self._set_span_cursor(enabled=e_dist <= self.grab_range)
 
     def _onmove(self, event):
         """Motion notify event handler."""

lib/matplotlib/widgets.pyi

@@ -337,6 +337,7 @@ class SpanSelector(_SelectorWidget):
         _props: dict[str, Any] | None = ...,
         _init: bool = ...,
     ) -> None: ...
+    def _set_span_cursor(self, *, enabled: bool) -> None: ...
     def connect_default_events(self) -> None: ...
     @property
     def direction(self) -> Literal["horizontal", "vertical"]: ...