D14688: [libkwineffects] Expose getting/setting activeFullScript to scripted effects

Vlad Zagorodniy <[email protected]>
Newsgroups gmane.comp.kde.devel.kwin
Message-ID <[email protected]>
zzag added a comment.


  In D14688#305350 <https://phabricator.kde.org/D14688#305350>, @davidedmundson wrote:
  
  > We disable all effects in initTestCase
  
  
  Oh, silly me, I forgot about
  
    const auto builtinNames = BuiltInEffects::availableEffectNames() << loader.listOfKnownEffects();
    for (QString name : builtinNames) {
        plugins.writeEntry(name + QStringLiteral("Enabled"), false);
    }
  
  sorry.
  
  ---
  
  One last nitpick: please rename AnimationSettings::Fullscreen to AnimationSettings::FullScreen and isFullscreen to isFullScreen(or just fullScreen).

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D14688

To: davidedmundson, #kwin
Cc: zzag, kwin, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, 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.