Fixed Image quality for client side resizing is hardcoded

[Kirby]

Affected version

2.3.0 Beta 2

The image qulity (0.85) for client side resizing is hardcoded somewhat deep inside XF.ImageTools.resize.

With XenForo 2.2 it was rather easy to change to quality (via a class extension), but doing this with client side resizing currently seems to be much more invloved.

Could XF.ImageTools.resize be changed to smth. like

XF.ImageTools =
{
    getQuality (file) { return 0.85 },

    resize (file, maxWidth, maxHeight, asType, quality)
    {
        return new Promise((resolve, reject) =>
        {
            [...]

            quality = quality || this.getQuality(file)

            [...]
                canvas.toBlob(
                    [...],
                    asType,
                    quality

?

 

[XF Bug Bot]

Thank you for reporting this issue, it has now been resolved. We are aiming to include any changes that have been made in a future XF release (2.3.0 Beta 3).

Change log:

Add a new config.php option for image quality that is used as a default for Imagick and GD drivers, and XF.ImageTools.resize. Also add a `getQuality` to XF.ImageTools for easier overriding.

There may be a delay before changes are rolled out to the XenForo Community.

 

[Mr Lucky]

What is the new config.php option? Thanks

 

[Chris D]

Default is 85. Add this and adjust the number accordingly:

$config['maxImageResizeQuality'] = 85;

 

[Chromaniac]

Is this in the RC just released? coz this setting does not seem to change image size on RC4!

nm. it does. nothing changed between 85 and 95. but it did change on 15. thanks for adding this!