search

LadybirdBrowser / ladybird

Truly independent web browser
https://ladybird.org
BSD 2-Clause "Simplified" License
19.33k stars 784 forks source link

JXL support #683

Closed RubenKelevra closed 1 month ago

RubenKelevra commented 1 month ago

I've searched in the source code, and there seems to be test-cases for the JPEG-XL data format. But the browser itself doesn't load any JXL-files on the test-website.

https://jpegxl.info/test-page/

Is support still unfinished or is it currently broken?

skyrising commented 1 month ago

We have a home-grown JPEG-XL decoder, but it's incomplete. The images on that page happen to use features we don't support yet (Encoding VarDCT and animation). The decoder will likely be replaced by libjxl soon-ish.

LucasChollet commented 1 month ago

The current decoder is the unfinished one that I developed from scratch for SerenityOS. It only supports a subset of lossless so you might want to test with the jxl-art, the success rate is better there. I will open a PR to replace it with libjxl soon.

RubenKelevra commented 1 month ago

That's exciting to hear that the support will be improved. Thanks for the fast response.

Quite cool actually, that you wrote your own decoder!

RubenKelevra commented 1 month ago

I've checked, it works great. Thanks @LucasChollet and @awesomekling!