davidedmundson created this revision. davidedmundson added a reviewer: Plasma. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. davidedmundson requested review of this revision.
REVISION SUMMARY There's an async round trip after requesting isMinimized to kwin and then back to Plasma with the updated roles. There's a quirk on wayland that IsActive will update before we get isMinimized changed. The old code would see that a window is currently not minimised and deactivate the effect. This patch checks the roles to only check after IsMinimized changes. TEST PLAN More clicking REPOSITORY R114 Plasma Addons BRANCH master REVISION DETAIL https://phabricator.kde.org/D11133 AFFECTED FILES applets/minimizeall/package/contents/ui/main.qml To: davidedmundson, #plasma Cc: plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart