D18728: [kcmkwin] Properly load effects
Vlad Zagorodniy <[email protected]>
| Newsgroups | gmane.comp.kde.devel.kwin |
|---|---|
| Message-ID | <[email protected]> |
zzag created this revision. zzag added a reviewer: KWin. Herald added a project: KWin. Herald added a subscriber: kwin. zzag requested review of this revision. REVISION SUMMARY In order to have actual value of SupportedRole, EffectsHandler has to communicate with KWin asynchronously. So, in theory the model should notify about loaded effects after receiving a response from KWin, but that's not the case. TEST PLAN Desktop Effects KCM no longer flashes when resetting changes. REPOSITORY R108 KWin BRANCH split-desktop-effects-kcm REVISION DETAIL https://phabricator.kde.org/D18728 AFFECTED FILES kcmkwin/common/effectsmodel.cpp kcmkwin/common/effectsmodel.h kcmkwin/kwindesktop/animationsmodel.cpp To: zzag, #kwin Cc: kwin, jraleigh, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, ngraham, alexeymin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart