Page MenuHomePhabricator

Allow video embeds in formats other than OGV (e.g. WEBM)
Closed, ResolvedPublic

Assigned To
None
Authored By
matmarex
Feb 5 2019, 10:29 PM
Referenced Files
F28128846: Lunar_libration_with_phase2.avi
Feb 5 2019, 10:29 PM
F28128843: Lunar_libration_with_phase2.mp4
Feb 5 2019, 10:29 PM
F28128844: Lunar_libration_with_phase2.ogv
Feb 5 2019, 10:29 PM
F28128847: Lunar_libration_with_phase2.gif
Feb 5 2019, 10:29 PM
F28128842: Lunar_libration_with_phase2.webm
Feb 5 2019, 10:29 PM

Description

I humbly request that we allow video embeds in formats other than OGV. The Theora coding it uses produces relatively large files that often run afoul of the ~4 MB file size limit on Phabricator (or alternatively, extremely ugly files).

Could we allow embedding for some other file type as well, e.g. WEBM? WEBM file format is also free and widely supported by browsers, like OGV, and it uses much better coding (VP8/VP9).

For your reference, here's a bunch of tiny videos in different formats. Only the OGV video is embedded, the others are shown as links.

(generated from https://commons.wikimedia.org/wiki/File:Lunar_libration_with_phase2.gif using ffmpeg -i Lunar_libration_with_phase2.gif Lunar_libration_with_phase2.webm etc.)

AVI
MP4
WEBM
OGV
GIF (heh)
Lunar_libration_with_phase2.gif (334×356 px, 932 KB)

Event Timeline

People are using GIFs instead, because no one wants to spend 15 minutes every time re-encoding an OGV until you get one that is <4 MB and not utterly messed up by the encoder, and it makes me sad :( T210630#4949865 T215361

Change 569627 had a related patch set uploaded (by Zoranzoki21; owner: Zoranzoki21):
[operations/puppet@production] Add .webm in files.viewable-mime-types of Phabricator

https://gerrit.wikimedia.org/r/569627

Change 569627 merged by Dzahn:
[operations/puppet@production] Add .webm in files.viewable-mime-types of Phabricator

https://gerrit.wikimedia.org/r/569627

config change has been applied in prod by puppet. please confirm?:)

@matmarex @mmodell Looks like webm gets embedded now in the ticket description :)