lsix icon indicating copy to clipboard operation
lsix copied to clipboard

add darktile support

Open SirCipherz opened this issue 4 years ago • 3 comments

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

SirCipherz avatar Aug 09 '21 10:08 SirCipherz

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

hackerb9 avatar Aug 09 '21 19:08 hackerb9

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.

SirCipherz avatar Aug 10 '21 18:08 SirCipherz

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.

hackerb9 avatar Aug 20 '21 06:08 hackerb9