Commit a4b8511d authored by Andrew Crouthamel's avatar Andrew Crouthamel Committed by Henrik Fehlauer

Remove symbolic icons in favor of standard icons

Summary:
This removes -symbolic icons in favor of theme-neutral standard icon names to ensure compatibility.

Follow-up to efaebe29 as discussed in D10770.

Before:
{F5729929}
After:
{F5729928}

Test Plan: Test Open Containing Folder and Trash menu options to ensure functionality.

Reviewers: #gwenview, rkflx, ngraham

Reviewed By: #gwenview, rkflx, ngraham

Subscribers: rkflx, ngraham

Tags: #gwenview

Differential Revision: https://phabricator.kde.org/D10843
parent 75f1996c
......@@ -158,7 +158,10 @@ FileOpsContextManagerItem::FileOpsContextManagerItem(ContextManager* manager, QL
mTrashAction = file->addAction("file_trash", this, SLOT(trash()));
mTrashAction->setText(i18nc("Verb", "Trash"));
mTrashAction->setIcon(QIcon::fromTheme("user-trash-symbolic"));
// TODO: Remove trash-empty from the below line when a new user-trash icon
// is released by Breeze that adheres to monochrome on HiDPI scaled displays.
// See Bug #391078
mTrashAction->setIcon(QIcon::fromTheme("trash-empty", QIcon::fromTheme("user-trash")));
actionCollection->setDefaultShortcut(mTrashAction, Qt::Key_Delete);
mDelAction = file->addAction(KStandardAction::DeleteFile, this, SLOT(del()));
......@@ -184,7 +187,7 @@ FileOpsContextManagerItem::FileOpsContextManagerItem(ContextManager* manager, QL
mOpenContainingFolderAction = file->addAction("file_open_containing_folder", this, SLOT(openContainingFolder()));
mOpenContainingFolderAction->setText(i18n("Open Containing Folder"));
mOpenContainingFolderAction->setIcon(QIcon::fromTheme("folder-open-symbolic"));
mOpenContainingFolderAction->setIcon(QIcon::fromTheme("document-open-folder"));
mRegularFileActionList
<< mRenameAction
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment