- π¬π§United Kingdom catch
I think we can do this now that Catalina is out of support, but I've asked other committers what they think too.
- π¬π§United Kingdom catch
Note that β¨ Add fallback format support to responsive images Needs work would make OSX Catalina a non-issue, but I'm not sure we should block this issue on that one. It'll still be useful later on for AVIF and other formats.
- π¬π§United Kingdom catch
Discussed this with @lauriii and he also thinks we should go ahead here despite Catalina - given it's been out of support since November, Apple's not supporting it so neither should we.
The core change won't prevent sites reverting back to png or using the contrib webp module with fallback support if they still want to, it's just making defaults (much) faster/smaller for everyone else.
- π¬π§United Kingdom catch
Untagging now there's been a +1 from a front end framework manager, and looking at things like fermilab dropping support has also reassured me it should be fine. Also anecdotally I know one person who was still on OS Catalina last year, and I asked them last week if they still are, and they're not any more. We can't get proper usage data because Safari reports Big Sur as Catalina, but there's every other sign that people are getting off it.
- Status changed to Needs review
over 1 year ago 1:57am 15 June 2023 - last update
over 1 year ago Patch Failed to Apply - π«π·France andypost
It probably needs to CR and upgrade path but let;s start from all places I can find with
git grep 'png gif jpg jpeg'
- π«π·France andypost
Probably it makes sense to convert images in Demo umami from jpeg as each file becoming smaller more the 15KB
- Open on Drupal.org βEnvironment: PHP 8.2 & MySQL 8
1:36 1:07 Queued - π«π·France andypost
Probably duplicate β¨ Allow user-uploaded WebP images everywhere in Drupal by default: image fields, media library, text editors Needs work
- last update
over 1 year ago 29,450 pass - Status changed to RTBC
over 1 year ago 2:10pm 15 June 2023 - πΊπΈUnited States smustgrave
Change looks good.
Would be nice to see this get into 10.2 early.
- Status changed to Needs review
over 1 year ago 3:36pm 15 June 2023 - π¬π§United Kingdom longwave UK
I searched for "jpeg", excluding tests and "webp", and found a few other places we might want to consider adding webp while we are here:
core/assets/scaffold/files/gitattributes:*.jpeg -text diff core/assets/scaffold/files/robots.txt:Allow: /core/*.jpeg core/assets/scaffold/files/robots.txt:Allow: /profiles/*.jpeg core/lib/Drupal/Core/File/file.api.php: $cdn_extensions = ['css', 'js', 'gif', 'jpg', 'jpeg', 'png']; core/modules/ckeditor5/ckeditor5.ckeditor5.yml: types: ["jpeg", "png", "gif"] core/modules/ckeditor5/src/Controller/CKEditor5ImageController.php: 'file_validate_extensions' => ['gif png jpg jpeg'], core/modules/editor/src/Form/EditorImageDialog.php: 'file_validate_extensions' => ['gif png jpg jpeg'], core/modules/file/src/Upload/FileUploadHandler.php: const DEFAULT_EXTENSIONS = 'jpg jpeg gif png txt doc xls pdf ppt pps odt ods odp'; core/modules/image/src/Plugin/Field/FieldType/ImageItem.php: $extensions = array_intersect(explode(' ', $settings['file_extensions']), ['png', 'gif', 'jpg', 'jpeg']); core/modules/media/media.install: $files = $file_system->scanDirectory($source, '/.*\.(svg|png|jpg|jpeg|gif)$/');
- π¬π§United Kingdom longwave UK
Actually I see β¨ Allow user-uploaded WebP images everywhere in Drupal by default: image fields, media library, text editors Needs work covers some of these.
- π¬π§United Kingdom catch
I think it's worth leaving the image derivative issue (this one) and the upload issues separate, unless they really need to be combined - just to have less things to to worry about in each.
- Status changed to RTBC
over 1 year ago 4:13pm 15 June 2023 - π¬π§United Kingdom longwave UK
OK, let's just land this here, and leave the other one open to handle the remaining cases.
-
longwave β
committed a87a72fa on 11.x
Issue #3275557 by andypost, catch, heddn: Add webp image conversion to...
-
longwave β
committed a87a72fa on 11.x
- π¬π§United Kingdom longwave UK
Committed and pushed a87a72fa063 to 11.x (10.2.x). Thanks!
- Status changed to Fixed
over 1 year ago 4:16pm 15 June 2023 - π¬π§United Kingdom catch
Added a release notes snippet, don't think there's anything else to mention so leaving a CR. Also tagging for 10.2.0 release highlights since this will be a nice performance improvement on new sites. Also not sure how many site admins actually know the webp option is in core.
Automatically closed - issue fixed for 2 weeks with no activity.
- πΊπΈUnited States adamsilverstein
Great to see this merged into Drupal core!
I noticed that the Lighthouse Stack Pack for Drupal for the `modern-image-formats` audit point to this documentation : https://www.drupal.org/docs/core-modules-and-themes/core-modules/image-m... β which makes no mention of WebP or how to enable modern image formats. Should we update the documentation here, or is there a better place we can link the Stack Pack message? Since the Stack Packs are open source we can proposes changes for the link or recommendation text directly on the repository.
- Status changed to Fixed
about 1 year ago 12:12pm 4 November 2023 - π«π·France ericdsd France
Even if i've been waiting for real webp support for a long time, I sadly think doing so without fixing at least Add fallback format support to responsive images β¨ Add fallback format support to responsive images Needs work is not a really a news, as lot of old mac users cannot update their os (eg. lots of artists still use macbook pro 2012, not mentioning poor people). It means that doing this we'll collectively enforce Apples programmed obsolescence by providing massivelly degraded web experience for people not able to change their computer every 6 years as Apple now forces people to to). In a more general way we should provide a more robust out of the box fallback methods to ease adopting new images and video formats like avif.
Do you think there is a chance to see fallback issue embedded in 10.2 too ? - π¬π§United Kingdom catch
This issue didn't actually add webp conversion to core's shipped image styles at all, just as an option for upload to image fields (which is fine, but not what the issue was originally for) I've opened π All core install profile image styles should include webp conversion Active to do that bit.
- π¬π§United Kingdom catch
Untagging from 10.2.0 release highlights, hopefully we can get the follow-up into 10.3/11.x