Tue, Oct 20
Yeah I'm in firefox so that explains the exception diff :)
Yep, patch seems to fix it and the new version is nice and clean. :) +2!
So if I manipulate the volume slider instead of the audio position slider, I can repro an error "Uncaught TypeError: closestHandle is undefined" but that's a different error than was reported?
Are you dragging the volume slider instead of the audio position slider? That might be doing it...
I can't reproduce this on current git master following the directions in the original post. Can you provide an exact repro?
Sorry, hadn't seen this. Sure, as far as I know it's unused for years.
Thu, Sep 24
Aug 3 2020
Worth double-checking yeah, these should be the same iirc. I'm giving it a low priority/severity because it seems relatively harmless but it should still get gotten to. :)
Note that the current audio placeholder was selected to have a similar shape and size to existing inline players to minimize disruption to article layouts.
I think I lost my work patch for the subtitles >:( Will have to recreate it shortly
I might actually prefer to delay until the remember-subtitles-setting fix is merged. I'm honestly not sure what the situation with Score is at this point, but if playback is broken that might be good to fix as well.
Jul 29 2020
Jul 27 2020
Jul 16 2020
merged and done.
Jul 14 2020
Hmm, looks like the whole test case is pretty broken:
- assumes jobs are stored in the job table and looks there for available jobs?
- calls Job::pop_type() which no longer exists
- and is all disabled since years ago
Jul 13 2020
The TimedMediaHandler support won't help here directly, except insofar as actually functionally supporting ingestion of AV1 WebM video uploads will require video transcode servers to have libaom installed, so will establish that we can install the codec library on our machines in a sensible way. :)
Jun 22 2020
I'm closing this one out as I'm retiring the incomplete but mostly working OGVKit in favor of recommending MobileVLCKit. It's more mature and better supported, and these days has better build tools than when I started.
Apr 14 2020
Apr 13 2020
The other buttons don't seem to highlight (anymore?) so closing this.
$wgMinimumVideoPlayerSize is a pretty difficult hammer to use for fine editorial control, and is not supported by the new videojs mode. The general intention is to add an explicit keyword for an 'inline' player mode which would always display in-place, rather than making it dependent on player size, though that's a separate bit of work and not yet done.
Apr 6 2020
Now works as written. :D
I presume I have to create an .env file in some other way, like using a text editor? Ideally there should be a ready-to-copy-and-paste chunk of text that requires no execution in a platform-specific shell. What should I use for MW_DOCKER_UID and MW_DOCKER_GID? UID and GID are Unix-specific concepts and I don't know if they have any meaning in Windows?
The instructions to create an .env file from a bash command fail in both PowerShell and CMD.exe... There is a mention of a /docker/install.sh file later in the instructions but I don't know where it's supposed to come from?
@kostajh trying a fresh checkout now. :)
Ok, that CSS doesn't take as it's for the post-transformed player. Looks like I need to redo how the placeholders are styled to make this work. :) Easy peasy, lemon squeezy...
Ok, the popup dialog is good but the thumbnail still doesn't get shrunk as on https://deployment.m.wikimedia.beta.wmflabs.org/wiki/Video_test
Going through the backlog of visible regressions to fix up; this may have been obsoleted by the switchover to the popup player dialog but let me check.
Ok, oddly it's started working for me even with live preview on. I don't know what changed that could have caused it to start working if it was previously broken...
Apr 2 2020
Looks like there's two things going on:
- the video preview doesn't resize to fit the gallery container (it's an actual <video> with a poster image under the lid, not an <img>, so styles might not be applying to it that need to be); this usually adds extra space around the image
- in Firefox, the video sometimes overflows the available space, causing insufficient space around the image as in the screenshot
Can't reproduce in Chrome, but can in Firefox. Something seems wonky with the gallery layout in Firefox specifically.
Ah, I see what's going on... the first loading indicator is running during the time that we load up OOUI etc, before we have a dialog, so it's just sitting on the page globally instead of being anchored anywhere in specific.
Mar 31 2020
@MichaelSchoenitzer looks like you have 'use live preview' on (labeled in English as "Show previews without reloading the page" in prefs editing tab). I can confirm that the player fails to work in the preview in this case -- it's probably not loading the required JS module -- but I haven't been able to repro the visual offset issue yet, at least in Chrome.
Thanks for the confirmation, I'll see if I can figure out which option is troublesome...
Mar 30 2020
Hmm, it's working ok for me in Chrome, Firefox, and Safari on your test page and a copy I made here: https://de.wikipedia.org/wiki/Benutzer:Brion_Vibber_(WMF)/Test
Mar 19 2020
Grabbing a couple high-prio TMH issues to work on the next week or two. Looks like a bug in the surrounding JS that assumes some things are marked up in one way but they aren't for Score stuff.
Mar 7 2020
Confirmed working on a fresh install, thanks all!
Mar 5 2020
T206957 is the related ticket.
Mar 4 2020
I think principle of least surprise leads us to want a job queue that is processed automatically without manual intervention, as it's meant to be a continuously-operating part of the MediaWiki service that code can rely on being in working condition.
Feb 25 2020
The alternative would be to replace the player 'inline', but then we'd have to jump through hoops to get subtitles for audio again.
Feb 17 2020
Feb 10 2020
From caniuse it looks like IE and old Edge are the holdouts (new Chromium based edge supports it, as well as reasonably current versions of other browsers)
Neat! We should probably confirm the behavior is consistent, predictable, and either falls back cleanly or can be emulated sanely (eg in IE 11 if it doesn't support it). Also double check that neither tag is used as an extension. Then I probably have no further objection. :)
Feb 9 2020
I think I wrote this yes... we'd consider this as GPLv2-or-later as part of MediaWiki. It wasn't originally labeled in detail as it was a one-off production hack that ended up being kept around, and we weren't as careful with explicit labeling on extensions back then.
Feb 7 2020
Patch for thumbor is in the works, needs to finish adding test cases which I'll try to do this weekend so it's ready to go out when ready.
Feb 4 2020
Feb 1 2020
From the IRC log, a couple of the transaction IDs that returned 500 errors with odd messages about Swift being missing or disabled:
Roughly planning to soft-launch this and try replacing some of the manual .ogv conversions of old mpeg1 and 2 files from scientific papers with the originals. I'll track that on a separate task once the release train brings it live.
@Aklapper it is an svg file being rendered to png, so I think that should stay unless we can confirm that rendering succeeded and the error was at a later stage of serving.
Ah fun, intermittent problems. :) Should be findable in logs ...
https://upload.wikimedia.org/wikipedia/commons/thumb/a/a2/X_mark.svg/526px-X_mark.svg.png is rendering ok for me at present. What error message is displayed for you?
Jan 31 2020
Seem to have worked around the deprecation issue by simply not shutting down the lock manager or file group, but will test more thoroughly to see if that breaks anything later...
Looks like a little bitrot has hit this patch, with the deprecation of resetServiceForTesting outside unit tests the closing of services ends up failing.
Jan 28 2020
Can you check with this page? https://brionv.com/misc/vidtest/
Jan 26 2020
This is much better than my originally suggested hack. :) +2'ing it.
Still think this'd be a great base for us to work on, should there be time and energy to spend on it.
Jan 25 2020
Hmm, https://commons.wikimedia.org/wiki/File:Wikipedia_-_Edit_2019.webm?embedplayer=yes is currently working for me in IE 11 (though with low performance) and should be working in Safari as well (with much better performance). Can you confirm whether the URL works independently of the iframe, or whether it only fails in the iframe?
Jan 17 2020
I suspect it's the file size limit yes -- ffmpeg seems to be exiting with SIGXFSZ (signal 25, exit code 153) which means 'exceeded file size'. This is probably not the file system limit, but the resource limit applied to the transcode process? However transcodes going over 4 GB may have trouble being stored anyway. :(