davidedmundson added inline comments.

INLINE COMMENTS

> StatusNotifierItem.qml:63
>      onClicked: {
> -        var pos = plasmoid.nativeInterface.popupPosition(taskIcon, 0, 0);
> +        var pos = plasmoid.nativeInterface.popupPosition(taskIcon, mouse.x, 
> mouse.y);
>  

This change is unrelated, and not a change I would support.

> systemtray.cpp:297
>          KAcceleratorManager::manage(menu);
> +        menu->winId();
> +        
> menu->windowHandle()->setTransientParent(statusNotifierIcon->window());

This part is all fine.

REPOSITORY
  R120 Plasma Workspace

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

To: fvogt, #plasma
Cc: davidedmundson, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

Reply via email to