Review Request 125589: [Task Manager] Align context menu to task item

[Kai Uwe Broulik]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125589/
-----------------------------------------------------------

Review request for Plasma, KDE Usability and Eike Hein.


Repository: plasma-desktop


Description
-------

One thing that always bugged me with the Plasma 5 Task Manager was the context menu covering the task item (I have a top panel) when I, conforming to Fitt's Law, threw my mouse at the edge and right clicked.

This also restores Plasma 4 behavior where the context menus din't overlap the panels. I chose to align the context menu with the item itself rather than just the panel, which is what Plasma 4 did, to communicate some affinity to the task item, especially in multi-row task managers.

I changed the QMetaObject::invokeMethod to a QTimer::singleShot with lambda, don't know if that's the right approach. Also it's perhaps not optimal for me to copy the QPoint into the lambda.


Diffs
-----

  applets/taskmanager/plugin/backend.h 2b9d9e9 
  applets/taskmanager/plugin/backend.cpp 97ed755 

Diff: https://git.reviewboard.kde.org/r/125589/diff/


Testing
-------

Tested with single and multi row task manager in top/bottom/left/right panels. Works. Did not test with multiple monitors.

Perhaps some margins need to be taken into account additionally, note the small gap between the indicator line and the menu border


File Attachments
----------------

Outer left two columns
  https://git.reviewboard.kde.org/media/uploaded/files/2015/10/10/f62b81df-4b0b-4b11-8c66-aa96dc56850a__taskmanageroverlap1.png
Inner left two columns
  https://git.reviewboard.kde.org/media/uploaded/files/2015/10/10/7b128305-04e8-4734-82d0-b1853bbc8471__taskmanageroverlap2.png


Thanks,

Kai Uwe Broulik

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151010/34fa8084/attachment-0001.html>