RadioButtons: Fix label_props argument and set_label_props method

doronbehar · doronbehar · commit f772d0a82de4 · 2025-08-10T02:39:08.000+03:00
Fixes #30393
diff --git a/doc/users/next_whats_new/RadioButtons-label_props.rst b/doc/users/next_whats_new/RadioButtons-label_props.rst
@@ -0,0 +1,11 @@
+Improve Label Properties Settings in ``RadioButtons``
+-----------------------------------------------------
+
+Since styling was introduced to `RadioButtons` in version 3.7.0, the
+``label_props`` argument of `~.RadioButtons.__init__` and the
+`~.RadioButtons.set_label_props` class method, both required a single element
+list per every property in the given dictionary. Now these dictionary values
+can be a scalar, which will be used to all radio button labels, or a list. In
+case of a list, every value in the list will be used per label, and the list's
+length should fit the amount of labels. A list with an improper amount of items
+raises an error message.
diff --git a/lib/matplotlib/tests/test_widgets.py b/lib/matplotlib/tests/test_widgets.py
@@ -1177,7 +1177,7 @@ def test_check_buttons(fig_test, fig_ref):
 
 @check_figures_equal()
 def test_check_button_props(fig_test, fig_ref):
-    label_props = {'color': ['red'], 'fontsize': [24]}
+    label_props = {'color': 'red', 'fontsize': 24}
     frame_props = {'facecolor': 'green', 'edgecolor': 'blue', 'linewidth': 2}
     check_props = {'facecolor': 'red', 'linewidth': 2}
 
diff --git a/lib/matplotlib/widgets.py b/lib/matplotlib/widgets.py
@@ -1017,7 +1017,11 @@ def __init__(self, ax, labels, actives=None, *, useblit=True,
             .. versionadded:: 3.7
 
         label_props : dict, optional
-            Dictionary of `.Text` properties to be used for the labels.
+            Dictionary of `.Text` properties to be used for the labels. Per dict
+            key, if the value is a list or a tuple, its length should be the
+            amount of labels, and then a value per label is set. If the dict's
+            value is not a list or a tuple, the same property value is applied
+            to all labels
 
             .. versionadded:: 3.7
         frame_props : dict, optional
@@ -1050,12 +1054,12 @@ def __init__(self, ax, labels, actives=None, *, useblit=True,
 
         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 in zip(ys, labels)]
+        self.set_label_props(label_props)
         text_size = np.array([text.get_fontsize() for text in self.labels]) / 2
 
         frame_props = {
@@ -1117,12 +1121,26 @@ def set_label_props(self, props):
         Parameters
         ----------
         props : dict
-            Dictionary of `.Text` properties to be used for the labels.
+            Dictionary of `.Text` properties, just like `label_props` argument
+            of __init__ function.
         """
         _api.check_isinstance(dict, props=props)
         props = _expand_text_props(props)
-        for text, prop in zip(self.labels, props):
-            text.update(prop)
+        for propk,propv in props.items():
+            if isinstance(propv, (list, tuple)):
+                if len(propv) != len(self.labels):
+                    raise ValueError(
+                        f"When setting labels' property {propk}, we "
+                        f"encountered a {type(propv)} as a label dict value, "
+                        "with a length different from the amount of labels. "
+                        "Please use a single value, or a list of values per "
+                        "label."
+                    )
+                for text,propv_single in zip(self.labels, propv):
+                    text.update({propk: propv_single})
+            else:
+                for text in self.labels:
+                    text.update({propk: propv})
 
     def set_frame_props(self, props):
         """
