Verified Commit cbb707fe authored by ivan tkachenko's avatar ivan tkachenko
Browse files

[applets/showdesktop] Fix "Minimize All" context action on Wayland

Let's reuse the one from MinimizeAllController that "just works".

This change means that applet will be tracking minimized clients
regardless of its primary role (Minimize / Peek). This needs some
follow up changes to represent such state explicitly. But also it makes
"Minimize All Windows" action menu affect the "active" state of the
 applet when it has "Minimize..." role, which is kind of an improvement.

BUG: 457809
parent cbf9349f
......@@ -177,7 +177,7 @@ MouseArea {
}
function action_minimizeall() {
showdesktop.minimizeAll();
minimizeAllController.toggle();
}
function action_showdesktop() {
......
......@@ -39,14 +39,6 @@ void ShowDesktop::setShowingDesktop(bool showingDesktop)
Q_EMIT showingDesktopChanged(showingDesktop);
}
void ShowDesktop::minimizeAll()
{
const auto &windows = KWindowSystem::windows();
for (WId wid : windows) {
KWindowSystem::minimizeWindow(wid);
}
}
void ShowDesktop::toggleDesktop()
{
setShowingDesktop(!showingDesktop());
......
......@@ -24,7 +24,6 @@ public:
bool showingDesktop() const;
void setShowingDesktop(bool showingDesktop);
Q_INVOKABLE void minimizeAll();
Q_INVOKABLE void toggleDesktop();
Q_SIGNALS:
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment