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