D8722: Use new folder-stash icon
Christoph Feck
noreply at phabricator.kde.org
Thu Nov 9 11:14:21 GMT 2017
cfeck added inline comments.
INLINE COMMENTS
> cfeck wrote in dolphinmainwindow.cpp:1099
> 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).
And thinking about it, fallbacks with QIcon::fromTheme() are probably useless, because our icon loader does automatic fallback according to XDG spec. In other words, if "folder-stash" is not available, it will load "folder". Even if this was not available, it would load a default "unknown" icon.
If you really want the fallback to a different icon to work, you probably need to use KIconLoader classes directly.
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/bbe3a815/attachment.htm>
More information about the kfm-devel
mailing list