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

Additional Terminal Support #17

Open
AlexAtkinson opened this issue Dec 2, 2022 · 1 comment
Open

Additional Terminal Support #17

AlexAtkinson opened this issue Dec 2, 2022 · 1 comment

Comments

@AlexAtkinson
Copy link

AlexAtkinson commented Dec 2, 2022

Please add support for the following terminal emulators. They support SIXEL.
https://www.arewesixelyet.com

Thanks

@tclift
Copy link

tclift commented Jan 28, 2025

It appears that Yakuake and Konsole are not supported because in response to "send device attributes" they send:

$ echo -e "\e[c"

^[[?62;1;4c

(Konsole source), where jplot expects 4 (not 4c) here (jqplot source).

If I change:

if len(t) == 1 && t[0] == '4' {

to:

if t[0] == '4' {

then indeed, jplot starts working in Yakuake.

Looking at the xterm docs on Sixel as a reference, I don't know what the significance of 4c is (vs 4). Was the len(t) == 1 added for a specific reason/fix? If not, maybe it can be removed.

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