D8722: Use new folder-stash icon
Elvis Angelaccio
noreply at phabricator.kde.org
Wed Nov 8 17:11:07 GMT 2017
elvisangelaccio requested changes to this revision.
elvisangelaccio added inline comments.
This revision now requires changes to proceed.
INLINE COMMENTS
> dolphinmainwindow.cpp:1099
> stashSplit->setToolTip(i18nc("@info", "Opens the stash virtual directory in a split window"));
> - stashSplit->setIcon(QIcon::fromTheme(QStringLiteral("folder-visiting")));
> + stashSplit->setIcon(QIcon::fromTheme(QStringLiteral("folder-stash")));
> stashSplit->setCheckable(false);
Please use the old `folder-stash` icon as fallback (in the `fromTheme` call), in case the user has a version of breeze-icons older then 5.40
REPOSITORY
R318 Dolphin
REVISION DETAIL
https://phabricator.kde.org/D8722
To: ngraham, elvisangelaccio, #dolphin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20171108/acc83763/attachment.htm>
More information about the kfm-devel
mailing list