Provide drimage support for core Webp

Certainly a direction I would like go with for drimage.
Supporting webp natively from core looks like the preferred way to go. For now webp was added more as an afterthought ...

Adding a checkbox will probably not be enough.
The current webp implementation will (I think) be incompatible with the the new webp implementation you propose. So theses 2 checkboxes would need to exclude one another...

As far as the changes to the drimage delivery mechanism: On a high level your assumption seems good to me, but I would have to examine it more in depth. I personally have not worked (yet) with native Drupal webp so I'm uncomfortable giving directions here.

If you are willing to work on this, I would be happy to review any patch/MR/fork you make.

Hi weseze,

I am currently at Drupal Con Lille and will try to make a first draft. To see if I can get this working.

Greetings,

Fabio

All core shipped image styles now include standard webp conversion: https://www.drupal.org/node/3406267 →

Hi Dieter,

I will test the new version, so that this ticket can be put on RTBC.

Greetings,

Fabio

Hi Dieter,

I finished testing the change, I have tested the module on three different Drupal instances.

First on a clean D9 installation core recommended version 9.5.11 PHP versions 8.0 and 8.1

Secondly on a clean D10 installation core recommended version 10.2.4, PHP version 8.1

And lastly on our Calibrate DXP install profile core recommended version 10.2.3, PHP version 8.1

Our install profile works with Imagemagic as the image toolkit and here I noticed that it did not take into account the status of Webp in the toolkit after the Drimage functionality was already enabled once (here I mean config wize, because the field was disabled on the form, but still enabled in config).

So have added another check in the DrimageSettingsForm on line 178 after the check on image toolkit support to see if Core Webp is enabled. So that it automaticly disables when support is removed, just as happens when enabling the image optimise webp module.

If you can review this change and it's ok for you, then I think this ticket may be put on RTBC.

Greetings,

Fabio

Just tested this one last time on another project and changing from ImageAPI Optimize WebP → to core webp works perfectly. I'll merge this.

@weseze okay for you if I create a 2.3.0 release?

I'll go ahead and create a new release.