search

hackerb9 / lsix

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

add darktile support #45

Open SirCipherz opened 2 years ago

SirCipherz commented 2 years ago

i added support for https://github.com/liamg/darktile

hackerb9 commented 2 years ago

Would you be willing to file a bug report with the darktile project to let them know it is not working with lsix? Thanks!

SirCipherz commented 2 years ago

thanks for your review hackerb9, i removed the creation of the code file which only was a debug file, the method the project is currently using for yaft doesn't work because the TERM variable for darktile is xterm-256colors, the method i'm using is the best i found yet.

hackerb9 commented 2 years ago

thanks for your review hackerb9, i removed the creation of the code file which only was a debug file, the method the project is currently using for yaft doesn't work because the TERM variable for darktile is xterm-256colors, the method i'm using is the best i found yet.

Unfortunately, it seems like this is something the authors of darktile will have to address. Have you filed a bug report with that project? If not, please do so so that programs like lsix can automatically detect that darktile has sixel support.

In the meantime, here is an ugly environment variable kludge based on your method that may help you. You can add it to your login script (e.g., .bash_profile):

if [[ "$(ps -o comm= -p "$(($(ps -o ppid= -p "$(($(ps -o sid= -p "$$")))")))")" == darktile ]]; then
   export LSIX_FORCE_SIXEL_SUPPORT=yuck
fi

It's not the right solution, but you can try it if you don't get any response from the darktile developers.