Page MenuHomePhabricator

Animated Gifs do not play in-line in comments
Closed, ResolvedPublic

Event Timeline

Feature. They are explicitly disabled in our configuration. We don't want to encourage distracting avatars and lolcatz. After tokens, this might be the next hot topic. :P

Proposing Declined.

Feature!
Possibly this is the case because animated avatars are turned off. (We do not want to see animated avatars. They are very annoying to some users.)

However, I can see the animation in Jared's comment at T68819#835013 if I click it.
That seems sufficient to me, given that compressed thumbnails (what we see embedded in comments) are generally bad, and even tiny ones are reduced in size by phabricator's thumbnail system. Compare:

mediawiki.feedback.spinner.gif (18×18 px, 1 KB)

with https://github.com/wikimedia/mediawiki/blob/master/resources/src/mediawiki/mediawiki.feedback.spinner.gif

But now we have to explain to someone to click on an image in order to even determine if it is animated. Thats like having to explain the punchline.

I know, the joy of having users posting animated gifs combined with the joy of reading the comments of users upset, and of course also the joy of waiting for the comments of those that will ask for the licensing of the animated gifs. :)

Quiddity claimed this task.

Seconding.

Jdlrobson subscribed.

This should be reconsidered. Fun is important.

This used to work and worked in the preview.

thumbs-up-down-left-right

You just need to create a macro.

Aklapper triaged this task as Lowest priority.Jan 18 2017, 7:17 PM

Any reason we can't just add a file attachment? The macro approach seems a bit creepy as I can make gifs show on all sorts of keywords...

Personally, if the "solution" is to use macros, yeah, that sucks for a number of reasons. So then we must weigh that with what Quim said above:

In T78072#835181, @Qgil wrote:

I know, the joy of having users posting animated gifs combined with the joy of reading the comments of users upset, and of course also the joy of waiting for the comments of those that will ask for the licensing of the animated gifs. :)

:/

We require imagemagick to thumbnail animated GIFs. If it isn't installed, or files.enable-imagemagick isn't enabled in Config, we can't generate animated thumbnails.

You can embed animated GIFs without using macros by disabling thumbnailing, using {F...., size=full} like this (I solemnly swear and affirm that I definitely have 100% of the rights to this banana):

{F5309100, size=full}

banana.gif (325×340 px, 45 KB)

Even without imagemagick, animated avatars may work if the source image you upload is exactly the right size (160x160, maybe? I don't recall) but I think we may shove them through the resizer anyway. If they do work, don't tell anyone I told you.

imagemagick thumbnails some GIFs wrong (when they have unusual backgrounds or layers or transitions or frame timings) which is sort of a bug, but not one we can necessarily do much about. But the rest of this is a configuration issue / feature.

(There's currently no way to enable animated GIF thumbnailing in comments without also enabling users to use GIFs as avatar photos.)

giphy (2).gif (282×500 px, 1 MB)

This changes everything.
(!!!!)

Jdlrobson claimed this task.

Looks like fun in the world has been restored.

I hope everyone has certificates documenting their rights to use those images!