Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings
This repository was archived by the owner on Oct 20, 2023. It is now read-only.
/theme-scriptsPublic archive

Add support for WEBP images#149

Open
matthew-petrie wants to merge2 commits intoShopify:master
base:master
Choose a base branch
Loading
frommatthew-petrie:patch-1
Open

Add support for WEBP images#149

matthew-petrie wants to merge2 commits intoShopify:masterfrommatthew-petrie:patch-1

Conversation

@matthew-petrie
Copy link

@matthew-petriematthew-petrie commentedFeb 11, 2021
edited
Loading

Add support for.jpeg.webp &.jpg.webp images in thegetSizedImageUrl function.

Shopify outputs images in WEBP format which need to be able to have sizes added, for example this real Shopify URL:https://cdn.shopify.com/s/files/1/0896/0640/products/IMG_20190622_130045_clipped_rev_1.jpeg.webp?v=1582896583

joshistoast reacted with heart emoji
Add support for `.jpeg.webp` images in the `getSizedImageUrl` function.Shopify outputs images in WEBP, for example this real Shopify URL: `https://cdn.shopify.com/s/files/1/0896/0640/products/IMG_20190622_130045_clipped_rev_1.jpeg.webp?v=1582896583`
@ghostghost added the cla-needed labelFeb 11, 2021
@SpurIT
Copy link

SpurIT commentedFeb 11, 2021 via email

Hello,I am out of the office till March 1. Please reach out to the Sales Manager that you are working with or to philip.p@spur-i-t.com if you have any questions.

@matthew-petrie
Copy link
Author

I've signed the CLA but not sure how to re-run the check

@ghostghost removed the cla-needed labelFeb 16, 2021
@joshistoast
Copy link

Would love to boost our performance with webp images, but the lack of support and this dangling pr is frankly sad

matthew-petrie and joshistoast reacted with thumbs up emoji

Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@matthew-petrie@SpurIT@joshistoast

[8]ページ先頭

©2009-2025 Movatter.jp