rommapp
misc: Optimized covers storing them as webp format
Description
Now any kind of new artowrk (uploaded or fetched) will be converted to .webp format to improve performance
Checklist Please check all that apply.
- [x] I've tested the changes locally
- [x] I've updated relevant comments
- [x] I've assigned reviewers for this PR
Running Code Quality on PRs by uploading data to Trunk will soon be removed. You can still run checks on your PRs using trunk-action - see the migration guide for more information.
![trunk-io[bot] avatar](https://avatars.githubusercontent.com/in/120106?v=4 "Published by a pub.dev verified publisher"){kind=small}
Test Results
93 tests ±0 92 ✅ ±0 29s ⏱️ -1s 1 suites ±0 0 💤 ±0 1 files ±0 1 ❌ ±0
For more details on these failures, see this check.
Results for commit 2f683138. ± Comparison against base commit cf575cc9.
:recycle: This comment has been updated with latest results.
![github-actions[bot] avatar](https://avatars.githubusercontent.com/in/15368?v=4 "Published by a pub.dev verified publisher"){kind=small}
what kind of "performance" are you seeking by switching to webp, loading times in the browser or smaller file sizes?
what kind of "performance" are you seeking by switching to webp, loading times in the browser or smaller file sizes?
I want to reduce the load on large libraries both in network loading times since files are smaller but also increase performance in the interactivity. I am still testing but seems like browsers handle better webp and it feels smoother than with png.
We will also need to build some kind of virtual DOM/virtual scrolling for large collections also in order to REALLY improve performance in large libraries, but that's for another PR
Thinking about this more it's likely webp images won't be supported on muos or playnite, for when we decide to fetch cover art for this plugins. Going to close this PR given that case, and I'm sure there are other image perf changes we can make in the future.