Merge pull request #9793 from tacaswell/fix_none_zorder

efiring · web-flow · commit c66c4230bd4e · 2017-11-15T20:37:14.000-10:00
FIX: treat `zorder=None` as falling back to the default
diff --git a/lib/matplotlib/artist.py b/lib/matplotlib/artist.py
@@ -885,6 +885,8 @@ def set_zorder(self, level):
 
         ACCEPTS: any number
         """
+        if level is None:
+            level = self.__class__.zorder
         self.zorder = level
         self.pchanged()
         self.stale = True
diff --git a/lib/matplotlib/tests/test_artist.py b/lib/matplotlib/tests/test_artist.py
@@ -246,3 +246,13 @@ def test_setp():
     sio = io.StringIO()
     plt.setp(lines1, 'zorder', file=sio)
     assert sio.getvalue() == '  zorder: any number \n'
+
+
+def test_None_zorder():
+    fig, ax = plt.subplots()
+    ln, = ax.plot(range(5), zorder=None)
+    assert ln.get_zorder() == mlines.Line2D.zorder
+    ln.set_zorder(123456)
+    assert ln.get_zorder() == 123456
+    ln.set_zorder(None)
+    assert ln.get_zorder() == mlines.Line2D.zorder
