TYPO3 Core

Hi Alex,

whats your intention of this report? Would you have smaller processed thumbnails? But if you set jpgQuality globally to a lower value, you render also heavily compressed images at the frontend.

Btw: The setting jpgQuality is not considered in the master.

Best, Guido

Hi Guido

whats your intention of this report? Would you have smaller processed thumbnails?

I don't really recall what I thought 9 month ago but from reading my own report here again, I'd say, yes.

But if you set jpgQuality globally to a lower value, you render also heavily compressed images at the frontend.

I don't really get your point here.
If I am not using any cropping the jpg_quality configuration should work both in the frontend and backend. But when using the cropping the configuration is simply ignored. Should apply for backend and frontend as well here.

I don't know if images should be rendered differently in the backend and in the frontend but that's not part of the bug report. There is simply no compression at all for cropped images IIRC.

Btw: The setting jpgQuality is not considered in the master.

I don't get that.

Alex

Hi Alex, sorry I misunderstood you.

I have now added a patchset. The LocalCropScaleMaskHelper sets its own settings.

If the image is not cropped, the GraphicalFunctions::init() takes effect and sets the default quality values, but cropping overwrites them. Maybe my patch is not good enough and we should correct the GraphicalFunctions. But i think it's okay. Let's wait for the reviews :-)

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56779

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56779