----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101282/ -----------------------------------------------------------
(Updated May 6, 2011, 2:52 p.m.) Review request for KDE Base Apps. Changes ------- Implements --passiveicon support to display a custom icon in both Plasma Desktop notifications and legacy KPassivePopup fallback. Fixes KPassivePopup fallback notifications not showing up (or closing after a microsecond) Summary ------- Since I use kdialog a lot, I wanted it to be able to set an icon for passive Plasma notifications as well. I saw that the icon was hardcoded to dialog-information and tried to enable it to parse the --icon command line argument and pass it to the dbus call that triggers the notification. I could not test the patch however, since I do not (yet) have a full development environment or its dependencies, and I do not even know if this is the right approach, but I hope you can have a look at it. Diffs (updated) ----- kdialog/kdialog.cpp b80ad9a Diff: http://git.reviewboard.kde.org/r/101282/diff Testing ------- Thanks, Kai Uwe
