WEBP quick conversion skips larger files silently!

[porg]

Environment
GraphicConverter 11.7.1 (5632)
header.png — 984x72px
body.png — 984x24472px

1) Conversion of small file only → Works fine
Selecting header.png and triggering quick conversion to WEBP → header.webp is there instantly.

2) Conversion of large file only → Does nothing
Selecting body.png and triggering quick conversion to WEBP → Nothing happens.

I suspect it is just silently failing. B/c in Activity Monitor GraphicConverter idles and also has not spawned any child process.
Also when looking at the destinations in Finder or in GC's Browser the output file is not there.

3) Conversion of large file in set with small file(s) → Small file gets converted. But large file silently skipped.

[porg]

Update: Installed webp via brew and ran cwebp in the command line

Code: Select all

$ cwebp -lossless body.png  -o body-lossless.webp
Saving file 'body-lossless.webp'
Error! Cannot encode picture as WebP
Error code: 5 (BAD_DIMENSION: Bad picture dimension. Maximum width and height allowed is 16383 pixels.)

So instead of skipping silently the user should be informed that the image dimensions are not supported.

[forum_adm]

Thanks for your note.

Build 5633 (and later) will display an alert.

[porg]

Tested build 5633 and got a warning:

The file can't be saved as webP because the width and/or height is larger than the maximum size of 16383 pixels.

I would change the text slightly:

At least one file can't be saved as webP because its width and/or height are larger than the maximum size of 16383 pixels. Any other file(s) will be processed as planned.

Then it is true in all cases: Single improper file as well as mixed file selection with at least 1 improper files.

And I would offer two buttons: Cancel | Continue (=primary button)

Because maybe the user writes into an output folder with mixed content where it is a "all or nothing" situation. And an inbetween state (some files made it, some not) result in the need for manual cleanup, which is undesired.

[forum_adm]

Thanks for testing.

I add a separate message for batch conversions to build 5635.