<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/D7714" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Thanks for picking up old reviewboard requests. I like the feature in general, it is standard in a lot of applications today.</p>
<p>But to make it easier to use, users should not relearn the UI they are already familiar with. This is what Dolphin currently provides vs. the patch:</p>
<div class="remarkup-table-wrap"><table style="border-collapse: separate;
border-spacing: 1px;
background: #d3d3d3;
margin: 12px 0;">
<tr><td style="background: #ffffff;
padding: 3px 6px;"><a href="https://phabricator.kde.org/F3903699" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;" rel="noreferrer">F3903699: dolphin-tab-quit.png</a></td><td style="background: #ffffff;
padding: 3px 6px;"><a href="https://phabricator.kde.org/F3903701" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;" rel="noreferrer">F3903701: okular-tab-quit.png</a></td></tr>
<tr></tr>
</table></div>
<p>Besides the relearning aspect, I find the dialog in Dolphin preferable because of the following properties:</p>
<ul class="remarkup-list">
<li class="remarkup-list-item">Explicit actions on the buttons instead of ambiguous Yes/No which requires reading of the explanation.</li>
<li class="remarkup-list-item">Slightly better wording ("you have...open in this window" vs. "you have tabs.").</li>
<li class="remarkup-list-item">Warning icon instead of information icon.</li>
<li class="remarkup-list-item">Standard do-not-ask-again message instead of custom message (also, "when closing more than one tab" does not correspond directly to the "quit" action the user initiated and thus might be confusing, as you cannot close e.g. 4 out of 5 tabs).</li>
<li class="remarkup-list-item">Third button with the correct action for those of us hitting <kbd 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;">Ctrl</kbd><span class="kbd-join" style="padding: 0 4px; color: #92969D;">+</span><kbd 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;">Q</kbd> while we meant <kbd 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;">Ctrl</kbd><span class="kbd-join" style="padding: 0 4px; color: #92969D;">+</span><kbd 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;">W</kbd>.</li>
<li class="remarkup-list-item">(Okular's dialog title is better, though.)</li>
</ul>
<p>Thus, I would prefer something akin to the Dolphin dialog being implemented in Okular.</p>
<p>Lastly, Dolphin has a way to reset the warning status in the settings, is there any way to reset it in Okular too?</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<br /><strong>Cc: </strong>rkflx, Okular, aacid<br /></div>