EmbedPlayer only shows a blank page on Commons for video files
Closed, ResolvedPublic
Actions

Description

Hi, when opening https://commons.wikimedia.org/wiki/File:01_Editieren_und_Anmelden.ogg?embedplayer=yes the video does not load here (tested with Chromium and Firefox). What's wrong? Starting it from the file page works for me.

Best,
Martin

Details

Subject	Repo	Branch	Lines +/-
Fix regression that embedded video player didn't work	mediawiki/extensions/TimedMediaHandler	wmf/1.28.0-wmf.20	+6 -4
Fix regression that embedded video player didn't work	mediawiki/extensions/TimedMediaHandler	master	+6 -4
Fix regression that embedded video player didn't work	mediawiki/extensions/TimedMediaHandler	wmf/1.28.0-wmf.19	+6 -4
Fix regression that embedded video player didn't work	mediawiki/extensions/TimedMediaHandler	wmf/1.28.0-wmf.18	+6 -4

Customize query in gerrit

Related Objects

Mentioned Here: rETMH41fda1ba7f4f: iframe embed: Allow videojs inside iframe mode

Event Timeline

MartinRulsch created this task.Sep 20 2016, 8:14 AM

Restricted Application added subscribers: Poyekhali, Aklapper. · View Herald TranscriptSep 20 2016, 8:14 AM

DerHexer subscribed.Sep 20 2016, 8:15 AM

Steinsplitter subscribed.Sep 20 2016, 9:02 AM

Interesting; network tab says that
https://upload.wikimedia.org/wikipedia/commons/thumb/1/17/01_Editieren_und_Anmelden.ogg/mid-01_Editieren_und_Anmelden.ogg.jpg
and
https://commons.wikimedia.org/w/extensions/TimedMediaHandler/MwEmbedModules/EmbedPlayer/resources/skins/kskin/images/ksprite.png?3c85d
get loaded (HTTP 200 GET) but my screen goes blank, both in Firefox 48 and Chromium 52.

Aklapper renamed this task from EmbedPlayer does not load on Commons to EmbedPlayer only shows a blank page on Commons for video files.Sep 20 2016, 1:27 PM

Aklapper added a project: Regression.

Hi the only one that shows blank to me is https://commons.wikimedia.org/wiki/File:01_Editieren_und_Anmelden.ogg?embedplayer=yes if I go to https://commons.wikimedia.org/wiki/File:01_Editieren_und_Anmelden.ogg

It works

I'm using iPhone 6 Plus iOS 10 safari

Also @Aklapper your links work for me and they show.

• Nicolas_Rueck_WMDE subscribed.Sep 20 2016, 3:17 PM

Because of embedPlayerIframe style module, which contains:

.mediaContainer {
    visibility: hidden;
}

I suspect that the order in which the stylesheets or JS are loaded recently changed due to MediaWiki-ResourceLoader changes and that this has caused an unintended side effect...

Or maybe it was broken before, and we never noticed.... something like that.

Yeah I think this is new with the resourceloader tweaks or other recent changes, something went awry. I'm taking a peek...

@TheDJ looks like this got added in 41fda1ba7f4fc139318a9743eb1a84b9b890e6d8 ... I must not have tested it properly in mwembed mode or else it was interacting differently before. Do you remember why the visibility:hidden was there?

ah wait it was previously inline, changed by the transform.

Change 311852 had a related patch set uploaded (by Brion VIBBER):
Fix regression that embedded video player didn't work

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

gerritbot added a project: Patch-For-Review.Sep 20 2016, 10:38 PM

Change 311852 merged by jenkins-bot:
Fix regression that embedded video player didn't work

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

Change 311857 had a related patch set uploaded (by Brion VIBBER):
Fix regression that embedded video player didn't work

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

Change 311858 had a related patch set uploaded (by Brion VIBBER):
Fix regression that embedded video player didn't work

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

ReleaseTaggerBot added a project: MW-1.28-release (WMF-deploy-2016-10-04_(1.28.0-wmf.21)).Sep 20 2016, 11:00 PM

Change 311857 merged by jenkins-bot:
Fix regression that embedded video player didn't work

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

Change 311858 merged by jenkins-bot:
Fix regression that embedded video player didn't work

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

Merged fix to master; backported for SWAT deployment. Should be live shortly or already (cached pages may still be affected for a bit).

ReleaseTaggerBot added projects: MW-1.28-release (WMF-deploy-2016-09-13_(1.28.0-wmf.19)), MW-1.28-release (WMF-deploy-2016-09-06_(1.28.0-wmf.18)).Sep 21 2016, 12:00 AM

It's initially hidden, because the resource's thumbnail is initially shown, while we wait for the JS to load the player.

Since on videojs, we have the poster attribute, we don't need that for that case (it hides that #bgimage using css immediately I think...). I probably just forgot to update the mwembed js fragment that flipped the visibility flag (or missed adding it to the commit or something)..

Change 312055 had a related patch set uploaded (by Paladox):
Fix regression that embedded video player didn't work

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

Change 312055 merged by jenkins-bot:
Fix regression that embedded video player didn't work