<table><tr><td style="">ngraham 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><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D12538#255184" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D12538#255184</a>, <a href="https://phabricator.kde.org/p/ltoscano/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ltoscano</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D12538#255183" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D12538#255183</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ngraham</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D12538#255101" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D12538#255101</a>, <a href="https://phabricator.kde.org/p/ltoscano/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ltoscano</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Does it work if a newer KIO is used with an older version of those applications?</p></div>
</blockquote>

<p>In Kate's case, it overrides the default save dialog to not handle overwrite, just using a newer KIO wouldn't help and we would definitely want to ship that fix before rolling out the KIO change.</p></div>
</blockquote>

<p>If I understand it correctly if a the current Kate (or other applications handling the overwrite case) is used with KIO patched with this change, then it will still work because Kate will handle this feature as before.</p></div>
</blockquote>

<p>Correct. It will just be a bit annoying in the case that you didn't mean to overwrite, because the "are you sure" dialog will only appear <em>after</em>the save dialog has disappeared, so you'll need to open it again.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>So a patched KIO should be released before Kate and then Kate should be fixed and depend on this newer version. Is all of this correct?</p></blockquote>

<p>My preference would be to release a patched Kate first, if we think that's appropriate for an 18.04.1 point release. Here are our options, along with what will happen for each case:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item"><strong>Release patched KIO before patched Kate</strong>: Double-clicking on a file triggers overwrite in Kate itself, so the dialog will have annoyingly gone away</li>
<li class="remarkup-list-item"><strong>Release patched Kate before patched KIO</strong>: No drawbacks, but users don't get the desirable prompt-overwrite-from-the-save-dialog-on-double-click behavior</li>
<li class="remarkup-list-item"><strong>Release both simultaneously</strong> (mostly impossible due to different release schedules): users-double-clicking on a file in Kate's <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> dialog get the prompt above the dialog, and canceling it humanely keeps the dialog open.</li>
</ul></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>