From 848ec5a9b9bfc09379a1558c12a0b880db84a4a4 Mon Sep 17 00:00:00 2001 From: Leland Lucius Date: Mon, 4 May 2015 12:15:54 -0500 Subject: [PATCH] Disable Apply button while the effect is running --- src/effects/Effect.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/effects/Effect.cpp b/src/effects/Effect.cpp index f2ca82410..a559e1c46 100644 --- a/src/effects/Effect.cpp +++ b/src/effects/Effect.cpp @@ -3047,9 +3047,9 @@ void EffectUIHost::OnApply(wxCommandEvent & evt) return; } - // Prevent recursive apply...can happen because ProgressDialog yields, which allows - // events to flow. -// mApplyBtn->Disable(); + // Progress dialog no longer yields, so this "shouldn't" be necessary (yet to be proven + // for sure), but it is a nice visual cue that something is going on. + mApplyBtn->Disable(); mEffect->Apply();