Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

If I bind a command to <Ctrl-v>, I need to press Ctrl-v twice to invoke the command #314

Closed
clarkgrubb opened this issue Jul 24, 2014 · 3 comments
Labels
component:terminal Ncurses and terminal status:fixed-proposed Fix proposed
Milestone

Comments

@clarkgrubb
Copy link

I put this in my .tigrc

bind generic <Ctrl-v> move-page-down

But for some reason I need to press Ctrl-v twice to page down. If I bind move-page-down to <Ctrl-a>, I don't need to press Ctrl-a twice. I looked thru the online tig help, and I don't see any bindings for <Ctrl-v>. I am running this in Terminal.app with bash. Other curses apps seem to be able to bind Ctrl-v to commands, so it doesn't appear to be my environment.

@jonas
Copy link
Owner

jonas commented Jul 31, 2014

I can reproduce this. If your other curses apps do not have this behaviour I wonder if something is missing when initializing the terminal. In bash, Ctrl-v "means" print the following keybinding verbatim so I wonder if somehow this is a terminal setting that gets carried over.

Needs to be further investigated.

@jonas
Copy link
Owner

jonas commented Oct 3, 2014

OK, found a way to fix this. First verify that lnext is defined to ^V

> stty -a | grep lnext
    eol2 = <undef>; erase = ^?; intr = ^C; kill = ^U; lnext = ^V;

Then the actual fix is to "unbind" Ctrl-v to echo the next key combo using:

stty lnext undef

@jonas jonas added the status:fixed-proposed Fix proposed label Oct 3, 2014
@jonas
Copy link
Owner

jonas commented Jul 10, 2017

Fixed by @rolandwalker in #619

@jonas jonas closed this as completed Jul 10, 2017
@jonas jonas added this to the tig-2.3 milestone Jul 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:terminal Ncurses and terminal status:fixed-proposed Fix proposed
Projects
None yet
Development

No branches or pull requests

2 participants