Merge pull request matplotlib#21591 from anntzer/tfs

Make ToolFullScreen a Tool, not a ToolToggle.

Author: timhoffm

doc/api/next_api_changes/removals/21591-AL.rst

@@ -0,0 +1,6 @@
+``backend_tools.ToolFullScreen`` now inherits from ``ToolBase``, not from ``ToolToggleBase``
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+`.ToolFullScreen` can only switch between the non-fullscreen
+and fullscreen states, but not unconditionally put the window in a given state;
+hence the ``enable`` and ``disable`` methods were misleadingly named.  Thus,
+the `.ToolToggleBase`-related API (``enable``, ``disable``, etc.) was removed.

lib/matplotlib/backend_tools.py

@@ -428,16 +428,13 @@ def trigger(self, sender, event, data=None):
             mpl.backend_bases.key_press_handler(event, self.figure.canvas)
 
 
-class ToolFullScreen(ToolToggleBase):
+class ToolFullScreen(ToolBase):
     """Tool to toggle full screen."""
 
     description = 'Toggle fullscreen mode'
     default_keymap = mpl.rcParams['keymap.fullscreen']
 
-    def enable(self, event):
-        self.figure.canvas.manager.full_screen_toggle()
-
-    def disable(self, event):
+    def trigger(self, sender, event, data=None):
         self.figure.canvas.manager.full_screen_toggle()