[kde-doc-english] [trojita] src/Gui: GUI: add an icon for attachment download

Jan Kundrát jkt at flaska.net
Fri Aug 9 16:30:12 UTC 2013


Git commit f143b7bb10d3adb49682038b6d97f9a72919ddf7 by Jan Kundrát.
Committed on 07/08/2013 at 15:21.
Pushed by jkt into branch 'master'.

GUI: add an icon for attachment download

I'm not sure whether "document-save-as" is better than "download", but at least
on my icon theme, "download" shows a very unintuitive icon while
"document-save-as" shows something which I recognize. Comments welcome.

REVIEW: 111936

M  +2    -1    src/Gui/AttachmentView.cpp

http://commits.kde.org/trojita/f143b7bb10d3adb49682038b6d97f9a72919ddf7

diff --git a/src/Gui/AttachmentView.cpp b/src/Gui/AttachmentView.cpp
index 7e1dff0..8753e00 100644
--- a/src/Gui/AttachmentView.cpp
+++ b/src/Gui/AttachmentView.cpp
@@ -99,9 +99,10 @@ AttachmentView::AttachmentView(QWidget *parent, Imap::Network::MsgPartNetAccessM
     m_downloadButton = new QToolButton();
     m_downloadButton->setPopupMode(QToolButton::MenuButtonPopup);
     m_downloadButton->setSizePolicy(QSizePolicy::Fixed, QSizePolicy::Fixed);
+    m_downloadButton->setToolButtonStyle(Qt::ToolButtonTextBesideIcon);
 
     QMenu *menu = new QMenu(this);
-    m_downloadAttachment = menu->addAction(tr("Download"));
+    m_downloadAttachment = menu->addAction(loadIcon(QLatin1String("document-save-as")), tr("Download"));
     m_openAttachment = menu->addAction(tr("Open Directly"));
     connect(m_downloadAttachment, SIGNAL(triggered()), this, SLOT(slotDownloadAttachment()));
     connect(m_openAttachment, SIGNAL(triggered()), this, SLOT(slotOpenAttachment()));



More information about the kde-doc-english mailing list