-
-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
feat(ideal-image): support webp & avif formats #6051
Conversation
6aeae90
to
f6ec5d4
Compare
Interesting, the ideal image loader doesn't seem to upsize images from 640 to 1030... The previous build error was because our ideal image max width is 1030, but I resized the image to 640. I would need to locally test with #6043 to see if that's only a quirk with WebP |
49c6641
to
fc3e7ce
Compare
I think we should also allow passing multiple source sets to the ideal image component because the component is able to pick different sources. |
It wasn't even partially supported in Safari until 2020, but I think its widely supported enough to include. |
Could this be merged now since slorber/responsive-loader#2 was merged? |
Btw – since it wasn't mentioned here yet – AVIF doesn't seem to be supported on iOS and in macOS Safari. |
@matkoch The library we use under the hood doesn't support webp yet. See comments above. |
Hello! It's been a bit over a year since last update on this. Is this PR still going to happen on v2 or is it waiting for v3? Best regards |
This feature is not a top-priority for me. It won't be part of v3 initial release but can eventually come in a minor release once we have a working implementation, which we currently have not.
What's your point here? 🤔 If you need this feature fast, the best way is to submit a PR to add support that works. So far all attempts have failed. Closing in favor of #8686, more advanced, less stale PR |
Thank you for the update! I'll make an PR if I come up with something. How ever I don't have much experience yet, so I would not hold my breath.
Sorry if my message came out wrong (I'm not native english speaker). I was just pointing out that this PR has been inactive and I'm not sure what the current status of it is, so I thought that I should should ask. As a side note, thanks for the great software. I definetely will contribute somehow if I find out something I can do! |
Motivation
See #6043 (comment). Luckily
sharp
supports both WebP and AVIF, so it's a good opportunity to enable them.Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Converted one image to WebP for dogfooding.
Blocking PRs
Currently build will fail because blocked by slorber/responsive-loader#2