<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/D13882">View Revision</a></tr></table><br /><div><div><p>Awesome!</p>
<p>Here are the behavioral quirks that I found during testing:</p>
<ul class="remarkup-list">
<li class="remarkup-list-item">When drag-re-ordering tabs, the dragged tab visually stays on the bar now and only moves horizontally (yay!) but if you release the mouse outside of the tab bar, the dragged tab now detaches with no warning*</li>
<li class="remarkup-list-item">Tab context menu lost its <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">Close</span></span></span> menu item</li>
<li class="remarkup-list-item">When "Show new tab and close tab buttons" becomes checked after being unchecked, the space reserved for the <span><span class="phui-tag-view phui-tag-type-shade phui-tag-grey phui-tag-shade "><span class="phui-tag-core ">New Tab</span></span></span> button doesn't actually have that button in it until you quit and restart Konsole</li>
</ul>
<ul class="remarkup-list">
<li class="remarkup-list-item">It would also be nice if there was some sort of visual feedback for detaching a tab, and also a 30 or so pixel "dead zone" above and below the bar where releasing the mouse wouldn't detach the tab yet--but this is probably material for another patch. :)</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13882">https://phabricator.kde.org/D13882</a></div></div><br /><div><strong>To: </strong>tcanabrava, hindenburg, Konsole<br /><strong>Cc: </strong>ngraham, konsole-devel, herrold, maximilianocuria, hindenburg<br /></div>