Apr 9, 2015 · Jan 23, 2014 · Jan 23, 2014 · Jan 24, 2014 · Jan 24, 2014 · Jan 26, 2014
diff --git a/lib/matplotlib/backend_tools.py b/lib/matplotlib/backend_tools.py
        self._name = name
        self._figure = None
        self.navigation = navigation
        self.set_figure(navigation.canvas.figure)
        self.figure =navigation.canvas.figure

    @property
    def figure(self):

        pass

    def set_figure(self, figure):
    @figure.setter
    def figure(self, figure):
        """
        Set the figure
Original file line number	Diff line number	Diff line change
Expand Up		@@ -74,7 +74,7 @@ def __init__(self, navigation, name):
		self._name = name
		self._figure = None
		self.navigation = navigation
		self.set_figure(navigation.canvas.figure)
		self.figure =navigation.canvas.figure

		@property
		def figure(self):
Expand All		@@ -99,7 +99,8 @@ def trigger(self, sender, event, data=None):

		pass
Copy link Member tacaswellApr 1, 2015 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. Is there a use-case for a Tool that does not implement it's own`trigger`? If there is not (and you don't plan to use multiple inheritance) I suggest this raise`NotImplemented` Copy link MemberAuthor farizaApr 1, 2015 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. Yes there is, if you look at`backend_tools.ToolViewsPositions` doesn't use a`trigger` method Copy link Member tacaswellApr 1, 2015 Choose a reason for hiding this comment The reason will be displayed to describe this comment to others.Learn more. ok. sorry for the noise.

		def set_figure(self, figure):
		@figure.setter
		def figure(self, figure):
		"""
		Set the figure

Expand Down