-
-
Notifications
You must be signed in to change notification settings - Fork 121
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
No sixels in st terminal emulator with sixel patch #1783
Comments
So I looked into this yesterday, and am not sure it's possible to detect. #1773 basically, st doesn't allow itself to be discovered via answering any of the standard queries, and it does not define any of its own. it also fails to respond to any of the standard sixel queries =[ |
i wanted to file an issue against the patch, but the repo doesn't seem to accept PRs or even issues =[ this is why lsix is denying that it supports sixel, as well. |
i can't use |
I guess sixel support is not in the standard st cause of the suckless philosophy. People that want such a thing have to apply patches themselves. So I guess the options here are:
Maybe I'm missing something. Patching st would not be that much of a problem; I'm not bound to that specific st fork — it just had a sixel patch already applied and it did not get rid of older sixel images (in contrast to xterm or mlterm). |
Okay so through hackerb9/lsix#32 I found bakkeby/st-flexipatch#7. A few things regarding the situation with st and sixel are explained in both issues and it seems like there was even some experimenting with sixel on the standard st last year.. This st-flexipatch is actually a up-to-date st fork with an updated sixel patch for the current st version 0.8.4 (tho it seems to have a problem that sixels won't vanish after
so it doesn't seem like there is a difference regarding this. If I interpret this commit correctly then they actually implemented those needed sequences that notcurses could check: bakkeby/st-flexipatch@8f79391 |
did you try it out with this? if it responds to an try running |
At this point I only enabled the sixel and some of the scrollback patches. I guess I could enable the alpha patch to get rid of at least |
yeah the " didn't detect bitmap graphics support" means it didn't get an answer to the necessary queries =[ thanks for helping test. i'll come back to this as soon as i can, grab the code you pointed to, and see what exactly is missing, if you think they'll take a patch. |
ok, i think this ought work now! we interpret a DA1 with a 4 as advetising 256-creg Sixel in the absence of superior |
export | egrep 'LANG|LC_CTYPE|TERM'
notcurses-demo i
)When I execute notcurses-demo within st-sixel (st with sixel patch then it does not make use of sixel.
Tho even the popular lsix claims that the terminal emulator has no sixel support but it definitely has sixel support (my own https://github.com/schrmh/pdfgrepSIXEL works..).
The text was updated successfully, but these errors were encountered: