Commit 089c1c77 authored by Friedrich W. H. Kossebau's avatar Friedrich W. H. Kossebau
Browse files

Use "edit-delete" icon for buttons which delete data

Summary:
The "list-remove" icon does not signal the user that the related action
actually will results in non-recoverable data deletion. Removing an item
from a list does not necessarily mean the item is destroyed.

Using "edit-delete" instead makes the deletion effect more obvious to the
user. In the Breeze icon theme this also means the related signal color is
used to point out the nature of the action.

Reviewers: #kdevelop, aaronpuchert

Reviewed By: #kdevelop, aaronpuchert

Subscribers: aaronpuchert, kdevelop-devel

Tags: #kdevelop

Differential Revision: https://phabricator.kde.org/D14304
parent c15e8144
......@@ -41,7 +41,7 @@
<string>Remove</string>
</property>
<property name="icon">
<iconset theme="list-remove">
<iconset theme="edit-delete">
<normaloff/>
</iconset>
</property>
......
......@@ -59,7 +59,7 @@
<string>Remove profile</string>
</property>
<property name="icon">
<iconset theme="list-remove"/>
<iconset theme="edit-delete"/>
</property>
</widget>
</item>
......@@ -114,7 +114,7 @@
<string>Remove variable</string>
</property>
<property name="icon">
<iconset theme="list-remove"/>
<iconset theme="edit-delete"/>
</property>
</widget>
</item>
......
......@@ -47,7 +47,7 @@
<string>Delete</string>
</property>
<property name="icon">
<iconset theme="list-remove"/>
<iconset theme="edit-delete"/>
</property>
</widget>
</item>
......
......@@ -62,7 +62,7 @@
<string/>
</property>
<property name="icon">
<iconset theme="list-remove"/>
<iconset theme="edit-delete"/>
</property>
</widget>
</item>
......
......@@ -52,7 +52,7 @@
<string>Remove</string>
</property>
<property name="icon">
<iconset theme="list-remove"/>
<iconset theme="edit-delete"/>
</property>
</widget>
</item>
......
......@@ -35,7 +35,7 @@
<string>&amp;Remove</string>
</property>
<property name="icon">
<iconset theme="list-remove"/>
<iconset theme="edit-delete"/>
</property>
</widget>
</item>
......
......@@ -41,7 +41,7 @@ DefinesWidget::DefinesWidget( QWidget* parent )
QAction* delDefAction = new QAction( i18n("Delete Define"), this );
delDefAction->setShortcut( QKeySequence(Qt::Key_Delete) );
delDefAction->setShortcutContext( Qt::WidgetWithChildrenShortcut );
delDefAction->setIcon( QIcon::fromTheme(QStringLiteral("list-remove")) );
delDefAction->setIcon( QIcon::fromTheme(QStringLiteral("edit-delete")) );
ui->defines->addAction( delDefAction );
ui->defines->setContextMenuPolicy( Qt::ActionsContextMenu );
connect( delDefAction, &QAction::triggered, this, &DefinesWidget::deleteDefine );
......
......@@ -138,7 +138,7 @@
<string/>
</property>
<property name="icon">
<iconset theme="list-remove"/>
<iconset theme="edit-delete"/>
</property>
</widget>
</item>
......
......@@ -49,7 +49,7 @@
<string>Remove</string>
</property>
<property name="icon">
<iconset theme="list-remove">
<iconset theme="edit-delete">
<normaloff/>
</iconset>
</property>
......
......@@ -47,7 +47,7 @@
<string/>
</property>
<property name="icon">
<iconset theme="list-remove"/>
<iconset theme="edit-delete"/>
</property>
</widget>
</item>
......
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