Review Request: Add context menu entry to modify contact notifications

[Rohan Garg]

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

(Updated Sept. 28, 2012, 9:29 p.m.)


Review request for Telepathy.


Changes
-------

Also add a Icon to go with the context menu entry


Description
-------

Add context menu entry to modify contact notifications

The position of the configure entry is debatable, should it be at the bottom with "Show info" ? I'd like to hear your opinions :)


Diffs (updated)
-----

  CMakeLists.txt 7cb36cf9b065d572799308fdc52b746f3388f527 
  context-menu.h 5a30c6d326c7401d7f226fdca1cb90b65d650403 
  context-menu.cpp 10f3d90cc7de3c7ce5072d3a6dc449f9ecb8e721 

Diff: http://git.reviewboard.kde.org/r/106353/diff/


Testing
-------

Right click a contact > click "Configure Notifications" > Brings up notification configuration dialog


Screenshots
-----------


  http://git.reviewboard.kde.org/r/106353/s/716/


Thanks,

Rohan Garg

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20120928/4fa085cc/attachment-0001.html>