<table><tr><td style="">marten created this revision.<br />marten added a reviewer: KDE PIM.<br />Restricted Application added a project: KDE PIM.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D8162" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>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.</p>

<p>The dialogue text is converted to KUIT markup.</p>

<p>This requires passing the service 'offer' to AttachmentDialog::AttachmentDialog(), instead of the application name.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Built messagelib with this change, checked for correct operation and appearance of the dialogue in KMail.</p>

<p><a href="https://phabricator.kde.org/F4987121" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">F4987121: kmail5-attachmentopen-Dx.png</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R94 PIM: Message Library</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8162" rel="noreferrer">https://phabricator.kde.org/D8162</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>messageviewer/src/viewer/viewer_p.cpp<br />
messageviewer/src/widgets/attachmentdialog.cpp<br />
messageviewer/src/widgets/attachmentdialog.h</div></div></div><br /><div><strong>To: </strong>marten, KDE PIM<br /><strong>Cc: </strong>dvasin, winterz, vkrause, mlaurent, knauss, dvratil<br /></div>