Review Request 122171: Implement SystemTrayMenuItem::setMenu() correctly.

Dmitry Shachnev mitya57 at gmail.com
Thu Jan 29 23:08:20 UTC 2015


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

(Updated Jan. 29, 2015, 11:08 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma.


Repository: frameworkintegration


Description
-------

Implement SystemTrayMenuItem::setMenu() correctly.

We need to actually set the action menu, not just store it somewhere, to make submenus in tray icons working.


Diffs
-----

  src/platformtheme/kdeplatformsystemtrayicon.h 3c1bbf7 
  src/platformtheme/kdeplatformsystemtrayicon.cpp d3a1d4f 

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


Testing
-------

Tested this app: https://github.com/dglent/meteo-qt


Thanks,

Dmitry Shachnev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150129/f0df147e/attachment.html>


More information about the Plasma-devel mailing list