From 959a4bddedc371ef4d1a3c97c48cf90cf31d7a85 Mon Sep 17 00:00:00 2001 From: Seth Hillbrand Date: Wed, 14 Aug 2019 17:12:00 -0700 Subject: [PATCH] pcbnew: Don't stack same tool handler The Get and place tool used to live in selection tool and could activate the edit tool by stacking calls. Now that it is itself in the edit tool, we cannot stack another call to the edit tool that runs immediately, instead we allow the GetAndPlace to finish before beginning the next tool (Move) --- pcbnew/tools/edit_tool.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pcbnew/tools/edit_tool.cpp b/pcbnew/tools/edit_tool.cpp index 82650c7f50..9d36c41b60 100644 --- a/pcbnew/tools/edit_tool.cpp +++ b/pcbnew/tools/edit_tool.cpp @@ -203,7 +203,7 @@ int EDIT_TOOL::GetAndPlace( const TOOL_EVENT& aEvent ) m_toolMgr->RunAction( PCB_ACTIONS::selectItem, true, (void*) module ); selectionTool->GetSelection().SetReferencePoint( module->GetPosition() ); - m_toolMgr->RunAction( PCB_ACTIONS::move, true ); + m_toolMgr->RunAction( PCB_ACTIONS::move, false ); } return 0;