[Konsole-devel] Review Request: Add new action "Clone Tab" dedicated for cloning tab/session
Jekyll Wu
adaptee at gmail.com
Sat Feb 4 16:51:56 UTC 2012
> On Feb. 4, 2012, 3:08 p.m., Kurt Hindenburg wrote:
> > "Clone" to me would be take the tab and making an exact clone of it. This doesn't seem to do that. For example, if you change any temp parameters, they are not 'cloned'. I tested changing via qdbus and then cloning that tab.
> > If we do 'clone' it has to be separated from the 'Close Tab'.
The code should be able clone those temporary changes made to the profile associate to the current tab/session(examples are those command line options: -e, -p ). As for the temporary changes made through dbus methods, they take effect directly on the session without touch profile. On the other hand, the code should work with the 'konsoleprofile' script since that script influence current tab/session through modifying/changing its associated profile.
- Jekyll
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103859/#review10335
-----------------------------------------------------------
On Feb. 3, 2012, 6:22 p.m., Jekyll Wu wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103859/
> -----------------------------------------------------------
>
> (Updated Feb. 3, 2012, 6:22 p.m.)
>
>
> Review request for Konsole.
>
>
> Description
> -------
>
> Since "New Tab" always uses default profile now, It might be helpful to add this new action dedicated for the previous handy but potentially confusing functionality.
>
> The code is not complex. My main concern is whether it should be an action only accessible through keyboard shortcut or also accesible through menu. If accessible through menu, then the name "Clone Tab" is a bit problematic, since it is very similar to the name "Close Tab".
>
> "Duplicate Tab" might be another choice of name, but I don't think it is as clear as "Clone Tab".
>
>
> This addresses bugs 254976 and 292518.
> http://bugs.kde.org/show_bug.cgi?id=254976
> http://bugs.kde.org/show_bug.cgi?id=292518
>
>
> Diffs
> -----
>
> desktop/konsoleui.rc ace6bee
> src/MainWindow.h b3dd218
> src/MainWindow.cpp ab8bba6
>
> Diff: http://git.reviewboard.kde.org/r/103859/diff/diff
>
>
> Testing
> -------
>
>
> Thanks,
>
> Jekyll Wu
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20120204/e6850e00/attachment.html>
More information about the konsole-devel
mailing list