search

rosell-dk / webp-convert

Convert jpeg/png to webp with PHP (if at all possible)
MIT License
578 stars 102 forks source link

AVIF #256

Open S1SYPHOS opened 4 years ago

S1SYPHOS commented 4 years ago

Hey there, it's been a while since we last talked. I finished my studies and want to add AVIF support by including its Rust implementation - but is this really a job for webp-convert or a stand-alone library, what do you say?

Cheers, S1SYPHOS

rosell-dk commented 4 years ago

I've been pondering the same question. I think the cleanest solution is a new library even though much of the code will be identical.

rosell-dk commented 4 years ago

https://github.com/rosell-dk/webp-express/issues/432

rosell-dk commented 4 years ago

Just like with webp, there are several ways to convert. Vips and ImageMagick supports AVIF. So it makes sense to make a library like webp-convert, but for avif, which is what I intend to do: https://github.com/rosell-dk/avif-convert

rosell-dk commented 4 years ago

Just been sidetracked, developing this new library: https://github.com/rosell-dk/htaccess-capability-tester

S1SYPHOS commented 4 years ago

You're very productive, my friend, and I really admire what you have done since last we spoke!

S1SYPHOS commented 4 years ago

I've been pondering the same question. I think the cleanest solution is a new library even though much of the code will be identical.

That's what why it should be incorporated, no? Like php-nextgen-images (or whatever).

Maybe make all general purpose stuff (logging, read/write checks, ..) a separate package (like you did with the file type guesser) and make it a common dependency for both webp-convert and avif-convert?

rosell-dk commented 4 years ago

Thanks :) A common dependency sounds right!

dkrnl commented 3 years ago

hi all!

good news everyone:

  1. libgd merged support avif -- https://github.com/libgd/libgd/pull/671
  2. wait php -- https://github.com/php/php-src/pull/5127
S1SYPHOS commented 3 years ago

Still has to land on (shared) hosting servers, which should take a while - but that's great news indeed!

Thanks for letting us know!