D12760: Improve single-instance application behavior

Marco Martin noreply at phabricator.kde.org
Fri May 11 09:37:55 UTC 2018


mart added inline comments.

INLINE COMMENTS

> main.cpp:71
> +    connect(service, &KDBusService::activateRequested, this, [this]() {
> +        KWindowSystem::forceActiveWindow(display->winId());
> +    });

KWindowSystem::forceActiveWindow should never be used by normal applications, it's reserved for workspaces and won't work on Wayland.
QWidget::activateWindow should be enough, even if that's not guaranteed to actually raise the window, is more correct.

REPOSITORY
  R102 KInfoCenter

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

To: volkov, #plasma
Cc: mart, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180511/a4f60412/attachment.html>


More information about the Plasma-devel mailing list