D8722: Use new folder-stash icon

Christoph Feck noreply at phabricator.kde.org
Thu Nov 9 11:10:42 GMT 2017


cfeck added inline comments.

INLINE COMMENTS

> cfeck wrote in dolphinmainwindow.cpp:1099
> This probably does not compile, see http://doc.qt.io/qt-5/qicon.html#fromTheme-1

EDIT: Actually, it probably does compile, but you want QIcon::fromTheme() as a fallback, not an icon loaded from a filename, which is the implicit result of QIcon(QString).

REPOSITORY
  R318 Dolphin

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

To: ngraham, elvisangelaccio, #dolphin
Cc: cfeck
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20171109/35c626d2/attachment.htm>


More information about the kfm-devel mailing list