<div><div dir="auto">As usual thanks for the really good review.</div></div><div dir="auto">I’ll address the Points tomorrow as today I’m traveling.</div><div dir="auto"><br></div><div dir="auto">The detach tabs I believe that it takes the size of the terminal display; probably it’s something that’s broken with the current behavior already.</div><div dir="auto"><br></div><div dir="auto">The point two seems easy to fix.</div><div dir="auto">And I believe the fix for point three is the same for point two.</div><div dir="auto"><br></div><div dir="auto">I need to focus the newly created tab for point 3. And I need to focus the current tab for point 2.</div><div><br><div class="gmail_quote"><div dir="ltr">Em sáb, 8 de dez de 2018 às 18:16, Kurt Hindenburg <<a href="mailto:noreply@phabricator.kde.org">noreply@phabricator.kde.org</a>> escreveu:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><table><tbody><tr><td>hindenburg added a comment.
</td><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/D17267" target="_blank">View Revision</a></td></tr></tbody></table><br><div><div><p>I've noticed a few issues</p>
<ol class="m_3921916859132003530remarkup-list">
<li class="m_3921916859132003530remarkup-list-item">detach split tabs maintain the same size (ie not the full window) but the size the tab was - is this expected behavior?</li>
<li class="m_3921916859132003530remarkup-list-item">close one side of the view causing the menus to not have all the menus until switching tabs</li>
<li class="m_3921916859132003530remarkup-list-item">opening a new tab on the non-focus side will not update the tab text (ie should be "src:zsh", it is ":" until you click on tab).</li>
</ol></div></div><br><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br><div><strong>BRANCH</strong><div><div>dontDuplicateTabs</div></div></div><br><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17267" target="_blank">https://phabricator.kde.org/D17267</a></div></div><br><div><strong>To: </strong>tcanabrava, thsurrel, hindenburg, gennad, ngraham, Konsole<br><strong>Cc: </strong>ngraham, hindenburg, gennad, konsole-devel, thsurrel, maximilianocuria<br></div></blockquote></div></div>