hpereiradacosta accepted this revision. hpereiradacosta added a comment. This revision is now accepted and ready to land.
Makes sense, thanks ! In principle, the removal of Helper::connection, although is a change unrelated to the commit purpose. Maybe put it in a different commit ? (the story behind if I remember right is that QX11Info::connection have not always been available in the past. hence the method) REPOSITORY R31 Breeze BRANCH use-qx11info-is-compositing-manager-active REVISION DETAIL https://phabricator.kde.org/D26978 To: zzag, #plasma, hpereiradacosta Cc: plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart