[Konsole-devel] Review Request 124739: Set tabs movable again
Kurt Hindenburg
kurt.hindenburg at gmail.com
Mon Aug 17 13:18:37 UTC 2015
> On Aug. 17, 2015, 12:25 a.m., Kurt Hindenburg wrote:
> > Whoa nice job! This fixes moving of tabs and also dragging a tab outside its window to create another window. Thanks a lot.
>
> David Edmundson wrote:
> to 15.08 too?
we missed the cutof for 15.08 by a couple of weeks
- Kurt
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124739/#review83906
-----------------------------------------------------------
On Aug. 17, 2015, 1:16 p.m., David Edmundson wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124739/
> -----------------------------------------------------------
>
> (Updated Aug. 17, 2015, 1:16 p.m.)
>
>
> Review request for Konsole.
>
>
> Repository: konsole
>
>
> Description
> -------
>
> KTabBar used to do internal drag & drop re-ordering
> Konsole massively expanded on this to do cross window drag & drop.
>
> Now Qt does tabbar reordering, but Konsole relied on KTabBar's signals
> to say when a drag start before overriding the rest.
>
> To solve this I've added the same drag start signal manually into
> Konsole's tab bar.
>
> BUG: 348057
>
>
> Diffs
> -----
>
> src/ViewContainer.cpp 02be309fbd22cf52ea963bbd11c0e589b1e5004a
> src/ViewContainerTabBar.h 0e2985fd53021edbc9f5fb58aeadd40c564728bf
> src/ViewContainerTabBar.cpp 399c391d619d23b34bb03be9f61417931f80ee51
>
> Diff: https://git.reviewboard.kde.org/r/124739/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> David Edmundson
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20150817/b172dd79/attachment.html>
More information about the konsole-devel
mailing list