D6089: Show no notification icon if there is no

Aleix Pol Gonzalez noreply at phabricator.kde.org
Mon Jul 10 22:09:13 UTC 2017


apol added inline comments.

INLINE COMMENTS

> nicolasfella wrote in notification.cpp:105
> setIconName(QString()) gives me some default icon, but setIconName("foobar") gives me my desired behaviour (no icon at all)

Meh. Not your fault then.

Can you at least add a comment saying so?

REPOSITORY
  R224 KDE Connect

REVISION DETAIL
  https://phabricator.kde.org/D6089

To: nicolasfella, #kde_connect
Cc: albertvaka, apol, #kde_connect, tfella, aboudhar, seebauer, progwolff, MayeulC, menasshock, ach, hkaelberer
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20170710/ed19e5cf/attachment.html>


More information about the KDEConnect mailing list