Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Scripts: Update webpack config to allow modern image format #43880

Merged
merged 2 commits into from
Sep 7, 2022

Conversation

niklasp
Copy link
Contributor

@niklasp niklasp commented Sep 5, 2022

allow webp in wordpress plugins

What?

just add webp to webpack config as module rule

Why?

people might want to include webp images in their packages to serve smaller plugin bundles

@gziolo gziolo changed the title update webpack config to allow modern image format Scripts: Update webpack config to allow modern image format Sep 7, 2022
@gziolo gziolo added [Tool] WP Scripts /packages/scripts [Type] Enhancement A suggestion for improvement. labels Sep 7, 2022
@gziolo
Copy link
Member

gziolo commented Sep 7, 2022

@niklasp, thank you for your contribution. It sounds like a necessary addition 👍🏻

@gziolo
Copy link
Member

gziolo commented Sep 7, 2022

@niklasp, how did you test this change?

@niklasp
Copy link
Contributor Author

niklasp commented Sep 7, 2022

have a 10k active install WordPress plugin and switched to wp scripts recently. had to do the above change to get my webp media successfully to the build folder

@gziolo
Copy link
Member

gziolo commented Sep 7, 2022

Fair enough. Let's get this change in 👍🏻
Thank you again @niklasp.

@gziolo gziolo merged commit 2ae79f3 into WordPress:trunk Sep 7, 2022
@github-actions github-actions bot added this to the Gutenberg 14.1 milestone Sep 7, 2022
@niklasp
Copy link
Contributor Author

niklasp commented Sep 7, 2022

😎

@niklasp niklasp deleted the patch-3 branch September 7, 2022 16:27
@desrosj
Copy link
Contributor

desrosj commented Oct 21, 2022

@niklasp While collecting props and credits for 6.1, it seems you are one of a few dozen folks that have not connected their WordPress.org account to their GitHub profile.

Could you share your WordPress.org account so that we can ensure you're properly credited on the About page for WordPress 6.1? Connecting your accounts will also help those collecting this in the future. Thanks in advance!

@niklasp
Copy link
Contributor Author

niklasp commented Oct 21, 2022

done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Tool] WP Scripts /packages/scripts [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants