[Konsole-devel] Review Request: split view: automatically close the split area if it becomes empty.

[Kurt Hindenburg]

> On May 4, 2012, 3:08 p.m., Kurt Hindenburg wrote:
> > With this patch and following the steps in bko, I get this assert when I finish the d-n-d.
> > 
> > ASSERT: "containers().contains(container)" in file /mnt/kde/kde4/trunk/src/konsole/src/ViewSplitter.cpp, line 126
> >
> 
> Francesco Cecconi wrote:
>     Thanks for the test Kurt.

Francesco, have you had anytime to look at this further?


- Kurt


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104717/#review13430
-----------------------------------------------------------


On April 25, 2012, 9:02 a.m., Francesco Cecconi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104717/
> -----------------------------------------------------------
> 
> (Updated April 25, 2012, 9:02 a.m.)
> 
> 
> Review request for Konsole.
> 
> 
> Description
> -------
> 
> automatically close blank split, after that last tab is moved.
> 
> 
> This addresses bug 275771.
>     http://bugs.kde.org/show_bug.cgi?id=275771
> 
> 
> Diffs
> -----
> 
>   src/ViewContainer.h 605a4fa 
>   src/ViewContainer.cpp a0963fb 
>   src/ViewManager.h 0055dbf 
>   src/ViewManager.cpp 4885c86 
> 
> Diff: http://git.reviewboard.kde.org/r/104717/diff/
> 
> 
> Testing
> -------
> 
> - one tab + split + one tab -> move split right/left, no blank area
> - from bug: after step 4 no blank area
> 
> seems to work well but more test are necessary. ;)
> 
> 
> Thanks,
> 
> Francesco Cecconi
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20120618/a21c3809/attachment.html>