search

orgzly-revived / orgzly-android-revived

Outliner for taking notes and managing to-do lists
https://www.orgzlyrevived.com
GNU General Public License v3.0
669 stars 42 forks source link

Support avif image file inline displays #295

Open k4r4b3y opened 3 months ago

k4r4b3y commented 3 months ago

Old Orgzly app cannot render .avif image files inline. It can, however, render .webp image files inline. Add support for rendering .avif files as inline images, too.

Avif image files offer better compression and on-par visual quality with webp format. Users would essentially get nearly-same visual quality for lower sizes in their Orgzly directories.

amberin commented 3 months ago

It seems like it should be fairly easy to add avif support. We use Glide to render images, and it seems like we'd just need to add one dependency (according to https://bumptech.github.io/glide/int/avif.html) and update the "supported extensions" here: https://github.com/orgzly-revived/orgzly-android-revived/blob/master/app/src/main/java/com/orgzly/android/ui/ImageLoader.kt#L129

k4r4b3y commented 3 months ago

It seems like it should be fairly easy to add avif support. We use Glide to render images, and it seems like we'd just need to add one dependency (according to https://bumptech.github.io/glide/int/avif.html) and update the "supported extensions" here: https://github.com/orgzly-revived/orgzly-android-revived/blob/master/app/src/main/java/com/orgzly/android/ui/ImageLoader.kt#L129

Glad to hear your opinion on this. Looking forward to seeing development on the avif support.