Opened 3 years ago
Closed 3 years ago
#54146 closed defect (bug) (fixed)
The extension "webp" is missing
Reported by: | almendron | Owned by: | adamsilverstein |
---|---|---|---|
Milestone: | 5.9 | Priority: | normal |
Severity: | normal | Version: | 5.8.1 |
Component: | Media | Keywords: | |
Focuses: | Cc: |
Description
File wp-includes/js/media-views.js
Line 3449
return /\.(jpe?g|png|gif)$/i.test( attachment.get('filename') );
File wp-includes/js/dist/block-editor.js
Line 26680
}) => /^image\/(?:jpe?g|png|gif)$/.test(type)); // Only process files if no HTML is present.
Line 30251
}) => /^image\/(?:jpe?g|png|gif)$/.test(type)).map(file => `<img src="${Object(external_wp_blob_["createBlobURL"])(file)}">`).join('');
File wp-includes/js/plupload/wp-plupload.js
Line 379
mage = /(?:jpe?g|png|gif)$/i.exec( file.name );
Change History (7)
#4
@
3 years ago
I believe the usage in wp-includes/js/dist/block-editor.js
comes from Gutenberg.
I opened this PR to updates usages in Gutenberg (although I don't know exactly how these are used, I assume webp should be added): https://github.com/WordPress/gutenberg/pull/36361
cc: @gziolo
This ticket was mentioned in Slack in #core by audrasjb. View the logs.
3 years ago
#6
@
3 years ago
This is fixed upstream in https://github.com/WordPress/gutenberg/pull/36361 which I just merged. Marking this ticket as resolved.
@almendron let me know if you still see this issue anywhere, thanks again for raising.
Hi @almendron - thanks for the bug report.
Did you notice any effect of the missing
webp
extension in each of these cases? I agree thee likely need updating, however it would be good to understand what each does so we can document and test that the change fixes an issue.