Page MenuHomePhabricator

Flow: Parsoid HTML for videos doesn't work
Closed, DuplicatePublic

Details

Reference
bz61769

Event Timeline

bzimport raised the priority of this task from to Normal.Nov 22 2014, 3:02 AM
bzimport set Reference to bz61769.
bzimport added a subscriber: Unknown Object (MLST).
Quiddity created this task.Feb 21 2014, 8:25 PM

This is partly the same as bug 61786 (the image HTML generated by Parsoid is different from PHP parser output), but there are additional problems for video links.

For both [[File:Sequence-Test.ogv]] and [[ File:Sequence-Test.ogv|thumb]] in a Flow post, the links to the video are an invalid <a href="./File:Sequence-Test.ogv">. The HTML doesn't include a <video> tag, just an <img> tag. The full-size video uses the original .ogv file as the src of the nested image tag, which obviously doesn't work, while the thumb video correctly uses the Sequence-Test.ogv.jpg thumbnail.

Furthermore, the resulting HTML doesn't seem to trigger the MwEmbedSupport/TimedMediaHandler RL modules that add player controls and such to the video. Maybe those look for <div class="mediaContainer"> or the <video> tag.

O VisualEditor team, do you have CSS and JS that make Parsoid video HTML work? :)

fresh examples of webm video failing to render at fullsize, and ogg audio failing to display its player: https://www.mediawiki.org/wiki/Topic:S52i2un1zblzd8s8#flow-post-s5fley5jhc8a8pub (Does this bug/issue also cover audio files? I'm not sure whether to rename, or fork.)

Quiddity removed a subscriber: Maryana.Dec 19 2014, 1:42 AM
Restricted Application added a project: Collaboration-Team-Triage. · View Herald TranscriptJul 15 2015, 7:57 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 24 2015, 5:25 AM
QuimGil added a subscriber: QuimGil.May 6 2016, 6:02 PM

The "thumb" works ok

Not really. A thumbnail is created, but clicking it doesn't play the video, it leads to the File page.

Reported at w:ca.Tema:T3edcbkgt2ns31ff.