D8162: Attachment open confirmation dialogue: Show icons on buttons

Jonathan Marten noreply at phabricator.kde.org
Fri Oct 6 18:54:38 BST 2017


marten created this revision.
marten added a reviewer: KDE PIM.
Restricted Application added a project: KDE PIM.

REVISION SUMMARY
  This change shows theme icons on the "Open With..." button (the standard "document-open" icon) and on the "Open with 'application'" button (the application's icon as defined in its service file).  These buttons previously had no icons, but the other two in the dialogue did so unbalancing the appearance.
  
  The dialogue text is converted to KUIT markup.
  
  This requires passing the service 'offer' to AttachmentDialog::AttachmentDialog(), instead of the application name.

TEST PLAN
  Built messagelib with this change, checked for correct operation and appearance of the dialogue in KMail.
  
  F4987121: kmail5-attachmentopen-Dx.png <https://phabricator.kde.org/F4987121>

REPOSITORY
  R94 PIM: Message Library

REVISION DETAIL
  https://phabricator.kde.org/D8162

AFFECTED FILES
  messageviewer/src/viewer/viewer_p.cpp
  messageviewer/src/widgets/attachmentdialog.cpp
  messageviewer/src/widgets/attachmentdialog.h

To: marten, #kde_pim
Cc: dvasin, winterz, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20171006/4d7c7457/attachment.html>


More information about the kde-pim mailing list