Commit6476ff3

committed

Make NavigationToolbar.configure_subplots return value consistent

When the subplot tool already exists, then it is shown, but neverreturned. This seems to be an accident.

1 parent6083ecd commit6476ff3Copy full SHA for 6476ff3

File tree

-2

lines changed

-2

lines changed

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -3244,7 +3244,7 @@ def _update_view(self):`
`3244`	`3244`	`defconfigure_subplots(self,*args):`
`3245`	`3245`	`ifhasattr(self,"subplot_tool"):`
`3246`	`3246`	`self.subplot_tool.figure.canvas.manager.show()`
`3247`		`-return`
	`3247`	`+returnself.subplot_tool`
`3248`	`3248`	`# This import needs to happen here due to circular imports.`
`3249`	`3249`	`frommatplotlib.figureimportFigure`
`3250`	`3250`	`withmpl.rc_context({"toolbar":"none"}):# No navbar for the toolfig.`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -475,7 +475,7 @@ class NavigationToolbar2:`
`475`	`475`	`defrelease_zoom(self,event:Event)->None: ...`
`476`	`476`	`defpush_current(self)->None: ...`
`477`	`477`	`subplot_tool:widgets.SubplotTool`
`478`		`-defconfigure_subplots(self,*args): ...`
	`478`	`+defconfigure_subplots(self,*args:Any)->widgets.SubplotTool: ...`
`479`	`479`	`defsave_figure(self,*args)->str\|None\|object: ...`
`480`	`480`	`defupdate(self)->None: ...`
`481`	`481`	`defset_history_buttons(self)->None: ...`

Comments

(0)