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

Reply via email to