refactor: expose raise_window to user by reusing existing internal code

alberti42 · alberti42 · commit 591de9eb8fb7 · 2026-06-07T15:41:44.000+02:00
diff --git a/lib/matplotlib/backend_bases.py b/lib/matplotlib/backend_bases.py
@@ -2882,6 +2882,9 @@ def destroy(self):
     def full_screen_toggle(self):
         pass
 
+    def raise_window(self):
+        """For GUI backends, raise the figure window to the top."""
+
     def resize(self, w, h):
         """For GUI backends, resize the window (in physical pixels)."""
 
diff --git a/lib/matplotlib/backends/_backend_gtk.py b/lib/matplotlib/backends/_backend_gtk.py
@@ -221,15 +221,19 @@ def show(self):
         self.window.show()
         self.canvas.draw()
         if mpl.rcParams["figure.raise_window"]:
-            meth_name = {3: "get_window", 4: "get_surface"}[self._gtk_ver]
-            if getattr(self.window, meth_name)():
-                self.window.present()
-            else:
-                # If this is called by a callback early during init,
-                # self.window (a GtkWindow) may not have an associated
-                # low-level GdkWindow (on GTK3) or GdkSurface (on GTK4) yet,
-                # and present() would crash.
-                _api.warn_external("Cannot raise window yet to be setup")
+            self.raise_window()
+
+    def raise_window(self):
+        # docstring inherited
+        meth_name = {3: "get_window", 4: "get_surface"}[self._gtk_ver]
+        if getattr(self.window, meth_name)():
+            self.window.present()
+        else:
+            # If this is called by a callback early during init,
+            # self.window (a GtkWindow) may not have an associated
+            # low-level GdkWindow (on GTK3) or GdkSurface (on GTK4) yet,
+            # and present() would crash.
+            _api.warn_external("Cannot raise window yet to be setup")
 
     def full_screen_toggle(self):
         is_fullscreen = {
diff --git a/lib/matplotlib/backends/_backend_tk.py b/lib/matplotlib/backends/_backend_tk.py
@@ -615,10 +615,14 @@ def destroy(*args):
             else:
                 self.canvas.draw_idle()
             if mpl.rcParams['figure.raise_window']:
-                self.canvas.manager.window.attributes('-topmost', 1)
-                self.canvas.manager.window.attributes('-topmost', 0)
+                self.raise_window()
             self._shown = True
 
+    def raise_window(self):
+        # docstring inherited
+        self.window.attributes('-topmost', 1)
+        self.window.attributes('-topmost', 0)
+
     def destroy(self, *args):
         if self.canvas._idle_draw_id:
             self.canvas._tkcanvas.after_cancel(self.canvas._idle_draw_id)
diff --git a/lib/matplotlib/backends/backend_macosx.py b/lib/matplotlib/backends/backend_macosx.py
@@ -186,7 +186,11 @@ def show(self):
             self._show()
             self._shown = True
         if mpl.rcParams["figure.raise_window"]:
-            self._raise()
+            self.raise_window()
+
+    def raise_window(self):
+        # docstring inherited
+        self._raise()
 
 
 @_Backend.export
diff --git a/lib/matplotlib/backends/backend_qt.py b/lib/matplotlib/backends/backend_qt.py
@@ -664,8 +664,12 @@ def show(self):
         self.window._destroying = False
         self.window.show()
         if mpl.rcParams['figure.raise_window']:
-            self.window.activateWindow()
-            self.window.raise_()
+            self.raise_window()
+
+    def raise_window(self):
+        # docstring inherited
+        self.window.activateWindow()
+        self.window.raise_()
 
     def destroy(self, *args):
         # check for qApp first, as PySide deletes it in its atexit handler
diff --git a/lib/matplotlib/backends/backend_wx.py b/lib/matplotlib/backends/backend_wx.py
@@ -1002,7 +1002,11 @@ def show(self):
         self.frame.Show()
         self.canvas.draw()
         if mpl.rcParams['figure.raise_window']:
-            self.frame.Raise()
+            self.raise_window()
+
+    def raise_window(self):
+        # docstring inherited
+        self.frame.Raise()
 
     def destroy(self, *args):
         # docstring inherited
