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

(Updated 2010-03-18 22:44:19.497406)


Review request for Plasma.


Changes
-------

Version that is now working from all tests done in few minutes.
Icons are showing and are updating.
Tool tips work.


Summary
-------

Preliminary patch to use the data engine for status notifier items instead of 
directly the dbus protocol.
This patch is only a preliminary version to get some early feedback. More work 
is actually needed.


Diffs (updated)
-----

  
/trunk/KDE/kdebase/workspace/plasma/generic/applets/systemtray/protocols/dbussystemtray/dbussystemtrayprotocol.h
 1104901 
  
/trunk/KDE/kdebase/workspace/plasma/generic/applets/systemtray/protocols/dbussystemtray/dbussystemtrayprotocol.cpp
 1104901 
  
/trunk/KDE/kdebase/workspace/plasma/generic/applets/systemtray/protocols/dbussystemtray/dbussystemtraytask.h
 1104901 
  
/trunk/KDE/kdebase/workspace/plasma/generic/applets/systemtray/protocols/dbussystemtray/dbussystemtraytask.cpp
 1104901 
  
/trunk/KDE/kdebase/workspace/plasma/generic/applets/systemtray/protocols/dbussystemtray/dbussystemtraywidget.h
 1104901 
  
/trunk/KDE/kdebase/workspace/plasma/generic/applets/systemtray/protocols/dbussystemtray/dbussystemtraywidget.cpp
 1104901 
  
/trunk/KDE/kdebase/workspace/plasma/generic/applets/systemtray/statusnotifierwatcher/CMakeLists.txt
 1104901 
  
/trunk/KDE/kdebase/workspace/plasma/generic/applets/systemtray/statusnotifierwatcher/statusnotifierwatcher.desktop
 1104901 
  
/trunk/KDE/kdebase/workspace/plasma/generic/applets/systemtray/statusnotifierwatcher/statusnotifierwatcher.h
 1104901 
  
/trunk/KDE/kdebase/workspace/plasma/generic/applets/systemtray/statusnotifierwatcher/statusnotifierwatcher.cpp
 1104901 

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


Testing
-------

Kinda works. Tested in a kde 4.4 session. Even if the port is not complete, the 
applet is till functionnal.


Thanks,

Matthieu

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to