D16454: [effects/fadedesktop] Rewrite it
Vlad Zagorodniy <[email protected]>
| Newsgroups | gmane.comp.kde.devel.kwin |
|---|---|
| Message-ID | <[email protected]> |
zzag added inline comments.
INLINE COMMENTS
> davidedmundson wrote in main.js:29
> > In order to delete window.animation
>
> effect.animationEnded.connect(function(window) {
>
> delete window.animation;
>
> });
>
> (And if we were doing multiple animations in an array then we change the above code to remove the array entry and delete the property if it's empty)
Oh, I forgot that there's the second parameter.
REPOSITORY
R108 KWin
REVISION DETAIL
https://phabricator.kde.org/D16454
To: zzag, #kwin
Cc: davidedmundson, kwin, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart