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

[FEATURE REQUEST] - APNG and Open File diaolg improvment #28

Open
2 tasks done
amymor opened this issue Nov 17, 2023 · 1 comment
Open
2 tasks done

[FEATURE REQUEST] - APNG and Open File diaolg improvment #28

amymor opened this issue Nov 17, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@amymor
Copy link
Contributor

amymor commented Nov 17, 2023

Feature Description

Hi @SegoCode
Thanks for such great app, I noticed that mpv itself supports the APNG format, but the Open file dialog is limited to a few formats. I think adding an All Files (*.*) option to it would be beneficial for other formats like APNG.

Also, I think adding a lightweight image viewer like JPEGView for image formats would be good. It could be a user choice to use mpv or something like JPEGView for APNG and GIF. Here is the difference:
Screenshot (153)

Motivation

If someone prefers to use the CPU instead of the GPU, they might find JPEGView more suitable. I believe this would be particularly beneficial for those involved in mining or rendering systems.

Alternatives

Instead of using the 'Browse' button (Open file dialog), paste the path to the APNG file into the 'Browse and select video' textbox and then click the 'Apply' button.

Confirmation

  • I performed a search of the feature requests to avoid suggesting a duplicate feature
  • I understand that not filling out this template correctly may lead to the request being closed
@amymor amymor added the enhancement New feature or request label Nov 17, 2023
@SegoCode
Copy link
Owner

SegoCode commented Nov 18, 2023

Hey there!

Well, the JPEGView is not a bad idea at all, I will add it to the table of things to do. And so for the file types, I didnt add the . to prevent the app from breaking importing files that are not compatible, however, I added a note in the readme indicating that just what you say maybe add some catch for break mpv or something like that. i think about it

btw thanks for your interest in my app!

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

No branches or pull requests

2 participants