See the latest sync in https://gerrit.wikimedia.org/r/c/mediawiki/services/parsoid/+/776006
I'm seeing,
data-durationhint="5" data-mwtitle="Video.ogv" data-mwprovider="local"
and
data-bandwidth="590013" data-framerate="30"
See the latest sync in https://gerrit.wikimedia.org/r/c/mediawiki/services/parsoid/+/776006
I'm seeing,
data-durationhint="5" data-mwtitle="Video.ogv" data-mwprovider="local"
and
data-bandwidth="590013" data-framerate="30"
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Open | Arlolra | T305210 Parsoid's timed media output is missing data- attributes | |||
Open | Arlolra | T313875 Parsoid should probably be calling a File's MediaTransformOutput->toHtml() |
Change 933177 had a related patch set uploaded (by Arlolra; author: Arlolra):
[mediawiki/services/parsoid@master] Remove attributes deprecated in TMH
data-durationhint="5" data-mwtitle="Video.ogv" data-mwprovider="local"
These likely will remain.
Removing:
Change 959030 had a related patch set uploaded (by Arlolra; author: Arlolra):
[mediawiki/services/parsoid@master] Add durationhint to timed media
Change 959030 merged by jenkins-bot:
[mediawiki/services/parsoid@master] Add durationhint to timed media
Change 960644 had a related patch set uploaded (by Arlolra; author: Arlolra):
[mediawiki/vendor@master] Bump parsoid to 0.18.0-a25
Change 960644 merged by jenkins-bot:
[mediawiki/vendor@master] Bump parsoid to 0.18.0-a25
Note the File:Undefined from https://it.wikipedia.org/wiki/Timor_Est?useparsoid=1 while verifying T346703#9204822
<span class="mw-tmh-player audio mw-file-element"><audio preload="none" height="32" width="220" resource="./File:East_Timorese_national_anthem.wav" data-durationhint="130" class="" id="undefined_placeholder" disabled="disabled" tabindex="-1"></audio><a class="mw-tmh-play" href="/wiki/File:Undefined" title="Riproduci file multimediale" role="button"><span class="mw-tmh-play-icon"></span></a><span class="mw-tmh-duration mw-tmh-label" aria-label="2 minuti e 10 secondi">2:10</span></span>
In T133670, Parsoid implemented,
$fromFile = isset( $o['transcodekey'] ) ? '' : '-file'; if ( $hasDimension ) { $source->setAttribute( 'data' . $fromFile . '-width', (string)$o['width'] ); $source->setAttribute( 'data' . $fromFile . '-height', (string)$o['height'] ); }
but TMH does,
$prefixedSourceAttr = [ 'width', 'height', 'transcodekey', ]; foreach ( $mediaSources as &$source ) { foreach ( $source as $attr => $val ) { if ( in_array( $attr, $prefixedSourceAttr, true ) ) { $source[ 'data-' . $attr ] = $val; unset( $source[ $attr ] ); } } }
It looks like T133670 was closed before ever being implemented in TMH. Which direction should we go in consolidating? data-transcodekey is being used in resources/ext.tmh.player.inline.js, so that I will add regardless.
Change 961904 had a related patch set uploaded (by Arlolra; author: Arlolra):
[mediawiki/services/parsoid@master] Add optional data-transcodekey to media sources
Parsoid sets this as the resource attribute on the media element,
https://github.com/wikimedia/mediawiki-extensions-TimedMediaHandler/blob/master/includes/TimedMediaTransformOutput.php#L198-L203
Since that's at least present for timed media thumbs, the uses of mwtitle can probably be rewritten with that. I will look into doing that.
MultimediaViewer works by looking for the mw-file-description class. When that's present, the link around an image is pointing to the file description page so it uses the href of that link.
Change 933177 merged by jenkins-bot:
[mediawiki/services/parsoid@master] Remove attributes deprecated in TMH
Change 962706 had a related patch set uploaded (by C. Scott Ananian; author: C. Scott Ananian):
[mediawiki/vendor@master] Bump wikimedia/parsoid to 0.18.0-a26
Change 962706 merged by jenkins-bot:
[mediawiki/vendor@master] Bump wikimedia/parsoid to 0.18.0-a26
Change 961904 merged by jenkins-bot:
[mediawiki/services/parsoid@master] Add optional data-transcodekey to media sources
Change 964566 had a related patch set uploaded (by Jgiannelos; author: Jgiannelos):
[mediawiki/vendor@master] Bump wikimedia/parsoid to v0.18.0-a28
Change 964566 merged by jenkins-bot:
[mediawiki/vendor@master] Bump wikimedia/parsoid to v0.18.0-a28
So let's do another check on where we are...
Change 977278 had a related patch set uploaded (by TheDJ; author: TheDJ):
[mediawiki/extensions/TimedMediaHandler@master] Add resource attribute to video elements