D7761: [effects] Support xcbConnectionChanged for support properties

[David Edmundson]

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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170910/38914c85/attachment-0001.html>