D22255: Fix maximize Apply Now rule

Roman Gilg noreply at phabricator.kde.org
Mon Jul 8 09:21:54 BST 2019


romangg accepted this revision.
romangg added a comment.
This revision is now accepted and ready to land.


  That we need this conditional removed indicates that the structure is wrong how we check for the maximize rule. And looking at the code we have methods `maximize`, `setMaximize` and `changeMaximize` in quick succession. I'm pretty sure we could remove at least one of these three.
  
  Nevertheless let's get this in as a quick fix.

REPOSITORY
  R108 KWin

BRANCH
  make-maximize-apply-now-rules-work

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

To: zzag, #kwin, romangg
Cc: romangg, kwin, LeGast00n, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20190708/56b740eb/attachment.html>


More information about the kwin mailing list