PR comments and updated whats new

ewpardijs · ewpardijs · commit 39cfd5638830 · 2023-04-21T18:06:48.000+02:00
diff --git a/doc/users/next_whats_new/updated_scatter.rst b/doc/users/next_whats_new/updated_scatter.rst
@@ -1,7 +1,7 @@
 Enable configuration of empty markers in `~matplotlib.axes.Axes.scatter`
 ------------------------------------------------------------------------
 
-`~matplotlib.axes.Axes.scatter` can now be configured to plot empty markers without additional code. Setting ``facecolors`` to *'none'* and defining ``c`` now draws only the edge colors for fillable markers.
+`~matplotlib.axes.Axes.scatter` can now be configured to plot empty markers by setting ``facecolors`` to *'none'* and defining ``c``. In this case, ``c`` will be now used as ``edgecolor``.
 
 .. plot::
     :include-source: true
diff --git a/lib/matplotlib/axes/_axes.py b/lib/matplotlib/axes/_axes.py
@@ -4387,11 +4387,9 @@ def _parse_scatter_color_args(c, edgecolors, kwargs, xsize,
             if facecolors is None:
                 facecolors = kwcolor
 
-        edge_from_c = False
-        if edgecolors is None and c is not None:
-            edge_from_c = True
+        edge_from_c = edgecolors is None and c is not None
 
-        facecolors_none = isinstance(facecolors, str) and facecolors == 'none'
+        facecolors_none = cbook._str_lower_equal(facecolors, 'none')
         if (edgecolors is None and not mpl.rcParams['_internal.classic_mode']
                 and (not edge_from_c or not facecolors_none)):
 
