thestinger / termite

Termite is obsoleted by Alacritty. Termite was a keyboard-centric VTE-based terminal, aimed at use within a window manager with tiling and/or tabbing support.
https://github.com/alacritty/alacritty
2.74k stars 241 forks source link

Revisit sixel support? #772

Closed cprussin closed 3 years ago

cprussin commented 3 years ago

This ticket re-opens #511 (and expands on #539)

It seems vte now has sixel support but some work must be done to expose it in terminals that use VTE.

PR inbound to add support behind a config option.

cprussin commented 3 years ago

ah, while implementing this I discovered that the sixel support has been reverted. However, in this revert, the public API functions (most importantly vte_terminal_set_enable_sixel) was turned into a stub. So it should be safe to implement and ship in preparation for vte to ship the filled stub.

So with that I'll still put in a PR to add the option & API call. Any objections? @jelly tagging you in for advice as I see you seem to be the most active recent maintainer, sorry if you're not the right person to ask.

cprussin commented 3 years ago

No longer an issue for me as I no longer use termite. Feel free to re-open if it's relevant to folks who do.