Skip to content
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

[Bug] cover is low res on tmux #189

Closed
amigthea opened this issue Oct 29, 2024 · 9 comments
Closed

[Bug] cover is low res on tmux #189

amigthea opened this issue Oct 29, 2024 · 9 comments

Comments

@amigthea
Copy link

I know this is probably on tmux side, but when using kew on tmux all album cover are displayed low res like this

Screenshot_20241029_090156

is there a known workaround?

@ravachol
Copy link
Owner

A terminal needs sixel support to enable images to look as they should.

tmux officially supports sixel when compiled with ./configure --enable-sixel

There are other terminals which support it as well. Listed here:
https://www.arewesixelyet.com/

@amigthea
Copy link
Author

correct me if I'm wrong, but shouldn't sixel be already supported if this command succede?

tmux

@ravachol
Copy link
Owner

Yeah it should! kew uses chafa to display images, so it might be related to how chafa handles tmux.

I got various bugs regarding the covers when using different terminals with tmux.

What is the terminal you are using tmux with?

@amigthea
Copy link
Author

thank you for the insight! I'm using foot

@ravachol
Copy link
Owner

@hpjansson Do you perhaps know what's going on here?

@amigthea
Copy link
Author

can confirm that's some interaction between tmux on foot and chafa, top command was executed in tmux, bottom on bare foot (pun intended)

image

@hpjansson
Copy link
Contributor

@hpjansson Do you perhaps know what's going on here?

I think so. tmux sixel support was merged in 3.4, so we can't enable it unconditionally everywhere. I added code in hpjansson/chafa#217 to detect the version and enable it conditionally - this will be in the next release, probably fixing this issue.

@ravachol ravachol closed this as completed Nov 1, 2024
@hpjansson
Copy link
Contributor

Release done: https://github.com/hpjansson/chafa/releases/tag/1.14.5

Share and enjoy!

@amigthea
Copy link
Author

amigthea commented Nov 3, 2024

Release done: https://github.com/hpjansson/chafa/releases/tag/1.14.5

Share and enjoy!

thank you for your work hpjansson, but unfortunately I still get a broken image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants