From ad4ce627aef8f78969b17f5009704b3260d90d35 Mon Sep 17 00:00:00 2001 From: Nate Graham Date: Thu, 15 Jun 2023 17:30:32 -0600 Subject: [PATCH] plugins/overview: also do things when changing to active We were binding to the statusChanged signal, but then only doing anything if the status had changed to "Activating". This did not work for when the effect was activated without a gesture, because then its status would change straight to "Activated", bypassing the in-between state of "Activating." This fixes clearing the search text on activation without a gesture. --- src/plugins/overview/overvieweffect.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/plugins/overview/overvieweffect.cpp b/src/plugins/overview/overvieweffect.cpp index 5dabab3ce3..b3072133db 100644 --- a/src/plugins/overview/overvieweffect.cpp +++ b/src/plugins/overview/overvieweffect.cpp @@ -32,7 +32,7 @@ OverviewEffect::OverviewEffect() connect(m_state, &EffectTogglableState::inProgressChanged, this, &OverviewEffect::gestureInProgressChanged); connect(m_state, &EffectTogglableState::partialActivationFactorChanged, this, &OverviewEffect::partialActivationFactorChanged); connect(m_state, &EffectTogglableState::statusChanged, this, [this](EffectTogglableState::Status status) { - if (status == EffectTogglableState::Status::Activating) { + if (status == EffectTogglableState::Status::Activating || status == EffectTogglableState::Status::Active) { m_searchText = QString(); } setRunning(status != EffectTogglableState::Status::Inactive);