hackerb9 / lsix

Like "ls", but for images. Shows thumbnails in terminal using sixel graphics.
GNU General Public License v3.0
3.99k stars 129 forks source link

Can't detect sixel support in yaft #25

Closed hun10 closed 5 years ago

hun10 commented 5 years ago

yaft is a framebuffer terminal with sixel support: https://github.com/uobikiemukot/yaft

"convert foo.jpg -geometry 800x480 sixel:-" does work in yaft.

But lsix says "Error: Your terminal does not report having sixel graphics support."

hackerb9 commented 5 years ago

Thanks. Sounds like a bug in yaft. I'll let them know.

hackerb9 commented 5 years ago

Turns out the author of yaft thinks of it as a VT102 that just happens to also do sixel. Since the features needed to detect sixel support didn't exist until the the VT220, there's no way around it other than to simply check if TERM == yaft*. I've added that into the current version of lsix. Please try it again and let me know if it works.

By the way, thanks for reporting this bug. I actually hadn't known about yaft before and it seems pretty nifty.

hun10 commented 5 years ago

Thanks, this works for me.