From f77f5b234256af9b22b1852364dcedd05d1d9d6f Mon Sep 17 00:00:00 2001 From: Vlad Zahorodnii Date: Fri, 16 Feb 2024 11:14:22 +0200 Subject: [PATCH] plugins/overview: Make screen edge toggle overview rather than cycle between modes The user may have no usecase for the grid view mode, i.e. they may want to activate overview, do their thing, and then return back to normal desktop. However, with the current behavior, there's one extra step (switch to grid mode) in order to go back to desktop. In hindsight, we should have added different screen edge actions for overview and grid modes. This can be done in 6.1. BUG: 481335 --- 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 9c1a0585d6..45b9f4b265 100644 --- a/src/plugins/overview/overvieweffect.cpp +++ b/src/plugins/overview/overvieweffect.cpp @@ -269,7 +269,7 @@ int OverviewEffect::requestedEffectChainPosition() const bool OverviewEffect::borderActivated(ElectricBorder border) { if (m_borderActivate.contains(border)) { - cycle(); + m_overviewState->toggle(); return true; } return false;