----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103425/ -----------------------------------------------------------
Review request for kdelibs and Marco Martin. Description ------- Set the icon for KStatusNotifierItem by setIconByPixmap(), then by setIconByName(). If setIconByPixmap() is called to set the first icon again, the icon doesn't change. The same bug seems likely to happen with other pairs of icon setting methods. This patch fixes this bug. Diffs ----- kdeui/notifications/kstatusnotifieritem.cpp aecb81c Diff: http://git.reviewboard.kde.org/r/103425/diff/diff Testing ------- Tested switching setIconByPixmap()/setIconByName() in KAlarm. Thanks, David Jarvie
