Jan 23, 2014 · Jan 23, 2014 · Jan 23, 2014 · Jan 24, 2014 · Jan 24, 2014 · Jan 26, 2014
diff --git a/lib/matplotlib/backend_bases.py b/lib/matplotlib/backend_bases.py
        for a in self.canvas.figure.get_axes():
            a.set_navigate_mode(self._toggled)

        # Change the cursor inmediately, don't wait for mouse move
        self._set_cursor(event)

    def get_tools(self):
        """Return the tools controlled by `Navigation`"""

                       'keymap': keys}
        return d

    def _mouse_move(self, event):
    def _set_cursor(self, event):
        """Call the backend specific set_cursor method,
        if the pointer is inaxes
        """
        if not event.inaxes or not self._toggled:
            if self._last_cursor != self._default_cursor:
                self.set_cursor(self._default_cursor)
                    self.set_cursor(cursor)
                    self._last_cursor = cursor

    def _mouse_move(self, event):
        self._set_cursor(event)

        if self.toolbar is None or self.messagelock.locked():
            return
Original file line number	Diff line number	Diff line change
Expand Up		@@ -3434,6 +3434,9 @@ def _handle_toggle(self, name, event=None, from_toolbar=False):
		for a in self.canvas.figure.get_axes():
		a.set_navigate_mode(self._toggled)

		# Change the cursor inmediately, don't wait for mouse move
		self._set_cursor(event)

		def get_tools(self):
		"""Return the tools controlled by `Navigation`"""

Expand All		@@ -3446,7 +3449,10 @@ def get_tools(self):
		'keymap': keys}
		return d

		def _mouse_move(self, event):
		def _set_cursor(self, event):
		"""Call the backend specific set_cursor method,
		if the pointer is inaxes
		"""
		if not event.inaxes or not self._toggled:
		if self._last_cursor != self._default_cursor:
Copy link Member OceanWolfJul 24, 2014 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. Do we need this if statement? Do we get a significant performance hit without it? Copy link MemberAuthor farizaJul 24, 2014 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. In this case, I don't know, it is a copy paste from`NavigationToolbar2`. I would be inclined to leave it there, and check the performance impact, when it is merged.
		self.set_cursor(self._default_cursor)
Expand All		@@ -3458,6 +3464,9 @@ def _mouse_move(self, event):
		self.set_cursor(cursor)
		self._last_cursor = cursor

		def _mouse_move(self, event):
		self._set_cursor(event)

		if self.toolbar is None or self.messagelock.locked():
		return

Expand Down