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

mediainfo.yazi (newer fork) no longer working? #40

Closed
xfzv opened this issue Jan 29, 2025 · 7 comments
Closed

mediainfo.yazi (newer fork) no longer working? #40

xfzv opened this issue Jan 29, 2025 · 7 comments

Comments

@xfzv
Copy link

xfzv commented Jan 29, 2025

Just curious if someone else is also experiencing this.

The original project was archived on 2024-12-20 and the creator suggests using this newer fork instead (which I'm using).

It used to work fine until not too long ago but now, nothing is displayed in the preview panel when hovering an audio, video or image file.

Issues aren't enabled on the main repo so I thought I would ask here.

I'm using Linux and Yazi git:

% yazi -V
Yazi 0.4.3 (b6cb1fa8 2025-01-29)

Can reproduce with a minimal config:

~/.config/yazi
├── plugins
│   └── mediainfo.yazi
│       └── main.lua
└── yazi.toml

~/.config/yazi/yazi.toml content:

[plugin]
prepend_previewers = [
  { mime = "{image,audio,video}/*", run = "mediainfo" },
  { mime = "application/subrip", run = "mediainfo" },
]

@boydaihungst

@boydaihungst
Copy link
Contributor

boydaihungst commented Jan 29, 2025

I guess this PR makes the plugin no longer work. I will create a branch for nightly with the fix later. Also, I didn’t notice that Ape archived his repo. 🥲
sxyazi/yazi#2253

@xfzv
Copy link
Author

xfzv commented Jan 29, 2025

@boydaihungst Thanks for the heads up and thanks in advance for the upcoming fix!

@boydaihungst
Copy link
Contributor

Because ya pack -a still doesn't support branch selection, I had to create a nightly repo for it. Please reinstall and update your configuration according to the README. @xfzv
https://github.com/boydaihungst/mediainfo-nightly.yazi

@AnirudhG07
Copy link
Owner

Maybe I should add this to mediainfo info? Lol. It's better to have the non-archived active repo forks for the same. I did this for eza-preview as well.

@boydaihungst
Copy link
Contributor

Feel free to add my plugins/forks. 👍

@AnirudhG07
Copy link
Owner

I believe I can close this issue if things are resolved??

@xfzv
Copy link
Author

xfzv commented Jan 30, 2025

Yes, all good after switching to mediainfo-nightly.yazi.

Thanks!

@xfzv xfzv closed this as completed Jan 30, 2025
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