<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jan 30, 2019 at 4:10 PM Kurt Hindenburg <<a href="mailto:noreply@phabricator.kde.org">noreply@phabricator.kde.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><table><tbody><tr><td>hindenburg added a comment.
</td><td><a style="text-decoration:none;padding:4px 8px;margin:0px 8px 8px;float:right;color:rgb(70,76,92);font-weight:bold;border-radius:3px;background-color:rgb(247,247,249);background-image:linear-gradient(rgb(255,255,255),rgb(241,240,241));display:inline-block;border:1px solid rgba(71,87,120,0.2)" href="https://phabricator.kde.org/D18154" target="_blank">View Revision</a></td></tr></tbody></table><br><div><div><p>A quick look running and I'm not a big fan: </p></div></div></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div><p>1) It doesn't handle split views (assume that after the split tab changes go in, this could be fix)</p></div></div></blockquote><div>This should work (at least I remember to test this. I'll test again) <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div><p>, 2) if only tab, don't show anything, </p></div></div></blockquote><div>Easy to fix. <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div><p>3) releasing the Ctrl+Tab should close popup and go to selected</p></div></div></blockquote><div>Easy to fix. <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div><p> 4) I'm tempted to have a way to turn off the popup.</p></div></div></blockquote><div> easy to add but we should remember that kde used to be a monster for having options for everything. So I'd consider this only if really needeed.</div><div>(this is also the default behavior of kate and a few other multi-view apps that kde has.)</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div>

<p>Since other people seem to like it, let me use/review it more.</p></div></div><br><div><b>REPOSITORY</b><div><div>R319 Konsole</div></div></div><br><div><b>REVISION DETAIL</b><div><a href="https://phabricator.kde.org/D18154" target="_blank">https://phabricator.kde.org/D18154</a></div></div><br><div><b>To: </b>tcanabrava, hindenburg, Konsole<br><b>Cc: </b>ngraham, thsurrel, lucasew, mart, konsole-devel, maciejn, maximilianocuria, hindenburg<br></div></blockquote></div></div>