<table><tr><td style="">aacid 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/D7714" rel="noreferrer">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/D7714#146279" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D7714#146279</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;" rel="noreferrer">@ngraham</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>So for the record, I agree with folks that our "are you sure you want to close all these tabs" dialogs could stand for a bit of usability polishing.</p>

<p>However, that's a larger task--one that I am willing to spearhead, but nonetheless outside the scope of this particular patch. So for now, I would like to re-work the patch to match Dolphin and Konsole, so that we at least achieve a measure of consistency across KDE apps. After that, we will start the process of improving these dialogs' usability, and once we have some agreement on what the final form should look like, we will change them all at once to preserve consistency.</p>

<p>With this approach, we do the work in two discrete phases, with less risk of getting nothing at all because of bikeshedding that prevents the initial patch from ever getting committed--which is what Albert is rightly worried about.</p>

<p>Does that sound like a plan?</p></div>
</blockquote>

<p>I thought we had established my patch (minus wording) was better than Dolphin/Konsole?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7714" rel="noreferrer">https://phabricator.kde.org/D7714</a></div></div><br /><div><strong>To: </strong>aacid, Okular, KDE Applications<br /><strong>Cc: </strong>alexeymin, ngraham, colomar, rkflx, Okular, aacid<br /></div>