Check for other animated formats

[afranke]

There is a check for gif images to only draw the first frame if no filename is given. Other animated formats such as webp fall through the cracks.

[hackerb9]

Ah! That's not good. I never did get multi-image support complete, mostly because I do not have many such files and ImageMagick can be rather persnickety about workarounds.

Would you mind uploading a sample file and describing how you'd like lsix to behave? Thanks!

[afranke]

Sure. Had to zip it because GitHub didn’t like it otherwise.

not_a_smart_man.zip

I expect this to behave just like it does for a gif file, where it shows the first frame as if it was a still image. Here is a screenshot exhibiting the issue:

Notice how there are gif files that correctly only show a single frame, and then the (attached) webp file floods the terminal.

I’m guessing there are other file extensions (apng maybe?) that should behave the same as well, but I don’t seem to have any such file.

[hackerb9]

Thanks for the sample file. It was a simple fix: just changed "gif" to "gif|webp".

[afranke]

I get two frames for webp files now though. 🤔

[hackerb9]

Oh. That's odd. I guess it isn't as simple of a fix as I thought.

[hackerb9]

Oh, ho! It was a typo introduced by Github's new fancy-pants builtin code editor. I thought for such a simple change how could I go wrong? Please try it now.

[afranke]

That’ll teach you to trust fancy new tech! 😉

It works, thanks!