Bugfix - don't set sticky shift if a matrix pad was pressed (Synthstr…

…omAudible#2095)

* don't set sticky shift if a matrix pad was pressed

* relocate comment for clarity

src/deluge/deluge.cpp

@@ -294,10 +294,13 @@ bool readButtonsAndPads() {
  ActionResult result;
  if (Pad::isPad(util::to_underlying(value))) {
  auto p = Pad(util::to_underlying(value));
- result = matrixDriver.padAction(p.x, p.y, thisPadPressIsOn);
  /* while this function takes an int32_t for velocity, 255 indicates to the downstream audition pad
  * function that it should use the default velocity for the instrument
  */
+ result = matrixDriver.padAction(p.x, p.y, thisPadPressIsOn);
+ if (thisPadPressIsOn) {
+ Buttons::ignoreCurrentShiftForSticky();
+ }
  }
  else {
  auto b = deluge::hid::Button(value);

src/deluge/hid/buttons.cpp

@@ -283,4 +283,7 @@ void noPressesHappening(bool inCardRoutine) {
  }
  }
 }
+void ignoreCurrentShiftForSticky() {
+ considerShiftReleaseForSticky = false;
+}
 } // namespace Buttons

src/deluge/hid/buttons.h

@@ -27,7 +27,7 @@ ActionResult buttonAction(deluge::hid::Button b, bool on, bool inCardRoutine);
 bool isButtonPressed(deluge::hid::Button b);
 bool isShiftButtonPressed();
 void noPressesHappening(bool inCardRoutine);
-
+void ignoreCurrentShiftForSticky();
 /**
  * Notify the button management code that the shift button should no longer be considered sticky. We need an explicit
  * notification so we can clear the buttons.cpp:shiftIsHeld to avoid shift getting permanantly stuck.