From b4fb0984d34b5babf59f8f3099e575348c96778a Mon Sep 17 00:00:00 2001 From: "Charles J. Cliffe" Date: Sat, 28 Mar 2015 01:11:33 -0400 Subject: [PATCH] prevent mouseup tuner step on drag stop --- src/visual/TuningCanvas.cpp | 9 ++++++++- src/visual/TuningCanvas.h | 1 + 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/src/visual/TuningCanvas.cpp b/src/visual/TuningCanvas.cpp index a10ca11..e8686e0 100644 --- a/src/visual/TuningCanvas.cpp +++ b/src/visual/TuningCanvas.cpp @@ -30,7 +30,10 @@ TuningCanvas::TuningCanvas(wxWindow *parent, int *attribList) : glContext = new TuningContext(this, &wxGetApp().GetContext(this)); hoverIndex = 0; + downIndex = 0; hoverState = TUNING_HOVER_NONE; + downState = TUNING_HOVER_NONE; + dragging = false; freqDP = -1.0; freqW = (1.0 / 3.0) * 2.0; @@ -192,13 +195,16 @@ void TuningCanvas::OnIdle(wxIdleEvent &event) { while (dragAccum > 1.0) { StepTuner(downState, downIndex-1, true); dragAccum -= 1.0; + dragging = true; } while (dragAccum < -1.0) { StepTuner(downState, downIndex-1, false); dragAccum += 1.0; + dragging = true; } } else { dragAccum = 0; + dragging = false; } } @@ -264,12 +270,13 @@ void TuningCanvas::OnMouseReleased(wxMouseEvent& event) { int hExponent = hoverIndex - 1; - if (hoverState != TUNING_HOVER_NONE) { + if (hoverState != TUNING_HOVER_NONE && !dragging) { StepTuner(hoverState, hExponent, top); } mouseTracker.setVertDragLock(false); + dragging = false; SetCursor(wxCURSOR_ARROW); } diff --git a/src/visual/TuningCanvas.h b/src/visual/TuningCanvas.h index 46037c3..21b22c5 100644 --- a/src/visual/TuningCanvas.h +++ b/src/visual/TuningCanvas.h @@ -44,6 +44,7 @@ private: ActiveState downState; int hoverIndex; int downIndex; + bool dragging; float freqDP; float freqW;