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

Disable "Add postfix" doesn't seems to work #10

Open
guidoiaquinti opened this issue Feb 25, 2025 · 7 comments
Open

Disable "Add postfix" doesn't seems to work #10

guidoiaquinti opened this issue Feb 25, 2025 · 7 comments

Comments

@guidoiaquinti
Copy link

I'm on 2.1.0 and it seems that despite "Add postfix" is disabled, the tool still creates images with it

@blopker
Copy link
Owner

blopker commented Feb 25, 2025

Shoot, I'm unable to replicate. Could you post a screenshot of the settings used?

@guidoiaquinti
Copy link
Author

Thanks for the quick response! 🙏 Here are my current settings:
Image

I might be missing something, but my overall goal is to convert all files to .webp (and maybe remove the original files too?)

@blopker
Copy link
Owner

blopker commented Feb 25, 2025

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: ~/Library/Application Support/io.kbl.alic/settings.json. Additionally, it could be helpful if you provide the image file Alic is failing to overwrite, if possible.

@guidoiaquinti
Copy link
Author

guidoiaquinti commented Feb 26, 2025

Looks like a typo add_posfix vs add_postfix:

{
  "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
  }
}

pub add_postfix: bool,

Additionally, it could be helpful if you provide the image file Alic is failing to overwrite, if possible.

It's happening with any image.

@blopker
Copy link
Owner

blopker commented Feb 26, 2025

Thank you. That postfix change is not released yet, but I'm going to push it out to see if that fixes the issue. The code editor I use didn't have spell check until recently 😅

@blopker
Copy link
Owner

blopker commented Feb 26, 2025

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.

@blopker
Copy link
Owner

blopker commented Feb 26, 2025

BTW, if you enable Lossy compression and set webp quality to 100, you'll get much smaller file sizes without really any quality loss.

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

2 participants