Skip to content

Enable AVIF support in image crate - #140

Merged
mmstick merged 2 commits into
masterfrom
avif
Jun 25, 2026
Merged

mmstick merged 2 commits into
masterfrom
avif

Conversation

@mmstick

@mmstick mmstick commented Jun 25, 2026

Copy link
Copy Markdown
Member

Needed by pop-os/cosmic-settings#2047

  • I have disclosed use of any AI generated code in my commit messages.
    • If you are using an LLM, and do not fully understand the changes it is making to the code base, do not create a PR.
    • In our experience, AI generated code often results in overly complex code that lacks enough context for a proper fix or feature inclusion. This results in considerably longer code reviews. Due to this, AI authored or partially authored PRs may be closed without comment.
  • I understand these changes in full and will be able to respond to review comments.
  • My change is accurately described in the commit message.
  • My contribution is tested and working as described.
  • I have read the Developer Certificate of Origin and certify my contribution under its conditions.

@mmstick
mmstick requested review from a team June 25, 2026 14:15

@jacobgkau jacobgkau left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks from git history like this feature was initially enabled, but was dropped during #31.

I'm not seeing any difference in behavior with this update (an AVIF image is not showing up in Settings or as a slideshow when placed in a custom slideshow folder), so I assume it'll need a cosmic-settings update to pull it in, too?

I did regression testing, all settings on the Wallpapers page are still working as expected compared to before.

@mmstick
mmstick merged commit 71bb96b into master Jun 25, 2026
9 checks passed
@mmstick

mmstick commented Jun 25, 2026

Copy link
Copy Markdown
Member Author

Yes, I'll have avif support for cosmic-settings soon

@mmstick
mmstick deleted the avif branch June 29, 2026 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants