Fix minimise all applet on wayland with an active window
ClosedPublic

Authored by davidedmundson on Mar 7 2018, 4:12 PM.

Details

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

Diff Detail

Repository
R114 Plasma Addons
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
davidedmundson created this revision.Mar 7 2018, 4:12 PM
Restricted Application added a project: Plasma. · View Herald TranscriptMar 7 2018, 4:12 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson requested review of this revision.Mar 7 2018, 4:12 PM
broulik accepted this revision.Mar 15 2018, 12:04 PM
This revision is now accepted and ready to land.Mar 15 2018, 12:04 PM
This revision was automatically updated to reflect the committed changes.