<table><tr><td style="">ngraham created this revision.<br />ngraham added a reviewer: Frameworks.<br />Restricted Application added a project: Frameworks.<br />ngraham 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/D12239">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p><a href="https://phabricator.kde.org/D12227" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">D12227</a> fixes 209559 for the case when the user presses the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Cancel</span></span></span> button, but not when the dialog is closed via the Escape key or the titlebar's close button. In order to do that, changes to the QPA plugin are needed. To facilitate it, <tt style="background: #ebebeb; font-size: 13px;">KFileWidget</tt> needs a public interface to the <tt style="background: #ebebeb; font-size: 13px;">writeViewConfig()</tt> function needs so that the dialog can call it. THis patch does that.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Compile and deploy KIO, run tests (no regressions), interact with a file open/save dialog; all still work</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>BRANCH</strong><div><div>public-writeviewconfig-function</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12239">https://phabricator.kde.org/D12239</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/filewidgets/kfilewidget.cpp<br />
src/filewidgets/kfilewidget.h</div></div></div><br /><div><strong>To: </strong>ngraham, Frameworks<br /><strong>Cc: </strong>michaelh, ngraham, bruns<br /></div>