[00:17:52] 6Multimedia, 10MediaWiki-extensions-TimedMediaHandler, 10Wikimedia-Video, 5Patch-For-Review, and 2 others: Embedded video player broken - https://phabricator.wikimedia.org/T108359#1526264 (10brion) Hmm... it works in debug mode https://commons.wikimedia.org/wiki/File%3ACuriosity%27s_Seven_Minutes_of_Terror... [00:23:25] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer: Images - https://phabricator.wikimedia.org/T108651#1526289 (10erikandreolsen) 3NEW [00:26:50] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer: Images - https://phabricator.wikimedia.org/T108651#1526301 (10matmarex) What page and what image does this happen with, for example? [00:39:08] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer, 5Patch-For-Review: Test if replacing the blur effect with pixelation makes loading feel faster - https://phabricator.wikimedia.org/T71609#1526321 (10Tgr) Facebook is apparently also starting to use a Gaussian blur for image placeholders: https://code.face... [00:47:23] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer: Images - https://phabricator.wikimedia.org/T108651#1526341 (10erikandreolsen) More or less everyone at randomly. Sometimes it happens, sometimes it don't. [00:53:40] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer: Images - https://phabricator.wikimedia.org/T108651#1526350 (10matmarex) I'm asking for an example because it's rather difficult to resolve a problem one cannot see. It would be even better if you could also provide a screenshot. Are you sure it's not somet... [01:00:35] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer: Images - https://phabricator.wikimedia.org/T108651#1526374 (10erikandreolsen) {F1231238} Something like this. Whether I have ad blocking software on/off doesn't matter. It's not a problem with the internet connection either, this just started happening not... [01:04:49] 6Multimedia, 10MediaWiki-extensions-MultimediaViewer: Images - https://phabricator.wikimedia.org/T108651#1526383 (10Tgr) What kind of browser and operating system are you using? Could you open your browser's web toolbar and check for error messages (either javascript or network)? [01:06:10] 6Multimedia, 10MediaWiki-File-management, 10MediaWiki-extensions-MultimediaViewer: Images - https://phabricator.wikimedia.org/T108651#1526385 (10Tgr) p:5Unbreak!>3Normal [01:13:01] 6Multimedia, 10MediaWiki-File-management, 10MediaWiki-extensions-MultimediaViewer: Some images cannot be displayed - https://phabricator.wikimedia.org/T108651#1526412 (10Tgr) [01:13:02] 6Multimedia, 10MediaWiki-File-management, 10MediaWiki-extensions-MultimediaViewer: Some images cannot be displayed - https://phabricator.wikimedia.org/T108651#1526413 (10erikandreolsen) I use Chrome and Win 10. However, it seems like the images are showing now for some reason. But I will check for error mess... [01:13:20] 6Multimedia, 10MediaWiki-File-management, 10MediaWiki-Redirects: File redirects do not work for files with "=" in the filename - https://phabricator.wikimedia.org/T108654#1526414 (10Krenair) [01:40:18] 6Multimedia, 10MediaWiki-File-management, 10MediaWiki-extensions-MultimediaViewer: Some images cannot be displayed - https://phabricator.wikimedia.org/T108651#1526481 (10Tgr) 5Open>3Invalid a:3Tgr I am closing as invalid (could have been a temporary network problem; if it's on the server side we would... [02:53:33] 6Multimedia, 6Commons, 10MediaWiki-File-management, 10MediaWiki-Redirects: File redirects incorrectly categorized in to Commons [[Category:Files with no machine-readable license]] - https://phabricator.wikimedia.org/T108662#1526584 (10Jarekt) 3NEW [02:58:56] 6Multimedia, 6Commons, 10MediaWiki-extensions-CommonsMetadata: File redirects incorrectly categorized in to Commons [[Category:Files with no machine-readable license]] - https://phabricator.wikimedia.org/T108662#1526597 (10Krenair) [06:17:51] 6Multimedia: Ogg Vorbis has negative length displayed, cannot be played - https://phabricator.wikimedia.org/T108665#1526654 (10Derbeth) 3NEW [07:31:33] 6Multimedia, 6Commons, 10MediaWiki-extensions-CommonsMetadata: File redirects incorrectly categorized in to Commons [[Category:Files with no machine-readable license]] - https://phabricator.wikimedia.org/T108662#1526715 (10Tgr) That's weird. I would have expected [[ https://github.com/wikimedia/mediawiki-ext... [07:38:07] 6Multimedia, 10MediaWiki-File-management, 10MediaWiki-extensions-MultimediaViewer: Some images cannot be displayed - https://phabricator.wikimedia.org/T108651#1526722 (10Aklapper) For future reference, https://www.mediawiki.org/wiki/Project:Support_desk is a good plce to ask for support. [08:50:37] 6Multimedia, 10Continuous-Integration-Infrastructure, 6operations, 5Patch-For-Review: Investigate impact of switching from ffmpeg to libav (ffmpeg is not in Jessie) - https://phabricator.wikimedia.org/T103335#1526759 (10fgiunchedi) >>! In T103335#1524694, @brion wrote: > Sample command line for VP9->ogv co... [10:07:11] 6Multimedia, 10MediaWiki-General-or-Unknown, 6WMF-Design: Release thumbnail changes - https://phabricator.wikimedia.org/T77894#1526858 (10Aklapper) @Gilles: So this task is not actionable due to its vague description and should be closed? Or what does that mean? [10:08:38] 6Multimedia, 6Commons, 7Easy, 5Patch-For-Review, 7Upstream: Greyscale pngs without gAMA chunk rendered with incorrect contrast - https://phabricator.wikimedia.org/T106516#1526874 (10Aklapper) >>! In T106516#1509988, @McZusatz wrote: > As T84842 is solved, can image magick be upgraded now to a recent enou... [11:04:55] 6Multimedia, 10MediaWiki-File-management, 7Tracking: PNG thumbnails issues (tracking) - https://phabricator.wikimedia.org/T108517#1527002 (10McZusatz) The pipe for thumbnailing is a bit different for each file type, so a separate tracking ticket may be helpful. > https://commons.wikimedia.org/w/index.php?ti... [12:52:36] MatmaRex: 24 patch sets *phew* [12:52:47] MatmaRex: I finally switched to the new interface on Gerrit [12:53:33] MatmaRex: Is there going to be a FileDropWidget, that is essentially the SelectFileWidget with a larger drop zone? [12:58:34] prtksxna: we should probably add that option to SelectFileWidget [13:00:50] * prtksxna nods [13:01:51] I had a chat with Nirzar about integration with VE, but haven't updated the Phabricator task yet :( [13:02:17] MatmaRex: How do I write a plugin for VE? Couldn't find any docs on mw, but I didn't try to hard… [13:05:28] hm [13:08:03] prtksxna: this doesn't seem to actually be documented, heh. there is documentation on writing plugins as gadgets (https://www.mediawiki.org/wiki/VisualEditor_gadgets). to write them as MW extensions, it looks like you need to register your RL module in $wgVisualEditorPluginModules. examples are in Citoid and SyntaxHighlight_GeSHi [13:08:23] > DON'T PANIC [13:08:24] :D [13:08:24] prtksxna: hmm, but don't we actually want to do this in VisualEditor proper? [13:08:37] Hmmm [13:08:49] as i understand it: [13:09:15] * we have mw.Upload.Dialog in core, that provides a barebones interface suitable for small third-party wikis without weird licensing requirements [13:09:27] I am still confused about where files go and where the user thinks they go — commons vs local wiki [13:09:34] * we will have an extended upload dialog in UploadWizard, that puts in the million boxes required for commons [13:10:04] * we will have some integration in VisualEditor, that will use UploadWizard's or core's upload dialog. [13:10:21] hmm. [13:10:52] And marktraceur pointed out that the user sometimes would want to explicitly upload to the local wiki due to some licensing stuff I don't understand. [13:12:38] yeah, true. i hadn't considered that [13:15:27] so… i guess we need to make sure that mw.Upload works for cross-wiki uploads, and add a selector to mw.Upload.Dialog to let the user choose whether to upload locally, or to foreign repo, if any is confugred [13:16:40] i'm not sure if we can do this magically. there is $wgForeignFileRepos… [13:16:59] and the entries can have a 'apibase' property… so maybe [13:17:37] How often do users make that choice? Is it a power user thing? Can we hide this complexity from the others? Maybe James_F|Away knows. [13:20:22] prtksxna: afaik for Wikimedia there are two cases, first is users who think Commons people are generally dicks and want nothing to do with them, and second is users who want to upload copyrighted fair-use media to the few Wikipedias that allow them (including en.wp) [13:20:33] largely things like movie posters, logos, book covers [13:21:14] it should probably default to Commons. i'm not sure how visible the option should be. [13:22:01] So in the license picker we should be asking them if this is copyrighted fair-use media and silently switching them over to local wiki (if allowed). [13:23:12] hmm. that sounds neat [13:23:30] s/./? [13:24:27] yeah, that actually sounds like a good idea to me, but we should check iwth marktraceur in case that's not possible for some bizzarre reason [13:25:17] we just need to come up with a sane way to separate this between core, VisualEditor and UploadWizard [13:25:41] I think we'll move to the UW dialog next [13:25:57] 6Multimedia, 10MediaWiki-File-management, 6Performance-Team: Add support for using graphicsmagick instead of imagemagick for thumbnailing - https://phabricator.wikimedia.org/T21073#1527224 (10Jarry1250) Krinkle: is the plan to use testing to make sure this definitely is a perf benefit? [13:26:33] The VE plugin will in some cases skip the first, and last step. The file might get dropped anywhere and call the dialog, and once the save is complete we can switch back to VE instead of showing the stupid usage panel. [13:30:56] we should be putting notes somewhere [13:31:01] do we have tasks for this all? [13:33:45] MatmaRex: There is a generic task to integrate UW and VE. I'll update it. [13:34:13] https://phabricator.wikimedia.org/T40030 ? [13:35:15] MatmaRex: Yes [14:10:00] 6Multimedia, 10MediaWiki-File-management, 7Tracking: PNG thumbnails issues (tracking) - https://phabricator.wikimedia.org/T108517#1527321 (10Aklapper) > The pipe for thumbnailing is a bit different for each file type, so a separate tracking ticket may be helpful. Hmm, for who specifically //will// it be hel... [14:14:04] 6Multimedia, 6Commons, 10MediaWiki-extensions-CommonsMetadata: File redirects incorrectly categorized in to Commons [[Category:Files with no machine-readable license]] - https://phabricator.wikimedia.org/T108662#1527340 (10RP88) For what it is worth, I saw "Z 0002 7925.jpg" (i.e. https://commons.wikimedia.or... [14:22:09] 6Multimedia, 10MediaWiki-File-management: Ogg Vorbis has negative length displayed, cannot be played - https://phabricator.wikimedia.org/T108665#1527360 (10Aklapper) [14:22:16] 6Multimedia, 10MediaWiki-File-management: Ogg Vorbis has negative length displayed, cannot be played - https://phabricator.wikimedia.org/T108665#1527365 (10Aklapper) p:5Triage>3Low [14:24:13] * marktraceur reads up [14:26:47] MatmaRex, prtksxna: OK, let's see if I can clarify what I have in my head and if it makes any sense to you guys [14:27:59] MatmaRex, prtksxna: My current priority is to get a tool into VisualEditor, and secondarily WikiEditor, that uploads a file from enwiki to Commons, with a default license choice, and sends the user to WP:FUW or COM:UW if someone doesn't like it. [14:28:31] MatmaRex, prtksxna: I believe we can add licensing choices or whatever else (local vs. remote choice) in later iterations, I don't think they're necessary for the base case we're trying to address [14:29:22] MatmaRex, prtksxna: I think the Commons-specific mw.Upload and mw.Upload.Dialog subclasses will go into CommonsMetadata, because I dislike the idea that UploadWizard is Commons-specific, and I even more hate the idea of then loading it outside of Commons [14:30:40] MatmaRex, prtksxna: Those classes will add fields and getters/setters for descriptions (the model will have support for multilingual, I'm not sure we ever came to a decision about that in the UI), licenses (in text form, it has no clue about them though), categories, date, locations, and so on [14:31:08] Basically "mw.CommonsUpload" is a euphemism for "{{information}}-enabled upload model class" [14:31:27] As well as hard-coding the mw.Api URL to commons.wikimedia.org [14:32:24] I haven't yet tried to subclass mw.Upload.Dialog to add fields. That somewhat concerns me but I'll go do that now I think [14:33:15] prtksxna, MatmaRex: Also I definitely think that the upload dialog can go into VE-mediawiki at the end [14:33:42] Maybe we could make that more palatable by putting CommonsUpload into core, and not calling it CommonsUpload but MediaRepoUpload or something [14:35:40] hmm. [14:39:26] marktraceur: there is some precedent for hardcoding Commons in core, like $wgUseInstantCommons, so i think it could go in core. from the technical side, the only difference between uploading to Commons and any other foreign repo is just the API URL, right? [14:40:03] marktraceur: we might want to separate these concepts. a local upload vs foreign upload, and a simple form vs {{information}}-enabled form. [14:41:30] i haven't tried to subclass mw.Upload.Dialog either, but it looked well-engineered for subclassing. you override some render*() and get*() methods, and presto. [14:42:19] Urgh. InstantCommons. [14:42:47] marktraceur: soo… we just put everything in core, and say "foreign file repository" when we really mean "Commons"? ;) [14:42:57] Something like that [14:43:02] It's what we've always done! [14:43:07] i think it's sensible [14:43:14] more than splitting the code over four places [14:43:16] MatmaRex: "foreign file repository" almost universally means Commons, anyway [14:43:26] At least in the context of the things that we're writing [14:43:39] Wikia won't worry about it, their uploader is pretty much OK [14:44:05] Uncyclomedia Commons is a thing. Shouldn't be too hard for Isarra to hack to make it work. :P [14:44:41] MatmaRex: I'll explore the code again with an eye towards refactoring, maybe we can get the dialog merged today at least [14:44:58] mw.Upload.Dialog? i just merged it. :P [14:45:18] Oh. [14:45:24] Well shit! [14:45:29] Exciting times [14:46:01] prtksxna: What's this you say about a new Gerrit interface [14:53:53] 6Multimedia, 10MediaWiki-File-management, 7Tracking: PNG thumbnails issues (tracking) - https://phabricator.wikimedia.org/T108517#1527512 (10McZusatz) Not sure what the devs prefer, but I can put them back in the general tracking bug anytime, if that's preferred? [15:53:04] 6Multimedia, 10MediaWiki-File-management, 7Tracking: PNG thumbnails issues (tracking) - https://phabricator.wikimedia.org/T108517#1527688 (10Aklapper) >>! In T108517#1527512, @McZusatz wrote: > Not sure what the devs prefer That's something to first find out then, I'd say. :) I simply don't see why people s... [16:21:30] marktraceur: o/ [16:22:27] marktraceur: Something MatmaRex showed me last time we met. See "Change View" in Gerrit preferences :) [16:22:33] Takes a while to get used to [16:22:46] Don't mind if I start +1ing my patches by accident :P [16:23:30] 6Multimedia, 6Commons, 10MediaWiki-extensions-CommonsMetadata: File redirects incorrectly categorized in to Commons [[Category:Files with no machine-readable license]] - https://phabricator.wikimedia.org/T108662#1527771 (10Jarekt) RP88, you are right I did a small edit to Z 0002 7928.jpg (removed \n) and the... [16:28:10] Damn [16:29:25] prtksxna: That's way better [16:50:32] * prtksxna still hasn't responded to 40300 [16:51:15] I have everything in a google doc though :\ [16:51:25] ugh [16:51:33] I am going to sleep and respond tomorrow. [16:51:48] I am done with the Hovercards stuff btw, just need put stuff up on Phab and Gerrit [16:51:57] OK cool. [16:52:17] ┌[；`||～,]┐ z z Z [17:16:44] 6Multimedia, 10Continuous-Integration-Infrastructure, 6operations, 5Patch-For-Review: Investigate impact of switching from ffmpeg to libav (ffmpeg is not in Jessie) - https://phabricator.wikimedia.org/T103335#1527982 (10brion) Made an attempt to sidestep ffmpeg2theora by using ffmpeg for conversion and ogg... [17:48:04] 6Multimedia, 10MediaWiki-extensions-ImageMetrics: Use Cache-Control over cache-busting query args if possible, please - https://phabricator.wikimedia.org/T108707#1528125 (10BBlack) 3NEW [17:49:32] 6Multimedia, 10MediaWiki-extensions-ImageMetrics: Use Cache-Control over cache-busting query args if possible, please - https://phabricator.wikimedia.org/T108707#1528149 (10BBlack) [18:17:01] 6Multimedia, 10MediaWiki-File-management, 10MediaWiki-Redirects: File redirects do not work for files with "=" in the filename - https://phabricator.wikimedia.org/T108654#1528331 (10Jarekt) The file page of redirect pages shows categories in the top and "File history", "File usage on Commons", "Metadata" and... [18:20:06] 6Multimedia, 3Reading-Web, 3Wikipedia-Android-App, 3Wikipedia-iOS-App-Product-Backlog: In the news image doesn't invoke media viewer - https://phabricator.wikimedia.org/T100336#1528343 (10Dbrant) 5Open>3Resolved a:3Dbrant [18:59:39] 6Multimedia, 10Continuous-Integration-Infrastructure, 6operations, 5Patch-For-Review: Investigate impact of switching from ffmpeg to libav (ffmpeg is not in Jessie) - https://phabricator.wikimedia.org/T103335#1528512 (10brion) Ok I've got a provisional patch for ffmpeg2theora master, which gets a local bui... [18:59:44] 6Multimedia, 10MediaWiki-extensions-TimedMediaHandler, 6operations: Support VP9 in TMH (Unable to decode) - https://phabricator.wikimedia.org/T55863#1528513 (10brion) Ok I've got a provisional patch for ffmpeg2theora master, which gets a local build of ffmpeg2theora working in MediaWiki-Vagrant for me. https... [19:38:16] 6Multimedia, 10MediaWiki-extensions-ImageMetrics, 6Reading-Infrastructure-Team: Use Cache-Control over cache-busting query args if possible, please - https://phabricator.wikimedia.org/T108707#1528770 (10bd808) [21:19:19] 6Multimedia, 10MediaWiki-extensions-TimedMediaHandler: Line breaks on subtitle are ignored on TimedText namespace - https://phabricator.wikimedia.org/T78511#1529257 (10brion) 5duplicate>3Open This seems unrelated to T55926 which is about a specific user interface message. This is about subtitles. [21:23:10] 6Multimedia, 10MediaWiki-extensions-TimedMediaHandler: Line breaks on subtitle are ignored on TimedText namespace - https://phabricator.wikimedia.org/T78511#1529267 (10brion) Example: This text on https://commons.wikimedia.org/wiki/TimedText:Knowledge_for_Everyone_(no_subtitles).webm.ja.srt is explicitly brok... [21:27:18] i see 2 problems, one, wikitext parser is adding

