Diffusion KWin fec0ab818d24

Emphasize minimization in MinimizeAll script

Authored by smerkov on Sep 18 2019, 3:09 PM.

Description

Emphasize minimization in MinimizeAll script

Summary:
MinimizeAll should work as follows:

As long as there's something to minimize, do that.
Otherwise unminimize only the windows we minimized
on last invocation.

Quote above is from Comment 11 by Thomas Lübking of BUG: 356161.

But current code does not fulfill these duties. Try next case: run minimizeall on some windows, then click on one of windows, then run minimizeall again. All windows would be unminimized!

I have tried to make small changes to original code of main.js, but i can't because i don't understand it.

Thats why i wrote my own version. Then i changed my version to version of zzag, because his code is more simple. I have tested both versions.

BUG: 356161

Reviewers: colomar, KWin, zzag

Reviewed By: KWin, zzag

Subscribers: romangg, zzag, kwin

Tags: KWin

Differential Revision: https://phabricator.kde.org/D24044

Details

Committed
zzagSep 18 2019, 3:11 PM
Reviewer
KWin
Differential Revision
D24044: Emphasize minimization in MinimizeAll script
Parents
R108:2ad37c449fd6: Port away from deprecated QString::sprintf
Branches
Unknown
Tags
Unknown