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

feat: add support for animations #144

Closed
wants to merge 1 commit into from
Closed

feat: add support for animations #144

wants to merge 1 commit into from

Conversation

sarahetter
Copy link

This adds support for animated images, but it passes the animation modifier for all images, and I'm not sure if there's a performance hit for this.

To be investigated more thoroughly.

Closes netlify/next-runtime#787

@sarahetter sarahetter requested a review from a team April 24, 2023 14:59
@github-actions github-actions bot added the type: feature code contributing to the implementation of a feature and/or user facing functionality label Apr 24, 2023
@nickytonline
Copy link

Closing for now. We can reopen this if we want to revisit this.

@hrishikesh-k
Copy link

Based on this: unjs/ipx#53 (comment), it turns out that always passing the animated modifier is not an issue. So maybe this fix was enough to get this going? @nickytonline any context on why this was closed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature code contributing to the implementation of a feature and/or user facing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Automatic support for animated webp
3 participants