<table><tr><td style="">marten created this revision.<br />marten added a reviewer: KDE PIM.<br />Herald added a project: KDE PIM.<br />Herald added a subscriber: kde-pim.<br />marten requested review of this revision.
</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/D28431">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This is the confirmation question that is shown when an attachment is opened.  Currently the button order is "Open With", "Open with <default application>", "Save as", "Cancel" where the "Open with <default application>" is only shown if there is a default application available for the MIME type.</p>

<p>If the attachment does have an default application then the most likely action for the user would be to open it in that application.  Therefore it would be more useful to have the button to do this as the leftmost one - being first in the viewing order and more easily clickable.  So the button order would become "Open with <default application>", "Open With", "Save as", "Cancel".  The "Save as" button would still be the default, for security.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Built messagelib with this change, open an attachment in KMail and observe that the button order is as proposed above.</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/D28431">https://phabricator.kde.org/D28431</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>messageviewer/src/widgets/attachmentdialog.cpp</div></div></div><br /><div><strong>To: </strong>marten, KDE PIM<br /><strong>Cc: </strong>kde-pim, fbampaloukas, dcaliste, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil<br /></div>