Review Request: Enable kdialog to set an icon for passivepopup as well
Kai Uwe Broulik
kde at privat.broulik.de
Fri May 6 15:52:02 BST 2011
-----------------------------------------------------------
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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20110506/a27dd0ad/attachment.htm>
More information about the kde-core-devel
mailing list