<table><tr><td style="">tcanabrava edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-nhkd7odbo3c2tql/">(Show Details)</a><br />tcanabrava edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-s6hhflhtvh44uwz/">(Show Details)</a><br />tcanabrava added a reviewer: aacid.
</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/D14595">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>understand than the old one, and it also fixes <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">a</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">around 8</span> crash<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">es</span>.<br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">The bad thing is that it's ignoring when you have multiple tabs in the same window, it's always adding it on the first one that you have,<br />
Also it's always adding them in the last position of the tab, I'll update this review as soon as I have those features fixed.<br />
But please take a look, it already works for the majority of cases</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Ready to Review</span>.</div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;">Open konsole, create tabs, drag them, create new windows, reatach them<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">.</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">, split the window in two, drag & drop between the split views.<br />
</span></div></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/D14595">https://phabricator.kde.org/D14595</a></div></div><br /><div><strong>To: </strong>tcanabrava, Konsole, hindenburg, sandsmark, ngraham, aacid<br /><strong>Cc: </strong>konsole-devel, herrold, ngraham, maximilianocuria, hindenburg<br /></div>