[kwin/tabbox] Remove the modalActionSwitch

The modalActionSwitch was used to disable all of KWin's actions during
Alt+Tab. This is not needed as Alt+Tab uses a keyboard grab and thus
no action will be triggered anyway.

Furthermore the functionality had been broken for years. The effects
use an own KActionCollection so their actions aren't considered and
neither the scripts.
icc-effect-5.14.5
Martin Gräßlin 2013-12-10 09:31:41 +01:00
parent c14f798adf
commit d9c29805b1
2 changed files with 0 additions and 14 deletions

View File

@ -1086,16 +1086,6 @@ void TabBox::slotWalkBackThroughDesktopList()
}
}
void TabBox::modalActionsSwitch(bool enabled)
{
QList<KActionCollection*> collections;
collections.append(Workspace::self()->actionCollection());
collections.append(Workspace::self()->clientKeys());
foreach (KActionCollection * collection, collections)
foreach (QAction * action, collection->actions())
action->setEnabled(enabled);
}
bool TabBox::toggle(ElectricBorder eb)
{
if (!options->focusPolicyIsReasonable())
@ -1172,7 +1162,6 @@ bool TabBox::startKDEWalkThroughWindows(TabBoxMode mode)
m_tabGrab = true;
m_noModifierGrab = false;
tabBox->resetEmbedded();
modalActionsSwitch(false);
setMode(mode);
reset();
return true;
@ -1184,7 +1173,6 @@ bool TabBox::startWalkThroughDesktops(TabBoxMode mode)
return false;
m_desktopGrab = true;
m_noModifierGrab = false;
modalActionsSwitch(false);
setMode(mode);
reset();
return true;
@ -1456,7 +1444,6 @@ void TabBox::close(bool abort)
removeTabBoxGrab();
}
hide(abort);
modalActionsSwitch(true);
m_tabGrab = false;
m_desktopGrab = false;
m_noModifierGrab = false;

View File

@ -252,7 +252,6 @@ private:
void oneStepThroughDesktopList(bool forward);
bool establishTabBoxGrab();
void removeTabBoxGrab();
void modalActionsSwitch(bool enabled);
template <typename Slot>
void key(KActionCollection *keys, const char *actionName, Slot slot, const QKeySequence &shortcut = QKeySequence());