davidedmundson added a comment.

  We still need https://phabricator.kde.org/D6571, with my proposed change. 
That's a special case where we delete the connection before the QPA. 
  My added connect means that we get that signal regardless of whether we're in 
Kwin or not.
  
  What we wouldn't need is the ownership changes you did for Breeze (and would 
otherwise still need doing in Oxygen/QtCurve )
  
  > May I ask for a small unit test for the new functionality?
  
  For you, a *big* unit test.

REPOSITORY
  R127 KWayland

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

To: davidedmundson
Cc: graesslin, plasma-devel, #frameworks, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, eliasp, sebas, apol, mart, hein, lukas

Reply via email to