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

[BUG] Icon fetcher fails to load icons/thumbnails for animated GIFs #1974

Closed
2 tasks done
martinsifrar opened this issue Apr 15, 2024 · 4 comments · Fixed by #1975
Closed
2 tasks done

[BUG] Icon fetcher fails to load icons/thumbnails for animated GIFs #1974

martinsifrar opened this issue Apr 15, 2024 · 4 comments · Fixed by #1975

Comments

@martinsifrar
Copy link
Contributor

Rofi version (rofi -v)

Version: 1.7.5, also checked on (next 1.7.5-228-g6c38a49d)

Configuration

https://gist.github.com/martinsifrar/e2b3724060c81bafe4682698d2252174

Theme

https://gist.github.com/martinsifrar/5343660f12cd4fd8cec0125ba5296899

Timing report

No response

Launch command

rofi -show-icons -show filebrowser

Step to reproduce

Expected behavior

Rofi should show the first frame of the animated GIF.

Actual behavior

The thumbnail is not shown. Rofi prints out a debug Helpers.IconFetcher-WARNING warning

(process:360901): Helpers.IconFetcher-WARNING **: 12:01:47.513: Failed to load image: |/home/martin/testgif/animated/rotating-95.gif| 23 23 Not all frames of the GIF image were loaded. (0x7ee188003970)

Additional information

No response

Using wayland display server protocol

  • No, I don't use the wayland display server protocol

I've checked if the issue exists in the latest stable release

  • Yes, I have checked the problem exists in the latest stable version
@DaveDavenport
Copy link
Collaborator

We use gdkpixbuf for image loading, I think this might be relevant: https://gitlab.gnome.org/GNOME/gdk-pixbuf/-/issues/45

@martinsifrar
Copy link
Contributor Author

We use gdkpixbuf for image loading, I think this might be relevant: https://gitlab.gnome.org/GNOME/gdk-pixbuf/-/issues/45

I already had the fix running on my computer. It just kinda took me a while to make the issue and the PR, as I'm not very used to Github.

@DaveDavenport
Copy link
Collaborator

Thanks for the PR! merged.

Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants