Ok, it looks functional but there's no back-compatibility for old ParserCache renderings of pages containing media items.

This could probably be worked around by always loading a tiny stub module that checks at content-insertion / load time for the presence of transformable media and then loads the other modules needed to trigger the transform. This would might increase the FOUC effect on those pages, but they would continue to work...


If we were ready to go on VideoJS I'd say don't bother fixing this, but we're going to need some more time to swap out the player frontend properly... I think it makes sense to give just enough proper infrastructure for TMH's MwEmbedPlayer that we can make it hot-swappable via a Beta Features option, and this is a necessary step along that way.

That'll let us work on the new VideoJS stuff for early adopters and testers while limping along with existing MwEmbedPlayer with minimal breakage...

-- brion 


On Thu, Jul 23, 2015 at 8:56 AM, Brion Vibber <bvibber@wikimedia.org> wrote:
I'll take a look over it today and either merge it or recommend abandoning... :)

-- brion

On Wed, Jul 22, 2015 at 2:04 PM, Derk-Jan Hartman <d.j.hartman+wmf_ml@gmail.com> wrote:
I think we all want to ditch TMH sooner than later, but i’ve had this patch to do some cleanup to it for a while now (1,5 year):

Clearly few people are capable of reviewing it to begin with, and worse I think there are plenty of people so scared of this code, that they’d rather keep it like it is, instead of risking to break production...

So should I abandon it ?

DJ

_______________________________________________
Multimedia mailing list
Multimedia@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/multimedia