Fix scatter edgecolor for unfilled points

has2k1 · has2k1 · commit 28696b9d4ceb · 2017-12-14T13:26:08.000-06:00
For unfilled markers, the edgecolor -- if specified -- has precedence over the facecolor. closes has2k1/plotnine#100
diff --git a/doc/api/api_changes/2017-12-14-HK.rst b/doc/api/api_changes/2017-12-14-HK.rst
@@ -0,0 +1,6 @@
+Scatter plot unfilled marker edgecolor
+--------------------------------------
+
+For :meth:`~matplotlib.axes.Axes.scatter`, when both the `facecolor`
+and `edgecolor` are specified for unfilled marker types, the `edgecolor`
+is used. Previously, the points took on the `facecolor`.
diff --git a/lib/matplotlib/axes/_axes.py b/lib/matplotlib/axes/_axes.py
@@ -4024,7 +4024,8 @@ def scatter(self, x, y, s=None, c=None, marker=None, cmap=None, norm=None,
         path = marker_obj.get_path().transformed(
             marker_obj.get_transform())
         if not marker_obj.is_filled():
-            edgecolors = 'face'
+            if edgecolors is None:
+                edgecolors = 'face'
             linewidths = rcParams['lines.linewidth']
 
         offsets = np.column_stack([x, y])
