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

YAFT's terminfo reset should turn off sixel #45

Open
hackerb9 opened this issue Apr 27, 2019 · 1 comment
Open

YAFT's terminfo reset should turn off sixel #45

hackerb9 opened this issue Apr 27, 2019 · 1 comment
Assignees

Comments

@hackerb9
Copy link

Normally if one gets the terminal in a messed up state, a person can blindly type reset^J to fix the problem. This does not work in Yaft as it does not turn off sixel input when it gets a reset sequence.

For example, if someone accidentally writes a binary file to their screen and it contains the sequence ESC P, then YAFT will appear to lock up. Typing anything, even reset, has no effect.

One solution would be to add the ESC \ sequence to the terminfo reset string. However, it would be better if YAFT simply turned off Sixel input when it got the normal reset string.

By the way, in xterm, the clear screen escape sequence seems to also turn off sixel input. You can try this from the terminal:

echo $'\eP'
# now terminal appears frozen
clear             # Or press control-L
# now terminal works again

It would be nice if Yaft worked similarly.

@uobikiemukot uobikiemukot self-assigned this Sep 24, 2019
@uobikiemukot
Copy link
Owner

You're right.
Your pull request #47 fixes this issue.
I'll merge!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants