<table><tr><td style="">rthomsen created this revision.<br />rthomsen added a reviewer: elvisangelaccio.<br />rthomsen set the repository for this revision to rARK Ark.<br />rthomsen added a project: Ark.<br />Restricted Application added a subscriber: kde-utils-devel.
</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/D2868" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This diff sets the initial focus of the internal previewer to the Part widget and sets the autodefault property of the Close button to false. The latter prevents the Enter key from closing the dialog which allows the Part widget to handle the key press event instead.</p>

<p>Fixes two bugs:<br />
<a href="https://bugs.kde.org/show_bug.cgi?id=369401" class="remarkup-link" target="_blank" rel="noreferrer">Bug 369401</a><br />
<a href="https://bugs.kde.org/show_bug.cgi?id=369390" class="remarkup-link" target="_blank" rel="noreferrer">Bug 369390</a></p>

<p>Meant for 16.08 branch.</p></div></div><br /><div><strong>TEST PLAN</strong><div><ol class="remarkup-list">
<li class="remarkup-list-item">Open any archive and preview a file with internal previewer.</li>
<li class="remarkup-list-item">Check that the Part widget has focus.</li>
<li class="remarkup-list-item">Check that the Enter key doesn't close the dialog.</li>
</ol></div></div><br /><div><strong>REPOSITORY</strong><div><div>rARK Ark</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D2868" rel="noreferrer">https://phabricator.kde.org/D2868</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>part/arkviewer.cpp</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>rthomsen, elvisangelaccio<br /><strong>Cc: </strong>kde-utils-devel, tctara<br /></div>