*Buttons: use a semi-public self._buttons_ys variable

doronbehar · doronbehar · commit 74842be658ff · 2026-01-23T03:33:37.000+02:00
diff --git a/lib/matplotlib/widgets.py b/lib/matplotlib/widgets.py
@@ -1185,14 +1185,14 @@ def __init__(self, ax, labels, actives=None, *, useblit=True,
 
         self._useblit = useblit
 
-        ys = np.linspace(1, 0, len(labels)+2)[1:-1]
+        self._buttons_ys = np.linspace(1, 0, len(labels)+2)[1:-1]
 
         label_props = _expand_text_props(label_props)
         self.labels = [
             ax.text(0.25, y, label, transform=ax.transAxes,
                     horizontalalignment="left", verticalalignment="center",
                     **props)
-            for y, label, props in zip(ys, labels, label_props)]
+            for y, label, props in zip(self._buttons_ys, labels, label_props)]
         text_size = np.array([text.get_fontsize() for text in self.labels]) / 2
 
         frame_props = {
@@ -1204,7 +1204,11 @@ def __init__(self, ax, labels, actives=None, *, useblit=True,
         }
         frame_props.setdefault('facecolor', frame_props.get('color', 'none'))
         frame_props.setdefault('edgecolor', frame_props.pop('color', 'black'))
-        self._frames = ax.scatter([0.15] * len(ys), ys, **frame_props)
+        self._frames = ax.scatter(
+            [0.15] * len(self._buttons_ys),
+            self._buttons_ys,
+            **frame_props,
+        )
         check_props = {
             'linewidth': 1,
             's': text_size**2,
@@ -1217,7 +1221,11 @@ def __init__(self, ax, labels, actives=None, *, useblit=True,
             #       overhead on unsupported backends.
         }
         check_props.setdefault('facecolor', check_props.pop('color', 'black'))
-        self._buttons = ax.scatter([0.15] * len(ys), ys, **check_props)
+        self._buttons = ax.scatter(
+            [0.15] * len(self._buttons_ys),
+            self._buttons_ys,
+            **check_props
+        )
         # The user may have passed custom colours in check_props, so we need to
         # create the checks (above), and modify the visibility after getting
         # whatever the user set.
@@ -1715,7 +1723,7 @@ def __init__(self, ax, labels, active=0, activecolor=None, *,
         ax.set_yticks([])
         ax.set_navigate(False)
 
-        ys = np.linspace(1, 0, len(labels) + 2)[1:-1]
+        self._buttons_ys = np.linspace(1, 0, len(labels) + 2)[1:-1]
 
         self._useblit = useblit
 
@@ -1724,7 +1732,7 @@ def __init__(self, ax, labels, active=0, activecolor=None, *,
             ax.text(0.25, y, label, transform=ax.transAxes,
                     horizontalalignment="left", verticalalignment="center",
                     **props)
-            for y, label, props in zip(ys, labels, label_props)]
+            for y, label, props in zip(self._buttons_ys, labels, label_props)]
         text_size = np.array([text.get_fontsize() for text in self.labels]) / 2
 
         radio_props = {
@@ -1741,7 +1749,11 @@ def __init__(self, ax, labels, active=0, activecolor=None, *,
         radio_props.setdefault('edgecolor', radio_props.get('color', 'black'))
         radio_props.setdefault('facecolor',
                                radio_props.pop('color', activecolor))
-        self._buttons = ax.scatter([.15] * len(ys), ys, **radio_props)
+        self._buttons = ax.scatter(
+            [.15] * len(self._buttons_ys),
+            self._buttons_ys,
+            **radio_props,
+        )
         # The user may have passed custom colours in radio_props, so we need to
         # create the radios, and modify the visibility after getting whatever
         # the user set.
