https://bugs.kde.org/show_bug.cgi?id=356161

Vlad Zahorodnii <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|http://commits.kde.org/kwin |https://commits.kde.org/kwi
                   |/8885a6798a348cc709a3423f37 |n/fec0ab818d2460d2b2dd48e69
                   |593cc3f6ce6eee              |8eb2d48a2f1f0ab
             Status|REOPENED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #15 from Vlad Zahorodnii <[email protected]> ---
Git commit fec0ab818d2460d2b2dd48e698eb2d48a2f1f0ab by Vlad Zahorodnii, on
behalf of Anton Smerkov.
Committed on 18/09/2019 at 15:11.
Pushed by vladz into branch 'master'.

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.

Reviewers: colomar, #kwin, zzag

Reviewed By: #kwin, zzag

Subscribers: romangg, zzag, kwin

Tags: #kwin

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

M  +25   -15   scripts/minimizeall/contents/code/main.js

https://commits.kde.org/kwin/fec0ab818d2460d2b2dd48e698eb2d48a2f1f0ab

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to