Last modified: 2013-06-20 00:47:48 UTC
Goto special:Upload, put in a file name that's an already existing TMH file. The "This file already exists" ajax warning doesn't load the js player, and falls back to native browser <audio> or <video> element. Fairly minor issue, that may be difficult to fix, but ideally wouldn't happen.
We support dynamic loading of the player, so we should be able to fix this.
(In reply to comment #1) > We support dynamic loading of the player, so we should be able to fix this. I'm more thinking from the perspective that the interface on the MW core side is kind of klunky. OTOH, if the last remaining use of $wgAjaxExportList in core was removed and replaced with something better, I imagine that would make people happy for entirely other reasons.
MwEmbed modules include "loaders" that have some minimal code for invoking the async loading of all the components needed. i.e: we always export a $.fn.embedPlayer method[1] that can be run as soon as the video tag is in the DOM. [1] https://git.wikimedia.org/blob/mediawiki%2Fextensions%2FTimedMediaHandler/1864d1801f433562ce3e53a443d3f3196206b29a/MwEmbedModules%2FEmbedPlayer%2FEmbedPlayer.loader.js#L31
(In reply to comment #2) > (In reply to comment #1) > > We support dynamic loading of the player, so we should be able to fix this. > > I'm more thinking from the perspective that the interface on the MW core side > is kind of klunky. OTOH, if the last remaining use of $wgAjaxExportList in > core > was removed and replaced with something better, I imagine that would make > people happy for entirely other reasons. I tried to do this in Gerrit change #65641
