Verified Commit 58941662 authored by Shashwat Jolly's avatar Shashwat Jolly Committed by Daniel Vrátil
Browse files

Add icons for some menu options in KMail

Summary:
In KMail, the following menu options did not have icons:

- Settings -> Add Accounts (Main window)
- Settings -> Configure Filters (Main window)
- Go -> Jump to Folder (Main window)
- Tools -> Spellchecker (Compose window)

This commit adds relevant icons for these.

Reviewers: #kde_pim_kmail, dvratil

Reviewed By: dvratil

Subscribers: dvratil, kde-pim

Tags: #kde_pim

Differential Revision: https://phabricator.kde.org/D28063
parent 0ba46e17
...@@ -1341,7 +1341,7 @@ void KMComposerWin::setupActions() ...@@ -1341,7 +1341,7 @@ void KMComposerWin::setupActions()
KStandardAction::configureToolbars(this, &KMComposerWin::slotEditToolbars, actionCollection()); KStandardAction::configureToolbars(this, &KMComposerWin::slotEditToolbars, actionCollection());
KStandardAction::preferences(kmkernel, &KMKernel::slotShowConfigurationDialog, actionCollection()); KStandardAction::preferences(kmkernel, &KMKernel::slotShowConfigurationDialog, actionCollection());
action = new QAction(i18n("&Spellchecker..."), this); action = new QAction(QIcon::fromTheme(QStringLiteral("tools-check-spelling")), i18n("&Spellchecker..."), this);
action->setIconText(i18n("Spellchecker")); action->setIconText(i18n("Spellchecker"));
actionCollection()->addAction(QStringLiteral("setup_spellchecker"), action); actionCollection()->addAction(QStringLiteral("setup_spellchecker"), action);
connect(action, &QAction::triggered, this, &KMComposerWin::slotSpellcheckConfig); connect(action, &QAction::triggered, this, &KMComposerWin::slotSpellcheckConfig);
......
...@@ -3332,7 +3332,7 @@ void KMMainWidget::setupActions() ...@@ -3332,7 +3332,7 @@ void KMMainWidget::setupActions()
//----- Settings Menu //----- Settings Menu
{ {
QAction *action = new QAction(i18n("Configure &Filters..."), this); QAction *action = new QAction(QIcon::fromTheme(QStringLiteral("dialog-filters")), i18n("Configure &Filters..."), this);
action->setMenuRole(QAction::NoRole); // do not move to application menu on OS X action->setMenuRole(QAction::NoRole); // do not move to application menu on OS X
actionCollection()->addAction(QStringLiteral("filter"), action); actionCollection()->addAction(QStringLiteral("filter"), action);
connect(action, &QAction::triggered, this, &KMMainWidget::slotFilter); connect(action, &QAction::triggered, this, &KMMainWidget::slotFilter);
...@@ -3343,7 +3343,7 @@ void KMMainWidget::setupActions() ...@@ -3343,7 +3343,7 @@ void KMMainWidget::setupActions()
connect(action, &QAction::triggered, this, &KMMainWidget::slotManageSieveScripts); connect(action, &QAction::triggered, this, &KMMainWidget::slotManageSieveScripts);
} }
{ {
QAction *action = new QAction(i18n("&Add Account..."), this); QAction *action = new QAction(QIcon::fromTheme(QStringLiteral("list-resource-add")), i18n("&Add Account..."), this);
actionCollection()->addAction(QStringLiteral("accountWizard"), action); actionCollection()->addAction(QStringLiteral("accountWizard"), action);
connect(action, &QAction::triggered, mLaunchExternalComponent, &KMLaunchExternalComponent::slotAccountWizard); connect(action, &QAction::triggered, mLaunchExternalComponent, &KMLaunchExternalComponent::slotAccountWizard);
} }
...@@ -3453,7 +3453,7 @@ void KMMainWidget::setupActions() ...@@ -3453,7 +3453,7 @@ void KMMainWidget::setupActions()
actionCollection()->setDefaultShortcut(action, QKeySequence(Qt::Key_C)); actionCollection()->setDefaultShortcut(action, QKeySequence(Qt::Key_C));
} }
{ {
QAction *action = new QAction(i18n("Jump to Folder..."), this); QAction *action = new QAction(QIcon::fromTheme(QStringLiteral("go-jump")), i18n("Jump to Folder..."), this);
actionCollection()->addAction(QStringLiteral("jump_to_folder"), action); actionCollection()->addAction(QStringLiteral("jump_to_folder"), action);
connect(action, &QAction::triggered, connect(action, &QAction::triggered,
this, &KMMainWidget::slotJumpToFolder); this, &KMMainWidget::slotJumpToFolder);
......
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