Skip to content

Commit

Permalink
RightButton is not the only PopupMenu Gesture
Browse files Browse the repository at this point in the history
RMB is not the only PopupMenu getsture on mac and we use
isPopupMenu consistently elsewhere. Here use the || to be
conservative

Closes surge-synthesizer#4672
  • Loading branch information
baconpaul committed Jun 29, 2021
1 parent a042a77 commit 8a95d42
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/gui/SurgeGUIEditorValueCallbacks.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -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)
{
Expand Down Expand Up @@ -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;
Expand Down Expand Up @@ -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{};

Expand Down

0 comments on commit 8a95d42

Please sign in to comment.