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

Image preview does not work with TUI file managers. #19

Closed
keystroke3 opened this issue May 17, 2020 · 7 comments
Closed

Image preview does not work with TUI file managers. #19

keystroke3 opened this issue May 17, 2020 · 7 comments
Labels
lf Related to Lf (the file manager)

Comments

@keystroke3
Copy link

keystroke3 commented May 17, 2020

I am using Pistol to preview files in Lf but for some reason, it does not work with images, it only shows a blank preview window. In Ranger I use the default previewer script with previewer_image_method set to kitty. That works fine but when I switch to Pistol, the preview brakes. I am trying to add image previews to Lf like they are in Ranger, but it does not work. Calling Pistol from terminal with an image argument causes it to display the image without issues. In the pistol.conf file, I have set image/* kitty +kitten icat %s. It however, does not work in Lf or Ranger. I am using Kitty terminal. I have tried this on Alacritty, ST-manjaro and xfce4-terminal. Same results.

@doronbehar
Copy link
Owner

Hi, I won't have time to drill into this in any time soon, but just so that I'll be able to do it right away when the time comes, you wrote:

I have set image/* kitty +kitten icat %s

Just to make sure, you are not using Pistol 0.1 or 0.1.1 I assume? Because the syntax has changed, see the big banner at the README. I'm not saying it should work once you'll switch to the new version and the new syntax, I'm just saying that in general you should update.

@doronbehar
Copy link
Owner

Maybe it will work and this is the issue, IDK.

@keystroke3
Copy link
Author

I just made the correction. It still does not work. Please look into it when you get the time.

@doronbehar doronbehar added the lf Related to Lf (the file manager) label Jun 5, 2020
@duganchen
Copy link
Contributor

@keystroke3 I don't know if you were using the FISH shell, but there's actually been a known incompatibility between FISH and Ranger, affecting image previews:

ranger/ranger#1919

@keystroke3
Copy link
Author

I use zsh.

@kofm
Copy link

kofm commented Apr 22, 2022

Same here
The only way to get it sorta working is using viu with the -b option (force block output) but the output is far from ideal
If i use kitty's icat lf stops working

All work fine outside lf

@doronbehar
Copy link
Owner

Closing in favor of #2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lf Related to Lf (the file manager)
Projects
None yet
Development

No branches or pull requests

4 participants