D6799: Apply maximize effect to full screen changes

2018-01-02 Thread Kai Uwe Broulik
broulik abandoned this revision. broulik added a comment. Restricted Application edited projects, added Plasma; removed KWin. Superseded by https://phabricator.kde.org/D9391 REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D6799 To: broulik, #plasma, graesslin, #vdg Cc: n

D6799: Apply maximize effect to full screen changes

2017-10-22 Thread Nathaniel Graham
ngraham added a comment. Can we land this? It's a nice little polish change. REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D6799 To: broulik, #plasma, graesslin, #vdg Cc: ngraham, plasma-devel, kwin, #kwin, bwowk, ZrenBot, progwolff, lesliezhai, ali-mohamed, hardening

D6799: Apply maximize effect to full screen changes

2017-07-20 Thread Kai Uwe Broulik
broulik added a comment. I was surprised myself. :) > Would this fix: https://bugs.kde.org/show_bug.cgi?id=382161 ? I looked at it when I did this, actually, but this particular patch does not help. //But// since we now have the information we could fix this eventually. REPOSITORY

D6799: Apply maximize effect to full screen changes

2017-07-20 Thread Martin Flöser
graesslin added a comment. If we do this we need to rename the effect - it is no longer a maximize effect. Otherwise I'm surprised of how little impact it has on the code base. Would this fix: https://bugs.kde.org/show_bug.cgi?id=382161 ? REPOSITORY R108 KWin REVISION DETAIL https:/

D6799: Apply maximize effect to full screen changes

2017-07-20 Thread Kai Uwe Broulik
broulik created this revision. Restricted Application added a project: KWin. Restricted Application added subscribers: KWin, kwin, plasma-devel. REVISION SUMMARY Animates the window also when entering/leaving full screen like it does when maximizing/unmaximizing. TEST PLAN - Entering and lea