-
Notifications
You must be signed in to change notification settings - Fork 3
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
Disable "Add postfix" doesn't seems to work #10
Comments
Shoot, I'm unable to replicate. Could you post a screenshot of the settings used? |
Sadly, I'm still not able to replicate. That sucks. To understand this better, I'll need to see the full settings file. It's located at: |
Looks like a typo {
"settings": {
"profiles": [
{
"active": true,
"add_posfix": false,
"convert_extension": "WEBP",
"enable_lossy": false,
"gif_quality": 80,
"id": 0,
"jpeg_quality": 80,
"keep_metadata": false,
"name": "To WebP",
"png_quality": 80,
"postfix": "",
"resize_height": 1000,
"resize_width": 1000,
"should_convert": true,
"should_overwrite": true,
"should_resize": false,
"webp_quality": 80
}
],
"theme": "System",
"threads": 0,
"version": 1
}
} alic/src-tauri/src/settings.rs Line 54 in 0c81b3c
It's happening with any image. |
Thank you. That |
The update is live now, but I still haven't been able to replicate the issue, so I'm not sure if it has been fixed. I've also reviewed the code. It's not obvious to me what could be happening. |
BTW, if you enable Lossy compression and set webp quality to 100, you'll get much smaller file sizes without really any quality loss. |
I'm on 2.1.0 and it seems that despite "Add postfix" is disabled, the tool still creates images with it
The text was updated successfully, but these errors were encountered: