Animated GIFs are notorious for their large file size--700kb is not uncommon for GIFs are narrow as 350px. Therefore, a problem exists when a user uploads an animated GIF into an image field that is styled with retina_images.

Because the module will double the preset width, it can make some animated GIFs upwards of 20MB, the creation of which cripples the server and drags down the pageload. While a user can disable upscaling for presets that will encounter animated GIFs, that's an undesirable solution, as it negatively impacts non-GIF images (and still upscales the GIFs to the original preset size).

Forthcoming is a patch that introduces a second Image Style setting that allows site builders "ignore" upscaling animated GIFs.

CommentFileSizeAuthor
#1 retina_images-animated-gif-2113739-1.patch6.62 KBkmonty
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kmonty’s picture

kmonty
Location San Francisco, CA
CreditAttribution: kmonty commented
Status: Active » Needs review
FileSize
retina_images-animated-gif-2113739-1.patch6.62 KB

Please note: in order to test this, you must use the ImageMagick module.

mikey_p’s picture

mikey_p CreditAttribution: mikey_p commented
Issue summary: View changes

I'll take a look at this, although for the most part I would recommend limiting retina fields to non-GIF images and fields for GIFs to not use retina image effects.

mikey_p’s picture

mikey_p CreditAttribution: mikey_p commented

I took a quick look at the patch, and I don't love the idea of overriding both retina scaling and upscaling, since this module is only responsible for the retina scaling.