all over the place. This we can probably fix by setting the same nowiki option as we set with syntaxhighlight recently [21:27:59] 2nd problem is the line breaking. but if we have linebreaks consisted due to the first thing, then it will be much easier to replace \n with
basically [21:28:28] but we should really just move it over to vtt, so i'm not sure if there is enough value short term. [21:29:17] well it shouldn't be wikitext-parsing srt source [21:29:19] i would hope [21:29:26] it is [21:29:37] if i remember correctly [21:29:59] haha [21:30:18] this was all way back before we had multiple content models and nowiki flags at the parser level [21:31:19] *ugh* [21:31:28] https://commons.wikimedia.org/w/api.php?action=parse&page=TimedText%3AKnowledge_for_Everyone_(no_subtitles).webm.ja.srt&smaxage=3600&maxage=3600&format=json [21:31:39] see

around every double new line [21:31:53] yeah i was just seeing that in my debug console [21:31:58] must... kill... [21:34:00] $parser->mStripState->addNoWiki( $marker, $out ); [21:34:53] that should probalby do it. And probably we will have to switch the client side parser somewhere (hopefully it can detect it somehow) [21:37:25] 6Multimedia, 10Analytics, 10MediaWiki-extensions-ImageMetrics, 5Patch-For-Review: Measure how many users have CORS-hostile proxies - https://phabricator.wikimedia.org/T507#1529416 (10Tgr) [21:37:37] but they also contain categories and license templates etc, so you probably have to preparse to make sure inclusion tracking works at the database level. Another thing that Syntaxhighligher does as well I believe [21:38:13] // Remove all html tags for security reasons [21:38:14] srt = srt.replace(/<[a-zA-Z\/][^>]*>/g, ''); [21:38:28] that explains why explicit
didn't work i guess [21:39:35] hmm, this looks like it should almost work: [21:39:40] caption['content'] = s.slice(2).join("
"); [21:41:31] https://phabricator.wikimedia.org/diffusion/ESHG/browse/master/SyntaxHighlight_GeSHi.class.php;29f705b2d0471de9e22301b8b5343e3a05904f18$169 [21:42:00] https://phabricator.wikimedia.org/diffusion/ESHG/browse/master/SyntaxHighlight_GeSHi.class.php;29f705b2d0471de9e22301b8b5343e3a05904f18$404 [21:42:13] ah yeah i see the parser is removing the spaces (or the html tidy step maybe) [21:42:18] s/spaces/newlines/ [21:42:27] so they never get turned into
s in the output [21:42:39] (the output from TextSource) [21:42:45] 6Multimedia, 10MediaWiki-extensions-ImageMetrics, 10MobileFrontend, 3Reading-Web: cors-test.js 404s on mobile on beta labs - https://phabricator.wikimedia.org/T108206#1529458 (10Tgr) This seems to be a problem with MobileFrontend; the url works if I remove `.m` from the domain name. [21:44:37] 6Multimedia, 10MediaWiki-File-management, 10MediaWiki-Redirects: File redirects do not work for files with "=" in the filename - https://phabricator.wikimedia.org/T108654#1529479 (10Tgr) Is this different from T108662? [21:45:22] thedj: so i guess the question is do we need to do any preparsing of the .srt file (eg, why the heck is it using action=parse on them anyway?) [21:45:35] cause if we don't, then we should just fetch the source :) [21:45:48] but ... if people are using things like templates in the text, that might break that [21:46:08] 6Multimedia, 10MediaWiki-extensions-ImageMetrics, 10MobileFrontend, 3Reading-Web: cors-test.js 404s on mobile on beta labs - https://phabricator.wikimedia.org/T108206#1529487 (10Jdlrobson) It seems to be a redirect but the redirect isn't setup on the m domain. [21:46:14] there are templates [21:46:19] i'm just not sure how many [21:46:23] and where [21:46:49] i think this also used to work with action=raw initially [21:47:14] and then there work requests for limited html styling (allowed in srt) and links so it became parsed I think [21:47:49] and then someone said: "yikes, we aren't validating these files and injecting it into the dom, and the whole html was dropped again I think [21:51:38] 6Multimedia, 10MediaWiki-extensions-TimedMediaHandler: Line breaks on subtitle are ignored on TimedText namespace - https://phabricator.wikimedia.org/T78511#1529525 (10brion) So here's what seems to be happening: * TMH's TimedText plugin fetches the blah.srt page from the wiki's API with an action=parse * so,... [21:53:09] ok i think that makes some kind of sense :D [21:55:12] ok that's enough TMH for one day [21:55:15] * brion TMH tuesday! [21:56:56] i've got a fedora install to babysit.... be back later :D [21:56:58] TuesdayMediaHandler [21:57:18] if it's tuesday it must be [[Category:Videos about Belgium]] [22:02:43] (03CR) 10Legoktm: [C: 04-1] "GlobalUserPage can get embedded images from action=parse (https://en.wikipedia.org/w/api.php?action=parse&text={{User:Legoktm}}), and we c" (031 comment) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/228751 (https://phabricator.wikimedia.org/T105986) (owner: 10Gergő Tisza) [22:03:58] 6Multimedia, 10MediaWiki-extensions-GlobalUsage: GlobalUsage at testwiki shows redlinks instead of wiki names - https://phabricator.wikimedia.org/T108754#1529605 (10Quiddity) 3NEW