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(a)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(a)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):
https://gerrit.wikimedia.org/r/#/c/172421/
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(a)lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/multimedia