Video - Options

[jauyong]

Feature Description

As an editor utilizing videos, I want to see what options are available for my videos

Waiting for Feature Brief

https://www.figma.com/file/Cd9iacVknHK62RO4pRwb68/Stories-Editor?node-id=2671%3A55920

---

Do not alter or remove anything below. The following sections will be managed by moderators only.

1. The editor is presented with a looping option
2. The editor is able to mute a video (the video will have no sound when played) (Note: Videos are muted in AMP)

QA Instructions

[spacedmonkey]

WordPress it's supports added subtitle tracks and alternative version of file ( ogv, webm ). Should we also support these @swissspidy @pbakaus @jauyong ?

[spacedmonkey]

Worth noting that gutenberg itself does not support subtitles ( WordPress/gutenberg#7673 ) or alternate sources ( WordPress/gutenberg#9457 ), at time of writing.

[swissspidy]

Alternative versions: AMP Stories only support MP4, WEBM, HLS (docs), so this is a bit less relevant. I'd say we should just focus on MP4 for now. Other formats should be handled automatically by the editor in the future (WebAssembly, CDN, etc.)

Subtitles: Perhaps at some point in the future, but not a P0 in my eyes. Even then we should ideally just automate it and hide it from users.

Deferring to @pbakaus for making the decision though.

[pbakaus]

Alternative versions: AMP Stories only support MP4, WEBM, HLS (docs), so this is a bit less relevant. I'd say we should just focus on MP4 for now. Other formats should be handled automatically by the editor in the future (WebAssembly, CDN, etc.)

Subtitles: Perhaps at some point in the future, but not a P0 in my eyes. Even then we should ideally just automate it and hide it from users.

Deferring to @pbakaus for making the decision though.

I would say let's do MP4 and WEBM for now. Want to find out whether, and how we can support HLS in the future. /cc @newmuis

[newmuis]

We're in the midst of reevaluating what the best options are for video, but I don't think HLS makes the cut as a recommendation at this moment. We'll update the docs once this is finalized, but doing only WEBM/MP4 makes sense for now

[jauyong]

Hey @samitron7 is there a design for where the looping option shows? Or is it the same as Jonny's screenshot above?
I looked at the media panel section and the design panel - video section in Figma.

[jauyong]

@pbakaus the brief for Media-specific controls is also missing as denoted by the red text in the Product Spec doc.

[pbakaus]

@jauyong all related spec should be written. Note that there's a red "Panel:Video" in the spec, but we only really need that when we ship subsequent features not in v1.

[jauyong]

I couldn't find where looping was covered in the docs but since this is in code review I'd assume Jonny found it

[pbakaus]

arrgh you're so right. The loop option was missing. Will add now.

[pbakaus]

Alright, it can be found here: https://docs.google.com/document/d/1EQ12sD4a1plmMeBu7YMWOdUSg0FeegB9qi2_OlV3Tp0/edit#heading=h.h3uoy4bqqh92

[swissspidy]

Just realized that muting is deprecated in favor of using autoplay (which we already do), so we don't need to implement that.