<table><tr><td style="">rkflx added a comment.
</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/D12538">View Revision</a></tr></table><br /><div><div><p>TBH, I don't get what the overwrite and release schedule discussion has to do with this patch. All it does is add an additional method to trigger <tt style="background: #ebebeb; font-size: 13px;">slotOk()</tt>, i.e. instead of clicking on <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Save</span></span></span>, users can simply double-click.</p>
<p>The fact that either the dialog or the app ask before overwriting in response to the slot is not changed in any way here. Don't judge a Diff by its title, in particular as long as the request to "Please update your summary too" is still in state "red" ;)</p>
<hr class="remarkup-hr" />
<p>Regarding what you were discussing: If I understood Peter's analysis in <a href="https://phabricator.kde.org/D12346" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: line-through;">D12346</a> correctly, the API will always allow apps to opt-out of the dialog providing the overwrite confirmation, and instead add their own. Therefore to me it seems like it is only a matter of going through all apps and opt-in to the already existing functionality, i.e. I don't see why this would require changes in KIO which would result in dependency issues (unless I'm mistaken?).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12538">https://phabricator.kde.org/D12538</a></div></div><br /><div><strong>To: </strong>anemeth, Frameworks, VDG, ngraham, rkflx<br /><strong>Cc: </strong>ltoscano, rkflx, broulik, jtamate, ngraham, Frameworks, michaelh, bruns<br /></div>