Same get_tightbbox cache logic for axes3d

scottshambaugh · scottshambaugh · commit 7370321e5195 · 2026-02-04T14:30:48.000-07:00
diff --git a/lib/mpl_toolkits/mplot3d/axis3d.py b/lib/mpl_toolkits/mplot3d/axis3d.py
@@ -628,9 +628,8 @@ def draw(self, renderer):
             self._draw_labels(renderer, edgep1, edgep2, labeldeltas, centers, dx, dy)
 
         renderer.close_group('axis3d')
-        self.stale = False
-
         self._clear_ticks_cache()
+        self.stale = False
 
     @artist.allow_rasterization
     def draw_grid(self, renderer):
@@ -677,6 +676,12 @@ def get_tightbbox(self, renderer=None, *, for_layout_only=False):
         # docstring inherited
         if not self.get_visible():
             return
+
+        # We need to reset the ticks cache here - get_tightbbox() is called
+        # during layout calculations (e.g., constrained_layout) outside of
+        # draw(), and must always recalculate to reflect current state.
+        self._clear_ticks_cache()
+
         # We have to directly access the internal data structures
         # (and hope they are up to date) because at draw time we
         # shift the ticks and their labels around in (x, y) space
@@ -708,10 +713,7 @@ def get_tightbbox(self, renderer=None, *, for_layout_only=False):
 
         ticks = ticks_to_draw
 
-        # Don't use cached values here - get_tightbbox() is called during
-        # layout calculations (e.g., constrained_layout) outside of draw(),
-        # and must always recalculate to reflect current state.
-        bb_1, bb_2 = self._get_ticklabel_bboxes(ticks, renderer, _use_cache=False)
+        bb_1, bb_2 = self._get_ticklabel_bboxes(ticks, renderer, _use_cache=True)
         other = []
 
         if self.offsetText.get_visible() and self.offsetText.get_text():
@@ -722,6 +724,7 @@ def get_tightbbox(self, renderer=None, *, for_layout_only=False):
                 self.label.get_text()):
             other.append(self.label.get_window_extent(renderer))
 
+        self._clear_ticks_cache()
         return mtransforms.Bbox.union([*bb_1, *bb_2, *other])
 
     d_interval = _api.deprecated(
