NikitaIvanovV / ctpv

Image previews for lf file manager
https://www.nikitaivanov.com/man1/ctpv
MIT License
264 stars 24 forks source link

Renaming new file with old file show old file preview #45

Open harishnkr opened 1 year ago

harishnkr commented 1 year ago

https://user-images.githubusercontent.com/48408097/224785670-8ea269b1-7483-4f2e-9a69-0fb788b69f52.mp4

NikitaIvanovV commented 1 year ago

Thanks for opening the issue! Fixed in fbf7f93. Could you please test it?

harishnkr commented 1 year ago

No, I tested and it still shows the same error. What other details should I describe, if it helps?

Kitty terminal, v 0.27.1 lf version 28

NikitaIvanovV commented 1 year ago

I just realized that you are viewing text files on the video. Could you please do the same thing with some image files and check if previews are correct? I suspect that it's an issue on lf side because it caches text previews (while I handle caching for image previews).

harishnkr commented 1 year ago

I tested it on image preview and it works fine for images. I think you may be right with regards to this problem being lf caching. Thanks again.