-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.vidsolbach.de/r/331/
-----------------------------------------------------------

Review request for Plasma.


Summary
-------

This patch allows one to turn on and off jobs and notifications in the panel 
via the systemtray configuration dialog.  This is in response to Aaron Seigo's 
blog entry: 
http://aseigo.blogspot.com/2009/01/todays-tip-turning-off-fancy-schmancy.html

Withing manager.cpp, functions to unregister the notification and job protocols 
was created, as well as a function to unset a protocol (disconnect it).  These 
were based on the already present functions to register and setup protocols.

Within applet.cpp, a new page was added to the configuration dialog called 
"Notifications."  This page includes the two check boxes that allow one to turn 
on and turn off notifications.  A possible area where that may need to be fixed 
is the strings for the check boxes.  Presently they are "Show Jobs" and "Show 
Notifications."

Thank you for your time and review.


Diffs
-----

  /trunk/KDE/kdebase/workspace/plasma/applets/systemtray/core/manager.h
  /trunk/KDE/kdebase/workspace/plasma/applets/systemtray/core/manager.cpp
  /trunk/KDE/kdebase/workspace/plasma/applets/systemtray/ui/applet.cpp

Diff: http://reviewboard.vidsolbach.de/r/331/diff


Testing
-------

Testing of suppression of job progress.
Testing of suppression of notifications.
Testing of allowing job progress.
Testing of allowing notifications. 


Thanks,

distortedlogic

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

Reply via email to