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

Should some format options be merged? #870

Closed
JinEnMok opened this issue Mar 10, 2024 · 3 comments
Closed

Should some format options be merged? #870

JinEnMok opened this issue Mar 10, 2024 · 3 comments
Labels
enhancement New feature or request

Comments

@JinEnMok
Copy link

Is your feature request related to a problem? Please describe.

We've currently (as of 2.7.1-alpha02) got separate selectors for variations of formats, e.g. JPG/JPEG/MozJPEG, HEIF/HEIC, and separate lossy/lossless options for PNG, JXL, and WebP. This causes some confusion for an end user, especially since there aren't any descriptions of said formats in the app itself.

What's the difference between JPG and JPEG, or is it only an extension rename? And if MozJPEG uses the MozJPEG coder, what do the other two options use? Aren't HEIF and HEIC the same thing?

As for lossy/lossless, while it may be convenient to be able to easily choose the mode, it also introduces clutter in the UI (same for the scaling options, IMHO).

Describe the solution you'd like

Perhaps it'd be possible to only leave one button for each format, and then introduce separate selectors for sub-formats afterwards?

Same goes for lossy/lossless. Maybe a separate button switch, or just the quality slider itself, except it goes further to the right to a special "lossless" setting?

@JinEnMok JinEnMok added the enhancement New feature or request label Mar 10, 2024
@T8RIN
Copy link
Owner

T8RIN commented Mar 10, 2024

Sub-options should be possible, so i will look at how to implement this without breaking everything 👀

@T8RIN
Copy link
Owner

T8RIN commented Mar 10, 2024

Screenshot_20240310-224327.png

@T8RIN T8RIN closed this as completed Mar 10, 2024
T8RIN added a commit that referenced this issue Mar 10, 2024
@JinEnMok
Copy link
Author

You're the best, thanks for this! :D

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