[Konsole-devel] Review Request: Add 'Select All' action to select all output in the session

Jekyll Wu adaptee at gmail.com
Fri Aug 26 14:42:17 UTC 2011


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

(Updated Aug. 26, 2011, 2:42 p.m.)


Review request for Konsole.


Changes
-------

recreate against master, and increase the version in sessionui.rc


Summary
-------

I think this feature is quite handy in a few situations. For example, execute a couple of commands in a new tab then paste the whole output as example in a tutorial or forum post. It is not difficult to implement and gnome-terminal & xfce4-terminal both provide it, so why not?

The action use Ctrl+Shift+S as default shortcut. My first thought is using Ctrl+Shift+A, but that is already used by 'Moninor activity' action.  Actually, neither of them are very convenient to press. Any idea for better default shortcut?


This addresses bug 199990.
    http://bugs.kde.org/show_bug.cgi?id=199990


Diffs (updated)
-----

  desktop/sessionui.rc e9894da 
  src/ScreenWindow.h 51adb42 
  src/ScreenWindow.cpp 75ea689 
  src/SessionController.h 8ae33cf 
  src/SessionController.cpp fbb8531 

Diff: http://git.reviewboard.kde.org/r/102372/diff


Testing
-------

It works quite well when the first screen is in use(shell, of course).

However, when the second screen is in use(less,vim,nano,mc,etc), it does not work very well. I can still use it to select all text in the screen, but copying them will return garbage result. 

Maybe this action should be disabled when the second screen is used. That requires extra code in Emulation.cpp to emit some signal when the screen is switched.


Thanks,

Jekyll

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


More information about the konsole-devel mailing list