Merge pull request #30349 from anntzer/no-axes-set_navigate_mode

QuLogic · web-flow · commit 011d12f1d6f6 · 2025-07-25T23:27:55.000-04:00
Axes can't set navigate_mode.
diff --git a/doc/api/next_api_changes/deprecations/30349-AL.rst b/doc/api/next_api_changes/deprecations/30349-AL.rst
@@ -0,0 +1,3 @@
+``Axes.set_navigate_mode`` is deprecated
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+... with no replacement.
diff --git a/lib/matplotlib/axes/_base.py b/lib/matplotlib/axes/_base.py
@@ -724,7 +724,6 @@ def __init__(self, fig,
         self.fmt_ydata = None
 
         self.set_navigate(True)
-        self.set_navigate_mode(None)
 
         if xscale:
             self.set_xscale(xscale)
@@ -4187,17 +4186,22 @@ def get_navigate_mode(self):
         """
         Get the navigation toolbar button status: 'PAN', 'ZOOM', or None.
         """
-        return self._navigate_mode
+        toolbar = self.figure.canvas.toolbar
+        if toolbar:
+            return None if toolbar.mode.name == "NONE" else toolbar.mode.name
+        manager = self.figure.canvas.manager
+        if manager and manager.toolmanager:
+            mode = manager.toolmanager.active_toggle.get("default")
+            return None if mode is None else mode.upper()
 
+    @_api.deprecated("3.11")
     def set_navigate_mode(self, b):
         """
         Set the navigation toolbar button status.
 
         .. warning::
             This is not a user-API function.
-
         """
-        self._navigate_mode = b
 
     def _get_view(self):
         """
diff --git a/lib/matplotlib/backend_bases.py b/lib/matplotlib/backend_bases.py
@@ -2763,10 +2763,6 @@ class _Mode(str, Enum):
     def __str__(self):
         return self.value
 
-    @property
-    def _navigate_mode(self):
-        return self.name if self is not _Mode.NONE else None
-
 
 class NavigationToolbar2:
     """
@@ -3037,8 +3033,6 @@ def pan(self, *args):
         else:
             self.mode = _Mode.PAN
             self.canvas.widgetlock(self)
-        for a in self.canvas.figure.get_axes():
-            a.set_navigate_mode(self.mode._navigate_mode)
 
     _PanInfo = namedtuple("_PanInfo", "button axes cid")
 
@@ -3099,8 +3093,6 @@ def zoom(self, *args):
         else:
             self.mode = _Mode.ZOOM
             self.canvas.widgetlock(self)
-        for a in self.canvas.figure.get_axes():
-            a.set_navigate_mode(self.mode._navigate_mode)
 
     _ZoomInfo = namedtuple("_ZoomInfo", "button start_xy axes cid cbar")
 
diff --git a/lib/matplotlib/backend_tools.py b/lib/matplotlib/backend_tools.py
@@ -668,9 +668,6 @@ def disable(self, event=None):
     def trigger(self, sender, event, data=None):
         self.toolmanager.get_tool(_views_positions).add_figure(self.figure)
         super().trigger(sender, event, data)
-        new_navigate_mode = self.name.upper() if self.toggled else None
-        for ax in self.figure.axes:
-            ax.set_navigate_mode(new_navigate_mode)
 
     def scroll_zoom(self, event):
         # https://gist.github.com/tacaswell/3144287

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	+``Axes.set_navigate_mode`` is deprecated
	`2`	`+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~`
	`3`	`+... with no replacement.`