MNT: move canvas is None check to ignore method

tacaswell · tacaswell · commit 9f1da2127f08 · 2025-07-18T09:39:30.000-04:00
diff --git a/lib/matplotlib/widgets.py b/lib/matplotlib/widgets.py
@@ -146,6 +146,10 @@ def _get_data_coords(self, event):
         return ((event.xdata, event.ydata) if event.inaxes is self.ax
                 else self.ax.transData.inverted().transform((event.x, event.y)))
 
+    def ignore(self, event):
+        # docstring inherited
+        return super().ignore(event) or self.canvas is None
+
 
 class Button(AxesWidget):
     """
@@ -1087,7 +1091,7 @@ def __init__(self, ax, labels, actives=None, *, useblit=True,
 
     def _clear(self, event):
         """Internal event handler to clear the buttons."""
-        if self.ignore(event) or self.canvas is None or self.canvas.is_saving():
+        if self.ignore(event) or self.canvas.is_saving():
             return
         self._background = self.canvas.copy_from_bbox(self.ax.bbox)
         self.ax.draw_artist(self._checks)
@@ -1665,7 +1669,7 @@ def __init__(self, ax, labels, active=0, activecolor=None, *,
 
     def _clear(self, event):
         """Internal event handler to clear the buttons."""
-        if self.ignore(event) or self.canvas is None or self.canvas.is_saving():
+        if self.ignore(event) or self.canvas.is_saving():
             return
         self._background = self.canvas.copy_from_bbox(self.ax.bbox)
         self.ax.draw_artist(self._buttons)
@@ -2183,9 +2187,9 @@ def connect_default_events(self):
 
     def ignore(self, event):
         # docstring inherited
-        if self.canvas is None:
+        if super().ignore(event):
             return True
-        if not self.active or not self.ax.get_visible():
+        if not self.ax.get_visible():
             return True
         # If canvas was locked
         if not self.canvas.widgetlock.available(self):
