Merge pull request matplotlib#2415 from kshramt/kshramt/fix_fill_opacity

Bug: alpha parameter was ignored when fill color is #000000

Author: mdboom

lib/matplotlib/backends/backend_svg.py

@@ -406,8 +406,9 @@ def _get_style_dict(self, gc, rgbFace):
         else:
             if rgbFace is None:
                 attrib['fill'] = 'none'
-            elif tuple(rgbFace[:3]) != (0, 0, 0):
-                attrib['fill'] = rgb2hex(rgbFace)
+            else:
+                if tuple(rgbFace[:3]) != (0, 0, 0):
+                    attrib['fill'] = rgb2hex(rgbFace)
                 if len(rgbFace) == 4 and rgbFace[3] != 1.0 and not forced_alpha:
                     attrib['fill-opacity'] = str(rgbFace[3])
 

lib/matplotlib/tests/baseline_images/test_backend_svg/fill_black_with_alpha.svg

@@ -36,6 +36,14 @@ def test_visibility():
     parser.Parse(buf)  # this will raise ExpatError if the svg is invalid
 
 
+@image_comparison(baseline_images=['fill_black_with_alpha'], remove_text=True,
+                  extensions=['svg'])
+def test_fill_black_with_alpha():
+    fig = plt.figure()
+    ax = fig.add_subplot(1, 1, 1)
+    ax.scatter(x=[0, 0.1, 1], y=[0, 0, 0], c='k', alpha=0.1, s=10000)
+
+
 @image_comparison(baseline_images=['noscale'], remove_text=True)
 def test_noscale():
     X, Y = np.meshgrid(np.arange(-5, 5, 1), np.arange(-5, 5, 1))