davidedmundson accepted this revision.
davidedmundson added inline comments.
This revision is now accepted and ready to land.
INLINE COMMENTS
> blur.cpp:76
> connect(effects, SIGNAL(screenGeometryChanged(QSize)), this,
> SLOT(slotScreenGeometryChanged()));
> + connect(effects, &EffectsHandler::xcbConnectionChanged, this,
> + [this] {
I was expecting you to do what you for the present windows effect with calling
your local lambda announceSupportProperties.
IMHO it's neater than duplicating - but this is fine too.
REPOSITORY
R108 KWin
BRANCH
effects-xcb-connection-changed
REVISION DETAIL
https://phabricator.kde.org/D7761
To: graesslin, #kwin, #plasma, davidedmundson
Cc: davidedmundson, plasma-devel, kwin, ZrenBot, progwolff, lesliezhai,
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
