Add avif to the compatible Image Formats

[yoshnopa]

Avif is a new open image format which is very likely to be the next big format. It is supported by Chrome (and Chromium-derivates) as well as Firefox, although there is a config variable you have to set (should be gone any time now, it was announced to be standard at the end of February).

[ghost]

Unfortunately adding avif to the supported extension list isn't enough, we also need to add an image decoder for avif, and as far as I can tell there isn't one yet apart from go-avif which is a wrapper around libaom which is a C-library.

[yoshnopa]

What do we need the decoder for? I tested it and everything worked fine.

[yoshnopa]

To elaborate on that, when searching for "gif" in the repo, there is only one occurance in the Code which is only triggered on creating a thumbnail for a scene (which in turn is done by taking a shot from the video). There is no occurance associated with the actual image object (from the image tab) beside in the array of allowed filetypes, the line that was modified in this request.

[bnkai]

You missed the image.SetFileDetails part probably.
Having a decoder is needed for the thumbnail creation as you mentioned and to get the dimensions width/height and calculate the resolution.
I dont think we should add the extension to the default ones till we have a working (pure-go) decoder.
Is there a specific reason it had to be in the default anyway? Any user can add whatever extension they want if they can live with the limited usability.

EDIT: Didnt make that clear before. Gif support is added when we import the image go package. It has png, jpg, gif (encode,decode) support so it wouldnt show up in the search you made for gif (an image.decode can handle all the mentioned formats).

[ghost]

I'm not overly realistic about there ever being a pure go avif decoder since it seems a whole lot more complex than most image formats. A more realistic approach might be using ffmpeg, but even that doesn't support avif yet.

At some point I'd like to figure out a way to dynamically link libjpeg when available, and if we get there we might be able to add libaom as well, but I don't know how easy it is.

[bnkai]

Sadly avif/heiff are already requested for a couple of years in the ffmpeg trac. I wouldnt expect them soon.

[yoshnopa]

Thank you for your patience, you're right I didn't find the SetFileDetals.
Also, I wasn't aware that you can set the allowed file endings in the settings, maybe I should check that before trying to compile a new version 😅
Anyway, thank you for your patience, I agree that this might be far to soon and would suggest compatibility where there is none.