D3484: Center systemmonitor window properly on multi-screen setup

[Valeriy Malov]

valeriymalov added inline comments.

INLINE COMMENTS

> broulik wrote in ksystemactivitydialog.cpp:79
> Is it intentional you removed the `KeepAbove`?

Yes, per my interpretation of the comment that application shouldn't force it's window properties like that because it interferes with WM settings.

There doesn't really seem to be a consensus if that's right, though? From what I understand there doesn't seem to be any API to set preferred window geometry and keepabove AND respect specific kwin settings.

REPOSITORY
  R120 Plasma Workspace

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

To: valeriymalov, #plasma_workspaces, aacid, mart
Cc: broulik, cfeck, davidedmundson, sebas, aacid, graesslin, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180828/cf327aa0/attachment.html>