Fixes bug where if alpha is set and edgecolor='none', edges would still be rendered as black.

Westacular · Westacular · commit 461233cfc634 · 2013-04-24T14:32:31.000-04:00
diff --git a/lib/matplotlib/patches.py b/lib/matplotlib/patches.py
@@ -251,6 +251,7 @@ def set_edgecolor(self, color):
         """
         if color is None:
             color = mpl.rcParams['patch.edgecolor']
+        self._original_edgecolor = color
         self._edgecolor = colors.colorConverter.to_rgba(color, self._alpha)
 
     def set_ec(self, color):
@@ -304,8 +305,7 @@ def set_alpha(self, alpha):
         artist.Artist.set_alpha(self, alpha)
         self.set_facecolor(self._original_facecolor)  # using self._fill and
                                                       # self._alpha
-        self._edgecolor = colors.colorConverter.to_rgba(
-                                        self._edgecolor[:3], self._alpha)
+        self.set_edgecolor(self._original_edgecolor)
 
     def set_linewidth(self, w):
         """
@@ -421,9 +421,10 @@ def draw(self, renderer):
         if rgbFace[3] == 0:
             rgbFace = None  # (some?) renderers expect this as no-fill signal
 
-        gc.set_alpha(self._edgecolor[3])
         if self._edgecolor[3] == 0:
             gc.set_alpha(self._facecolor[3])
+        else:
+            gc.set_alpha(self._edgecolor[3])
 
         if self._hatch:
             gc.set_hatch(self._hatch)
