Commit 009a3bb3 authored by David Redondo's avatar David Redondo 🏎
Browse files

Use "mail-mark-important" icon consistently

Summary:
KMail already uses this icon in it's menu and context menu. For other statuses
 we  already use mail-* icons too. I think this also more correct since according
to the standard the emblem icons are "Icons for tags and properties of files,
 that are displayed in the file manager".
https://specifications.freedesktop.org/icon-naming-spec/icon-naming-spec-latest.html#context

Reviewers: #kde_pim, mlaurent, #vdg

Reviewed By: #kde_pim, mlaurent

Subscribers: kde-pim

Tags: #kde_pim

Differential Revision: https://phabricator.kde.org/D27507
parent c7283ca4
......@@ -666,7 +666,7 @@ void Manager::createDefaultThemes()
add_theme_simple_icon_column(s, i18n("Read/Unread"), QStringLiteral("mail-mark-unread-new"), Theme::ContentItem::ReadStateIcon, false, SortOrder::SortMessagesByUnreadStatus);
add_theme_simple_icon_column(s, i18n("Replied"), QStringLiteral("mail-replied"), Theme::ContentItem::RepliedStateIcon, false, SortOrder::NoMessageSorting);
add_theme_simple_icon_column(s, i18nc("Message importance indication", "Important"), QStringLiteral(
"emblem-important"), Theme::ContentItem::ImportantStateIcon, false, SortOrder::SortMessagesByImportantStatus);
"mail-mark-important"), Theme::ContentItem::ImportantStateIcon, false, SortOrder::SortMessagesByImportantStatus);
add_theme_simple_icon_column(s, i18n("Action Item"), QStringLiteral("mail-task"), Theme::ContentItem::ActionItemStateIcon, false, SortOrder::SortMessagesByActionItemStatus);
add_theme_simple_icon_column(s, i18n("Spam/Ham"), QStringLiteral("mail-mark-junk"), Theme::ContentItem::SpamHamStateIcon, false, SortOrder::NoMessageSorting);
add_theme_simple_icon_column(s, i18n("Watched/Ignored"), QStringLiteral("mail-thread-watch"), Theme::ContentItem::WatchedIgnoredStateIcon, false, SortOrder::NoMessageSorting);
......
......@@ -1263,7 +1263,7 @@ void Theme::populatePixmapCache() const
<< new QPixmap(QIcon::fromTheme(QStringLiteral("mail-mark-task")).pixmap(mIconSize, mIconSize))
<< new QPixmap(QIcon::fromTheme(QStringLiteral("mail-sent")).pixmap(mIconSize, mIconSize))
<< new QPixmap(QIcon::fromTheme(QStringLiteral("mail-forwarded")).pixmap(mIconSize, mIconSize))
<< new QPixmap(QIcon::fromTheme(QStringLiteral("emblem-important")).pixmap(mIconSize, mIconSize)) // "flag"
<< new QPixmap(QIcon::fromTheme(QStringLiteral("mail-mark-important")).pixmap(mIconSize, mIconSize)) // "flag"
<< new QPixmap(QIcon::fromTheme(QStringLiteral("messagelist/pics/mail-thread-watch.png")).pixmap(mIconSize, mIconSize))
<< new QPixmap(QIcon::fromTheme(QStringLiteral("messagelist/pics/mail-thread-ignored.png")).pixmap(mIconSize, mIconSize))
<< new QPixmap(QIcon::fromTheme(QStringLiteral("mail-mark-junk")).pixmap(mIconSize, mIconSize))
......
......@@ -210,7 +210,7 @@ void SearchLineStatus::createMenuSearch()
i18nc("@action:inmenu Status of a message", "Forwarded"),
Akonadi::MessageStatus::statusForwarded().toQInt32());
createFilterAction(QIcon::fromTheme(QStringLiteral("emblem-important")),
createFilterAction(QIcon::fromTheme(QStringLiteral("mail-mark-important")),
i18nc("@action:inmenu Status of a message", "Important"),
Akonadi::MessageStatus::statusImportant().toQInt32());
......
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