<table><tr><td style="">ngraham added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D26773">View Revision</a></tr></table><br /><div><div><p>There are legitimate questions regarding whether or not we should add this feature, and if this is the right place for it (a KWin effect intuitively seems more appropriate to me, if we upstream it). However aside from that, is this your code? The absence of a license header is therefore suspicious. Is this code taken straight from the plugin? If so, that's a big no-no; you need the author's permission to take their code and put it here.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26773">https://phabricator.kde.org/D26773</a></div></div><br /><div><strong>To: </strong>niccolove, Plasma<br /><strong>Cc: </strong>ngraham, anthonyfieroni, davidedmundson, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart<br /></div>