Review Request: PATCH: Configuration Dialog for Turning off Notifications in the Panel

guitarist198 at yahoo.com guitarist198 at yahoo.com
Sat Jan 17 07:08:08 CET 2009


-----------------------------------------------------------
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



More information about the Plasma-devel mailing list