Review Request: Draft: kstatusnotifieritem->notifications integration, kdelibs part

Marco Martin notmart at gmail.com
Tue Nov 17 21:20:23 CET 2009


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

(Updated 2009-11-17 20:20:23.217877)


Review request for Plasma.


Changes
-------

way simpler, just keep the signal


Summary
-------

This is an early, early draft just for testing the visualization.
Adds a new signal, where a kstatusnotifieritem can claim the ownership of a notification udi, and a property of a list of owned notifications (probably unneeded)
at the moment it does it only with the showmessage function, that i would remove in the final api (we should reeeally force to use knotification i think?)
it should be integrated with the Davide Bettio KNotification Pattch.


Diffs (updated)
-----

  /trunk/KDE/kdelibs/kdeui/notifications/kstatusnotifieritem.cpp 1050579 
  /trunk/KDE/kdelibs/kdeui/notifications/kstatusnotifieritemdbus_p.h 1050579 
  /trunk/KDE/kdelibs/kdeui/notifications/org.kde.StatusNotifierItem.xml 1050579 

Diff: http://reviewboard.kde.org/r/2121/diff


Testing
-------


Thanks,

Marco



More information about the Plasma-devel mailing list