[Konsole-devel] Review Request: Add new action "Clone Tab" dedicated for cloning tab/session

Jekyll Wu adaptee at gmail.com
Fri Feb 3 18:21:23 UTC 2012


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

(Updated Feb. 3, 2012, 6:21 p.m.)


Review request for Konsole.


Changes
-------

Forgot to upload the patch :)


Description (updated)
-------

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 (updated)
-----

  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/20120203/5ed5ffbc/attachment.html>


More information about the konsole-devel mailing list