Adding rcParams[‘scatter.edgecolors’] defaulting to ‘face’ (#12992)

bsipocz · timhoffm · commit 11def887fb06 · 2018-12-24T14:47:55.000+01:00
* Adding rcParams[‘scatter.edgecolors’] defaulting to ‘face’

* Adding scatter.edgecolors to matplotlibrc template

* Add back mentioning sequence of color

* Update lib/matplotlib/axes/_axes.py

Co-Authored-By: bsipocz &lt;b.sipocz@gmail.com&gt;
diff --git a/lib/matplotlib/axes/_axes.py b/lib/matplotlib/axes/_axes.py
@@ -4102,7 +4102,7 @@ def _parse_scatter_color_args(c, edgecolors, kwargs, xshape, yshape,
                 facecolors = kwcolor
 
         if edgecolors is None and not rcParams['_internal.classic_mode']:
-            edgecolors = 'face'
+            edgecolors = rcParams['scatter.edgecolors']
 
         c_was_none = c is None
         if c is None:
@@ -4248,12 +4248,16 @@ def scatter(self, x, y, s=None, c=None, marker=None, cmap=None, norm=None,
             is 'face'. You may want to change this as well.
             If *None*, defaults to rcParams ``lines.linewidth``.
 
-        edgecolors : color or sequence of color, optional, default: 'face'
+        edgecolors : {'face', 'none', *None*} or color or sequence of color, \
+optional.
             The edge color of the marker. Possible values:
 
             - 'face': The edge color will always be the same as the face color.
             - 'none': No patch boundary will be drawn.
-            - A matplotib color.
+            - A matplotib color or sequence of color.
+
+            Defaults to ``None``, in which case it takes the value of
+            :rc:`scatter.edgecolors` = 'face'.
 
             For non-filled markers, the *edgecolors* kwarg is ignored and
             forced to 'face' internally.
diff --git a/lib/matplotlib/rcsetup.py b/lib/matplotlib/rcsetup.py
@@ -1230,6 +1230,7 @@ def _validate_linestyle(ls):
 
     # scatter props
     'scatter.marker': ['o', validate_string],
+    'scatter.edgecolors': ['face', validate_string],
 
     # TODO validate that these are valid datetime format strings
     'date.autoformatter.year': ['%Y', validate_string],
diff --git a/matplotlibrc.template b/matplotlibrc.template
@@ -418,7 +418,7 @@
 #legend.scatterpoints : 1        ## number of scatter points
 #legend.markerscale   : 1.0      ## the relative size of legend markers vs. original
 #legend.fontsize      : medium
-#legend.title_fontsize    : None ## None sets to the same as the default axes.  
+#legend.title_fontsize    : None ## None sets to the same as the default axes.
 ## Dimensions as fraction of fontsize:
 #legend.borderpad     : 0.4      ## border whitespace
 #legend.labelspacing  : 0.5      ## the vertical space between the legend entries
@@ -488,6 +488,7 @@
 
 #### SCATTER PLOTS
 #scatter.marker : o               ## The default marker type for scatter plots.
+#scatter.edgecolors : face        ## The default edgecolors for scatter plots.
 
 #### Agg rendering
 #### Warning: experimental, 2008/10/10
