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