D14911: [effects/maximize] Make it work on Wayland

Vlad Zagorodniy <[email protected]> Sun, 19 Aug 2018 19:51:06 +0000

Newsgroups	gmane.comp.kde.devel.kwin
Message-ID	<[email protected]>

zzag added a comment.


  In D14911#310976 <https://phabricator.kde.org/D14911#310976>, @davidedmundson wrote:
  
  > Thanks for looking.
  >
  > I'm not convinced by the bugzilla analysis.
  >
  > Imho we shouldn't be updating the state (as far as effects are concerned) to maximised till we get the ack configure.
  >
  > Which should be synced to the surface commit which comes with the new buffer size.
  
  
  I'd would prefer the proposed solution. If I understand it correctly, when windowMaximizedSTateChanged signal is emitted, we can start animation.

REPOSITORY
  R108 KWin

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

To: zzag, #kwin
Cc: davidedmundson, kwin, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart