From 8a95d420264c7c5e453acf1080ea6078d0545ff3 Mon Sep 17 00:00:00 2001
From: Paul Walker <paul@pwjw.com>
Date: Tue, 29 Jun 2021 19:58:52 -0400
Subject: [PATCH] RightButton is not the only PopupMenu Gesture

RMB is not the only PopupMenu getsture on mac and we use
isPopupMenu consistently elsewhere. Here use the || to be
conservative

Closes #4672
---
 src/gui/SurgeGUIEditorValueCallbacks.cpp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/gui/SurgeGUIEditorValueCallbacks.cpp b/src/gui/SurgeGUIEditorValueCallbacks.cpp
index f7ea2b2f6ce..93f48f84a1b 100644
--- a/src/gui/SurgeGUIEditorValueCallbacks.cpp
+++ b/src/gui/SurgeGUIEditorValueCallbacks.cpp
@@ -144,7 +144,7 @@ int32_t SurgeGUIEditor::controlModifierClicked(Surge::GUI::IComponentTagValue *c
         return 0;
     }
 
-    if (button.isRightButtonDown())
+    if (button.isRightButtonDown() || button.isPopupMenu())
     {
         if (tag == tag_settingsmenu)
         {
@@ -255,7 +255,7 @@ int32_t SurgeGUIEditor::controlModifierClicked(Surge::GUI::IComponentTagValue *c
     {
         modsources modsource = (modsources)(tag - tag_mod_source0);
 
-        if (button.isRightButtonDown())
+        if (button.isRightButtonDown() || button.isPopupMenu())
         {
             auto *cms = dynamic_cast<Surge::Widgets::ModulationSourceButton *>(control);
             juce::PopupMenu contextMenu;
@@ -813,7 +813,7 @@ int32_t SurgeGUIEditor::controlModifierClicked(Surge::GUI::IComponentTagValue *c
         }
 
         // all the RMB context menus
-        if (button.isRightButtonDown())
+        if (button.isRightButtonDown() || button.isPopupMenu())
         {
             juce::Point<int> menuRect{};