MOC.ImageOptimizer icon indicating copy to clipboard operation
MOC.ImageOptimizer copied to clipboard

Optimizer is not reliable because of thumbnail generation strategy in Neos

Open Sebobo opened this issue 9 years ago • 5 comments

Since the package was changed to only be triggered when a thumbnail is refreshed we have to rely on thumbnails actually being created. Neos currently does NOT create thumbnails when the original image is smaller than the requested dimensions. This way it can happen that many images in a site use the optimized original.

I tricked the strategy in one project by always allowing Neos to upscale which creates a thumbnail which is then optimized and still smaller than the original. But that shouldn't be the way to go.

Sebobo avatar Jan 19 '17 07:01 Sebobo

True, the only way to solve it as I see it is to have a strategy that always creates thumbnails, even if the dimensions are equal to the original. The originals should never be touched.

It could be done by overwriting the ThumbnailService class to get rid of this logic https://github.com/neos/neos-development-collection/blob/master/Neos.Media/Classes/Domain/Service/ThumbnailService.php#L97

aertmann avatar Jan 19 '17 16:01 aertmann

I feared as much. An easy way would be to have a forceThumbnailGeneration setting in the ThumbnailConfiguration object and respect that in the ThumbnailService. But that of course is a core change.

I'm not super excited about overwriting the whole getThumbnail method.

Sebobo avatar Jan 20 '17 08:01 Sebobo

yeah thought the same, would be nice if there wasn't so much logic in that method so it would be easier to override that behavior.. but yeah it'll have to be a core change, but maybe better sooner than never? could also be a global configuration to use originals if dimensions match or not

aertmann avatar Jan 20 '17 17:01 aertmann

Hey @Sebobo is this fixed in the core by now?

aertmann avatar May 05 '19 15:05 aertmann

Don't think so. Every few months I stumble upon the issue again and think about it, but didn't take the time yet to work on the solution. I think most people use some trick like changing default quality or always resize pictures to force the generation of thumbnails.

But of course this is not reliable or the way it should be.

Sebobo avatar May 06 '19 13:05 Sebobo