asadkamran / connectbot

Automatically exported from code.google.com/p/connectbot
Apache License 2.0
0 stars 0 forks source link

Patch: Fix broken arrow keys to move "selecting for copy" cursor #382

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
The hardware keyboard "selecting for copy" code is broken in ConnectBot 1.7.1 
and earlier (not sure exactly when it happened).  Basically, the arrow keys no 
longer move the selection cursor, so text must be selected with the touch 
screen which is difficult on tiny-font terminals.

Attached is a patch that restores the arrow key functionality.

As a bonus, it also moves the initial selection cursor from the top-left corner 
to the same location as the terminal cursor.  This is the same behavior as 
screen's copy mode, and is generally more useful as the text often desired to 
copy is near the terminal cursor.

Original issue reported on code.google.com by mkas...@gmail.com on 26 Oct 2010 at 9:07

Attachments:

GoogleCodeExporter commented 8 years ago
confirmed on motorola milestone.

version 1.7-dev(r484 2010.03.07) is the last one where the Dpad works for 
selecting/copying on my phone

Original comment by bento...@gmail.com on 11 Mar 2011 at 1:55