D11584: Set a transient parent for SNI context menus

[Kai Uwe Broulik]

broulik 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);
>  

I think this change is fine. Except that the `MouseArea` covers the entire list item, so for hidden SNIs the app might get coordinates outside of its icon.

For context menu we ignore the coordinates anyway and place the menu relative to the icon (so it never covers it). For `Activate` the app might want to know where it was clicked. In any case this needs a bit of cleaning up, we pass parameters around in places where they're ignored and so on.

REPOSITORY
  R120 Plasma Workspace

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

To: fvogt, #plasma
Cc: broulik, davidedmundson, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180323/fd944adb/attachment-0001.html>