From 5b9458f4c286942d1c9cecc82a2c8499b78ea415 Mon Sep 17 00:00:00 2001
From: Jeff Young <jeff@rokeby.ie>
Date: Mon, 19 Nov 2018 11:25:23 +0000
Subject: [PATCH] Don't delegate context menu to Selection Tool.

Our superclass can handle it more correctly (including the Cancel
item at the top).
---
 pcbnew/tools/footprint_editor_tools.cpp | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/pcbnew/tools/footprint_editor_tools.cpp b/pcbnew/tools/footprint_editor_tools.cpp
index 6476ba2c11..f849bfba95 100644
--- a/pcbnew/tools/footprint_editor_tools.cpp
+++ b/pcbnew/tools/footprint_editor_tools.cpp
@@ -252,10 +252,9 @@ int MODULE_EDITOR_TOOLS::EnumeratePads( const TOOL_EVENT& aEvent )
             break;
         }
 
-        else
+        else if( evt->IsClick( BUT_RIGHT ) )
         {
-            // Delegate BUT_RIGHT, etc. to SELECTION_TOOL
-            m_toolMgr->PassEvent();
+            m_menu.ShowContextMenu();
         }
 
         // Prepare the next loop by updating the old cursor mouse position