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