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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.