https://bugs.kde.org/show_bug.cgi?id=360332

--- Comment #1 from David Edmundson <k...@davidedmundson.co.uk> ---
This fixes it:

diff --git
a/applets/systemtray/package/contents/ui/items/StatusNotifierItem.qml
b/applets/systemtray/package/contents/ui/items/StatusNotifierItem.qml
index 9cded5d..8bc9567 100644
--- a/applets/systemtray/package/contents/ui/items/StatusNotifierItem.qml
+++ b/applets/systemtray/package/contents/ui/items/StatusNotifierItem.qml
@@ -69,13 +69,13 @@ AbstractItem {
         case Qt.LeftButton: {
             var service =
statusNotifierSource.serviceForSource(DataEngineSource);
             var operation = service.operationDescription("Activate");
-            operation.x = pos.x;
+            operation.x = x;

             // kmix shows main window instead of volume popup if (parent.x,
parent.y) == (0, 0), which is the case here.
             // I am passing a position right below the panel (assuming panel
is at screen's top).
             // Plasmoids' popups are already shown below the panel, so this
make kmix's popup more consistent
             // to them.
-            operation.y = pos.y + parent.height + 6;
+            operation.y = y;
             service.startOperationCall(operation);
             break;
         }
@@ -95,9 +95,9 @@ AbstractItem {
         case Qt.MiddleButton:
             var service =
statusNotifierSource.serviceForSource(DataEngineSource);
             var operation = service.operationDescription("SecondaryActivate");
-            operation.x = pos.x;
+            operation.x = x;

-            operation.y = pos.y;
+            operation.y = y;
             service.startOperationCall(operation);
             break;
             break;


In rewrite someone is trying to work round dodgy clients, at the expense of
screwing up everyone else.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to