Same get_tightbbox cache logic for axes3d

scottshambaugh · scottshambaugh · commit 94799006ed1f · 2026-01-22T15:25:49.000-07:00
diff --git a/lib/mpl_toolkits/mplot3d/axis3d.py b/lib/mpl_toolkits/mplot3d/axis3d.py
@@ -677,6 +677,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 +714,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 +725,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(
