Merge pull request #9039 from fariza/doc-keymap

dstansby · web-flow · commit 5fd199ca691a · 2017-08-16T14:05:42.000+01:00
docstring for key_press_handler_id
diff --git a/lib/matplotlib/backend_bases.py b/lib/matplotlib/backend_bases.py
@@ -2685,12 +2685,7 @@ def __init__(self, canvas, num):
         canvas.manager = self  # store a pointer to parent
         self.num = num
 
-        if rcParams['toolbar'] != 'toolmanager':
-            self.key_press_handler_id = self.canvas.mpl_connect(
-                                                'key_press_event',
-                                                self.key_press)
-        else:
-            self.key_press_handler_id = None
+        self.key_press_handler_id = None
         """
         The returned id from connecting the default key handler via
         :meth:`FigureCanvasBase.mpl_connect`.
@@ -2701,6 +2696,10 @@ def __init__(self, canvas, num):
             canvas.mpl_disconnect(manager.key_press_handler_id)
 
         """
+        if rcParams['toolbar'] != 'toolmanager':
+            self.key_press_handler_id = self.canvas.mpl_connect(
+                'key_press_event',
+                self.key_press)
 
     def show(self):
         """
