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

Animated WebP support #5470

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Animated WebP support #5470

wants to merge 6 commits into from

Conversation

Aely0
Copy link

@Aely0 Aely0 commented Dec 13, 2024

Adds support for animated WebP images. Used the already existing GIF implementation as a template for most of it.

  • I have followed the instructions in the PR template

Copy link
Owner

@emilk emilk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, if tested

crates/egui/src/widgets/image.rs Outdated Show resolved Hide resolved
Copy link

Preview available at https://egui-pr-preview.github.io/pr/5470-animated-webp
Note that it might take a couple seconds for the update to show up after the preview_build workflow has completed.

@Aely0
Copy link
Author

Aely0 commented Dec 17, 2024

Needed this feature for a project I'm working on, currently using this branch as a dependency. So as far as testing goes, everything has been working as expected. All the quirks I came across were ironed out pre-PR.

@Aely0 Aely0 requested a review from emilk December 17, 2024 08:18
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's make sure this is on git lfs

Copy link
Owner

@emilk emilk Dec 17, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried to fix this for you, but failed:

Pushing to github.com:Aely0/egui
error: Authentication error: Authentication required: You must have push access to verify locks
error: failed to push some refs to 'github.com:Aely0/egui'

You need to run this:

git lfs track examples/images/screenshot.png
git lfs track examples/images/src/cat.webp
git checkout .gitattributes
git add -u
git commit -m "Move images onto git lfs"
git push

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done, though strange why you were not able to push to this branch
image
I thought that was the whole point of this checkbox

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Weird 🤔

@Aely0 Aely0 requested a review from emilk December 17, 2024 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants