Page MenuHomePhabricator

Score extension no longer features TMH player for generated <audio> elements
Closed, ResolvedPublic

Description

Probably when we fixed TMH with proper module loading, we forgot about this dependency and no one has noticed (for like a year ???).

See also: https://en.wikipedia.org/wiki/Help:Score#MIDI_instruments

Event Timeline

The non-TMH player is nicer and just as functional in this case IMO.

Ebe123 raised the priority of this task from Low to High.Jan 1 2018, 12:36 AM

Audio not in TimedMediaHandler's format cannot be played in the Safari browser (using Version 11.0.2 (13604.4.7.1.3)). Seems more serious as a bug now, and TMH should formalize how to instantiate players for other extensions.

Yeah, looks like it triggers if there's also a TMH file usage on the page, but not if there isn't. Will take a peek at this, though T181875 would also solve the Safari problem.

Change 402935 had a related patch set uploaded (by Brion VIBBER; owner: Brion VIBBER):
[mediawiki/extensions/Score@master] Regression fix for TMH Score integration

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

Change 402935 merged by jenkins-bot:
[mediawiki/extensions/Score@master] Regression fix for TMH Score integration

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

Ebe123 assigned this task to brion.

Guess we can close this now!