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

Binding meta-O causes arrow keys to stop working #430

Closed
ahihi opened this Issue Feb 29, 2016 · 9 comments

Comments

Projects
None yet
6 participants
@ahihi

ahihi commented Feb 29, 2016

I am using /bind meta-O change_window 49. In irssi 0.8.17 and earlier this works fine, but in 0.8.18 my arrow keys stop functioning normally — they insert ABCD characters instead.

A fresh config plus this binding suffices to reproduce the issue.

@ailin-nemui

This comment has been minimized.

Contributor

ailin-nemui commented Mar 2, 2016

that's a technical limitation on the one hand. Irssi 0.8.18 now enabled application keypad which makes your arrow keys send SS3-{A,B,C,D} (a change from the previous CSI) SS3 is one of the common control sequence initiators and translates to Esc, O (which is the same as your Alt+O) (CSI=Esc,[)

for now I would suggest the following work-around in your startup file and being aware that there are two different control sequence styles (of which one will occupy Alt+Shift+O):

script exec print STDERR `tput rmkx`

Another way to work around this issue is to disable application cursor key mode in your terminal emulator, if it offers that setting.
Finally, you can easily edit your terminfo file (infocmp -x >my.ti; $EDITOR my.ti; tic -x my.ti) to revert this behaviour, by deleting the smkx and rmkx codes. I suggest modifying any k... entry by changing \EO to \E[ in that case, to make other full screen applications using smkx aware of the now different key codes the terminal will produce after that change.

also note that meta-O is bound to meta2 in the default config for this reason, and has always been even before

Further reading:

@LemonBoy b5b73cb is causing this regression, what are the pros and cons of KX? maybe it's better to stick to non-kx?

@stintel

This comment has been minimized.

stintel commented Mar 14, 2016

Running into this issue as well. In fact, everybody using https://github.com/irssi/scripts.irssi.org/blob/gh-pages/scripts/keybinds.pl has "BIND meta-O change_window 39". Thanks for the workarounds, but as this is a regression (and quite the annoying one), I would like to see this fixed.

@LemonBoy

This comment has been minimized.

Member

LemonBoy commented Mar 14, 2016

I am able to bind whatever command I want to meta-o, I think the problem lies in your terminals (or fingers)

@dequis

This comment has been minimized.

Member

dequis commented Mar 14, 2016

@LemonBoy uppercase O, not lowercase

@LemonBoy

This comment has been minimized.

Member

LemonBoy commented Mar 14, 2016

Both meta-o and meta-O work just well in urxvt and xterm

@ailin-nemui

This comment has been minimized.

Contributor

ailin-nemui commented Mar 14, 2016

but it conflicts with up-arrow (now) because that also starts with meta-O

@trou

This comment has been minimized.

trou commented Mar 15, 2016

Chiming in : I'm bitten by the bug and I'd like it to be fixed.

@dequis dequis removed the cantfix label Mar 18, 2016

@dequis dequis added this to the 0.8.18a milestone Mar 18, 2016

ailin-nemui added a commit to ailin-nemui/irssi that referenced this issue Mar 22, 2016

Make use of terminal application keys configurable
adds a new setting term_appkey_mode which can enable or disable the use
of keyboard transmit (application keys) mode. Fixes irssi#430
@ailin-nemui

This comment has been minimized.

Contributor

ailin-nemui commented Mar 22, 2016

since there were very vocal people about meta-O I expect that @stintel @rhonda @trou and @ahf test #459 otherwise I don't see the point of adding that into 0.8.18a

ailin-nemui added a commit to ailin-nemui/irssi that referenced this issue Mar 22, 2016

Make use of terminal application keys configurable
adds a new setting term_appkey_mode which can enable or disable the use
of keyboard transmit (application keys) mode. Fixes irssi#430

ailin-nemui added a commit to ailin-nemui/irssi that referenced this issue Mar 22, 2016

Make use of terminal application keys configurable
adds a new setting term_appkey_mode which can enable or disable the use
of keyboard transmit (application keys) mode. Fixes irssi#430

ailin-nemui added a commit to ailin-nemui/irssi that referenced this issue Mar 22, 2016

Make use of terminal application keys configurable
adds a new setting term_appkey_mode which can enable or disable the use
of keyboard transmit (application keys) mode. Fixes irssi#430
@stintel

This comment has been minimized.

stintel commented Mar 23, 2016

After "/set term_appkey_mode off" with changes from #459 I no longer have this issue. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment