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