[Konsole-devel] Review Request: Allow user to control where the new tab will be placed: after current tab, or at the end?

[Kurt Hindenburg]

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


Perhaps have a combo w/ an Enum and method newTabPlacement()?  Show new tabs after current tab - Show new tabs at end

- Kurt


On Sept. 23, 2011, 7:47 p.m., Jekyll Wu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102685/
> -----------------------------------------------------------
> 
> (Updated Sept. 23, 2011, 7:47 p.m.)
> 
> 
> Review request for Konsole.
> 
> 
> Summary
> -------
> 
> This is a follow up of my suggestion of putting new tab right after current tab(current behavior on master branch).
> 
> The involved logic is not complex. My main concern is the UI part. Is checkbox the right choice? I mean,  technically, "putNewTabAtEnd" and "putNewTabRightAfterCurrentTab" are not a boolean pair. So is combobox a better choice? Also I think the text can be improved. Suggestions are really needed.
> 
> 
> This addresses bug 276872.
>     http://bugs.kde.org/show_bug.cgi?id=276872
> 
> 
> Diffs
> -----
> 
>   src/EditProfileDialog.h 46f5a60 
>   src/EditProfileDialog.cpp 3a89296 
>   src/EditProfileDialog.ui 4d7d175 
>   src/Profile.h 7fd1f6c 
>   src/Profile.cpp 6254d35 
>   src/ViewManager.cpp 30d09b2 
> 
> Diff: http://git.reviewboard.kde.org/r/102685/diff
> 
> 
> Testing
> -------
> 
> No problem found yet.
> 
> 
> Screenshots
> -----------
> 
> new option in EditProfileDialog
>   http://git.reviewboard.kde.org/r/102685/s/269/
> 
> 
> Thanks,
> 
> Jekyll
> 
>

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