D17188: Notification portal: support pixmaps in notifications

[Kai Uwe Broulik]

broulik added a comment.


  +1

INLINE COMMENTS

> notifybyflatpak.h:40
>  public:
> +    struct PortalIcon {
> +        QString str;

Does this need to be in the header file?

REPOSITORY
  R289 KNotifications

BRANCH
  master

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

To: jgrulich, broulik, apol
Cc: apol, kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20181127/f196d3a1/attachment-0001.html>