XKit-Rewritten icon indicating copy to clipboard operation
XKit-Rewritten copied to clipboard

AccessKit: Apply Disable GIFs to all WebP images

Open marcustyphoon opened this issue 1 year ago • 1 comments

Description

Still not sure if this is a good idea (in this state). There are, apparently, a fair number of non-animated WebP images.

This

  • Applies Disable GIFs to WebP images (including non-animated ones; it is not particularly easy to distinguish the two).
  • Makes the label accurately indicate the source image type, so at least when it false positives on a non-animated image the user knows why.

Resolves #1401.

Testing steps

Animated WebP: https://www.tumblr.com/changes/758172275271991296/threaded-replies-are-here Animated GIF: https://www.tumblr.com/@radar/757906400962920448, https://www.tumblr.com/april-codes/700206746833240064 Non-animated GIF: https://www.tumblr.com/@radar/753920129042153472 Non-animated WebP: https://www.tumblr.com/minmos/744793127262601216/front-facing-egrets-making-me-lose-control-of, https://www.tumblr.com/@radar/755279084134744066, https://www.tumblr.com/@radar/750568043810340864

marcustyphoon avatar Aug 09 '24 07:08 marcustyphoon

Ugh, yeah, I just keep finding more non-animated WebP. This appears to be a regression significantly more often than a feature.

marcustyphoon avatar Aug 09 '24 07:08 marcustyphoon

Closing in favor of #1709.

marcustyphoon avatar Jan 23 '25 14:01 marcustyphoon