Tests, fix, and whats new from code review

scottshambaugh · scottshambaugh · commit 6eb7b4394d9f · 2023-07-30T23:55:58.000-06:00
diff --git a/doc/users/next_whats_new/3d_axis_positions.rst b/doc/users/next_whats_new/3d_axis_positions.rst
@@ -9,9 +9,11 @@ You can now specify the positions of ticks and axis labels for 3D plots.
    import matplotlib.pyplot as plt
 
    positions = ['default', 'upper', 'lower', 'both', 'none']
-   fig, axs = plt.subplots(1, 5, subplot_kw={'projection': '3d'})
+   fig, axs = plt.subplots(1, len(positions), figsize=(20, 4),
+                           subplot_kw={'projection': '3d'})
    for ax, pos in zip(axs, positions):
        for axis in ax.xaxis, ax.yaxis, ax.zaxis:
-           axis._label_position = pos
-           axis._tick_position = pos
-       ax.set(xlabel='x', ylabel='y', zlabel='z', title=pos)
+           axis.set_label_position(pos)
+           axis.set_ticks_position(pos)
+       title = f'position="{pos}"'
+       ax.set(xlabel='x', ylabel='y', zlabel='z', title=title)
diff --git a/lib/mpl_toolkits/mplot3d/axis3d.py b/lib/mpl_toolkits/mplot3d/axis3d.py
@@ -345,7 +345,7 @@ def _get_all_axis_line_edge_points(self, minmax, maxmin, axis_position=None):
                                                                  position='lower')
             edgep1_u, edgep2_u = self._get_axis_line_edge_points(minmax, maxmin,
                                                                  position='upper')
-            if position in ('lower', 'both'):
+            if axis_position in ('lower', 'both'):
                 edgep1s.append(edgep1_l)
                 edgep2s.append(edgep2_l)
                 position.append('lower')
diff --git a/lib/mpl_toolkits/mplot3d/tests/test_axes3d.py b/lib/mpl_toolkits/mplot3d/tests/test_axes3d.py
@@ -57,6 +57,18 @@ def test_invisible_ticks_axis():
         axis.line.set_visible(False)
 
 
+@mpl3d_image_comparison(['axis_positions.png'], style='mpl20')
+def test_axis_positions():
+    positions = ['upper', 'lower', 'both', 'none']
+    fig, axs = plt.subplots(2, 2, subplot_kw={'projection': '3d'})
+    for ax, pos in zip(axs.flatten(), positions):
+        for axis in ax.xaxis, ax.yaxis, ax.zaxis:
+            axis.set_label_position(pos)
+            axis.set_ticks_position(pos)
+        title = f'{pos}'
+        ax.set(xlabel='x', ylabel='y', zlabel='z', title=title)
+
+
 @mpl3d_image_comparison(['aspects.png'], remove_text=False, style='mpl20')
 def test_aspects():
     aspects = ('auto', 'equal', 'equalxy', 'equalyz', 'equalxz', 'equal')
