<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/D12227">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Currently, for file Open & Save dialogs, any view settings you've changed are only remembered if you actually open or save the file; they're ignored if you cancel the dialog using 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. This patch fixes that, since there's no particular reason why a user should want to discard any view changes just because they decided not to open or save the file.</p>

<p>BUG: 209559<br />
FIXED-IN: 5.46</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Open a file open or save dialog, change the view, hit the cancel button, then open it again.</p>

<p>Note: this patch does <em>not</em> fix the case of the view not being saved when closing the dialog via the <kbd title="Escape" style="display: inline-block; min-width: 1em; padding: 4px 5px 5px; font-weight: normal; font-size: 0.8rem; text-align: center; text-decoration: none; line-height: 0.6rem; border-radius: 3px; box-shadow: inset 0 -1px 0 rgba(71, 87, 120, 0.08); user-select: none; background: #f7f7f7; border: 1px solid #C7CCD9;">⎋</kbd> key or the titlebar close button; those cases need to be separately handled in the QPA theme plugin, which actually creates the dialog window.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>BRANCH</strong><div><div>save-dialog-view-settings-on-cancel (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12227">https://phabricator.kde.org/D12227</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/filewidgets/kfilewidget.cpp</div></div></div><br /><div><strong>To: </strong>ngraham, Frameworks<br /><strong>Cc: </strong>michaelh, ngraham, bruns<br /></div>