[00:00:59] fabriceflorin: would it be OK to use a single "author and source" tooltip instead of the separate "author" and "source" tooltips that we have currently? [00:01:15] tgr [00:01:24] where is the staff/contractor wiki? [00:01:35] neilk_: You mean the page? [00:01:38] tgr: Yes, I think it would be OK, since they are technically one big field anyway. Thanks for asking :) [00:01:43] neilk_: http://wikimediafoundation.org/wiki/Staff_and_contractors?showall=1 [00:01:45] marktraceur: help with setting up mail and things [00:01:49] Oh hm. [00:01:50] neilk_: office.wikimedia.org? [00:01:55] Yeah, probably [00:02:01] Or cndiv just walks you through it [00:02:25] tgr: The proposed wording would then be: “Click to see full author and source information” [00:03:29] fabriceflorin: yeah, that part is in the spec, what I am saying is to change the tooltip on the untruncated version to "author and source information" to match it [00:04:43] tgr: Oh, thanks for the clarification. Yes, I also think that would be fine for now. [01:36:55] If anyone is around I have dumb questions about reviewing [01:37:29] Like, I'm assuming it should not actually download >4MB of stuff when reviewing a tiny change, so obviously something is wrong [01:38:21] also I'm finding that my repo URL was changed (by the review tool?) to https://gerrit.wikimedia.org, shouldn't it be the ssh:// url? [01:39:52] also also the docs are inconsistent, do I copy the incantation from Gerrit, or use git review -d ? [01:42:57] hm, ok I don't know how the repo URL was changed to http, but changing it back manually to ssh: worked. [01:43:17] (03PS10) 10Gergő Tisza: Reveal full text when user clicks on title etc. [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/144086 [01:43:28] (03CR) 10Gergő Tisza: "PS10: tooltips (tweaked the text a bit per IRC discussion)" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/144086 (owner: 10Gergő Tisza) [01:43:31] (03CR) 10jenkins-bot: [V: 04-1] Reveal full text when user clicks on title etc. [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/144086 (owner: 10Gergő Tisza) [01:43:58] (03CR) 10Gergő Tisza: "Or maybe it was a mail discussion. Whatever." [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/144086 (owner: 10Gergő Tisza) [02:50:47] neilk_: I use `git review --download` with no issues to download reviews. [03:08:18] bd808: I solved it. I have no idea how this happened but my repos slipped out of the right configuration. I hate it when I shift into a parallel universe [03:09:10] neilk_: welcome. We're all crazy here. :) [03:43:33] (03CR) 10Neilk: [C: 04-1] "ui.core should be retained. UploadWizard generates HTML elements that use jQuery UI classes, such as "ui-corner-bottom". So it's a direct " [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/144751 (owner: 10MarkTraceur) [07:50:46] 3MediaWiki extensions / 3UploadWizard: Chunked upload fails with internal_api_error_UploadStashFileNotFoundException - 10https://bugzilla.wikimedia.org/36587#c79 (10Sisa) Using the bigChunkedUpload.js to upload a new version (345.142KB) of https://commons.wikimedia.org/wiki/File:Clusiodes_-_2014-07-06kl.AVI.... [08:19:17] 3MediaWiki / 3File management: Error creating thumbnail: Multiple SVG files are only allowed for PDF and (E)PS output. - 10https://bugzilla.wikimedia.org/67733#c6 (10Nemo) (In reply to Bawolff (Brian Wolff) from comment #5) > To clarify, you're using rsvg-convert? (Its not the default). Looks so: https://gi... [08:20:17] 3MediaWiki / 3File management: Error creating thumbnail: Multiple SVG files are only allowed for PDF and (E)PS output. - 10https://bugzilla.wikimedia.org/67733#c7 (10Nemo) Rillke's config has a "-o" more though. [08:28:08] (03PS1) 10Pginer: Adjustments to repo icon [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/145248 [08:28:46] (03CR) 10jenkins-bot: [V: 04-1] Adjustments to repo icon [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/145248 (owner: 10Pginer) [12:02:42] (03CR) 10Gilles: [C: 032] Android native browser ending playback after 1 second [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/144352 (https://bugzilla.wikimedia.org/43748) (owner: 10Brian Wolff) [12:02:53] (03Merged) 10jenkins-bot: Android native browser ending playback after 1 second [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/144352 (https://bugzilla.wikimedia.org/43748) (owner: 10Brian Wolff) [12:05:58] (03CR) 10Gilles: "recheck" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/145248 (owner: 10Pginer) [12:16:46] 3MediaWiki / 3File management: Error creating thumbnail: Multiple SVG files are only allowed for PDF and (E)PS output. - 10https://bugzilla.wikimedia.org/67733 (10Andre Klapper) p:5Unprio>3Normal [12:18:01] 3MediaWiki extensions / 3UploadWizard: Chunked upload fails with internal_api_error_UploadStashFileNotFoundException - 10https://bugzilla.wikimedia.org/36587#c80 (10Andre Klapper) Whatever "bigChunkedUpload.js" is, this bug report is about UploadWizard instead... [12:33:51] (03CR) 10TheDJ: "I notice a similar problem with:" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/139592 (https://bugzilla.wikimedia.org/66366) (owner: 10Rillke) [12:39:47] 3MediaWiki / 3Uploading: Upload broken after upgrade to 1.23 - 10https://bugzilla.wikimedia.org/66467#c23 (10Andre Klapper) *** Bug 67744 has been marked as a duplicate of this bug. *** [12:41:17] 3MediaWiki / 3Uploading: Upload broken after upgrade to 1.23 - 10https://bugzilla.wikimedia.org/66467#c24 (10Andre Klapper) The last comment does not make any sense, and I have already explained in bug 67744 how to find the patch / code fix for this problem. [12:47:46] 3MediaWiki extensions / 3UploadWizard: Chunked upload fails with internal_api_error_UploadStashFileNotFoundException - 10https://bugzilla.wikimedia.org/36587#c81 (10Rainer Rillke @commons.wikimedia) (In reply to Andre Klapper from comment #80) > Whatever "bigChunkedUpload.js" is, this bug report is about Upl... [12:48:54] (03CR) 10Gilles: "recheck" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/145248 (owner: 10Pginer) [13:21:12] (03CR) 10Rillke: "> This convinces me that we should move this logic into a core script." [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/139592 (https://bugzilla.wikimedia.org/66366) (owner: 10Rillke) [13:24:38] thedj, before I am going into the merge-conflict-hell again I'd like to get https://gerrit.wikimedia.org/r/131971 reviewed [13:25:47] 3MediaWiki / 3File management: Chunked upload fails with internal_api_error_UploadStashFileNotFoundException - 10https://bugzilla.wikimedia.org/36587 (10Marco) [13:26:23] (03CR) 10Gilles: "The QUnit failure is caused by ea307eafa3b655b612f45a4a279c1010ae633466" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/145248 (owner: 10Pginer) [13:28:47] 3MediaWiki / 3File management: Chunked upload fails with internal_api_error_UploadStashFileNotFoundException - 10https://bugzilla.wikimedia.org/36587#c82 (10Rainer Rillke @commons.wikimedia) (In reply to Sisa from comment #79) Sisa, do you remember 1) how long it took uploading the 82 chunks 2) when you were... [13:33:20] (03CR) 10Gilles: [C: 032 V: 032] Adjustments to repo icon [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/145248 (owner: 10Pginer) [13:33:59] (03CR) 10jenkins-bot: [V: 04-1] Adjustments to repo icon [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/145248 (owner: 10Pginer) [13:46:08] (03CR) 10Gilles: "recheck" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/145248 (owner: 10Pginer) [13:56:51] (03CR) 10Gilles: [C: 032] Adjustments to repo icon [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/145248 (owner: 10Pginer) [13:58:50] (03Merged) 10jenkins-bot: Adjustments to repo icon [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/145248 (owner: 10Pginer) [14:04:32] 3MediaWiki / 3File management: Chunked upload fails with internal_api_error_UploadStashFileNotFoundException - 10https://bugzilla.wikimedia.org/36587#c83 (10Sisa) (In reply to Rainer Rillke @commons.wikimedia from comment #82) > (In reply to Sisa from comment #79) > Sisa, do you remember > 1) how long it too... [14:12:46] 3MediaWiki / 3File management: Tiff image with jpeg compressed data has incorrect colours (image magick needs upgrading?) - 10https://bugzilla.wikimedia.org/63907#c4 (10Jarek Tuszynski) Another file with that issue: https://commons.wikimedia.org/wiki/File:Herb_%26_Jan_Conn_-_Unclimbed_-1_-_1959_July_-_1.tif [14:25:37] rillke: got it... [14:39:48] * marktraceur wfhing today! Huzzah [15:15:29] marktraceur: if we use a GPLv3 library for UploadWizard, does that mean UploadWizard itself will have to be v3? I'm clueless about these matters [15:15:42] I just want to check whether FineUploader has that implication or not [15:16:08] Hm [15:16:21] gi11es: I think that's roughly the case, but there could be some argument about it [15:16:31] It depends [15:16:49] heh, the answet to every legal question ever, I guess [15:16:53] *answer [15:16:57] Yeah [15:17:07] I don't have the explanation of why. [15:17:35] if we have to, would that be a problem? i.e. can the license change without the input from past contributors [15:17:55] Hmmm [15:19:35] gi11es: Sadly, the UW license does not explicitly mention "or later", but the MediaWiki one does [15:19:40] neilk should be ashamed [15:20:23] Yuvi's files from the campaigns rewrite have "or later", but nothing else it seems [15:20:31] This is iffy [15:20:40] Not impossible, just iffy [15:21:26] what would the process be? get every past contributor to agree to the change? [15:21:51] First determine if a lack of explicit license for GPLv3 is a problem, ask...hm [15:22:15] Yeah, it's Luis [15:22:27] Also ask if we need to worry about shifting. [15:22:40] Depending on how we do it, we might not need to relicense [15:22:49] As nice as it would be :3 [15:27:15] alright, email sent. so far it seems like FineUploader is the better option, so... [15:27:30] gi11es: How's that? [15:27:41] much better test coverage and documentation [15:27:50] more modular, more complete [15:30:11] and fineuploader's development is a lot more active [15:30:18] Cool. [15:30:19] see: https://github.com/Widen/fine-uploader/graphs/commit-activity versus https://github.com/blueimp/jQuery-File-Upload/graphs/commit-activity [15:31:23] Hm, looks like FU gets some big sprints, but they're dwindling [15:31:51] a lot more contributors too https://github.com/Widen/fine-uploader/graphs/contributors https://github.com/blueimp/jQuery-File-Upload/graphs/contributors [15:33:11] Wow. [15:33:33] And it's not like it's all Widen employees, there aren't many of them [15:33:48] i'm guessing the recent slowdown could just be the summer effect? [15:33:56] the other project is equally slow over that period [15:35:49] Might could be [15:47:32] 3MediaWiki / 3File management: #ifexist does not say exist to existing redirects to file when using media pseudo-namespace - 10https://bugzilla.wikimedia.org/32031 (10Bawolff (Brian Wolff)) 5PATC>3RESO/FIX [15:49:02] 3MediaWiki extensions / 3TimedMediaHandler: Only first few seconds of video play on android phone using android's native browser - 10https://bugzilla.wikimedia.org/43748 (10Bawolff (Brian Wolff)) 5PATC>3RESO/FIX [16:04:34] tgr: Does this feedback thing include points for whatever the feedback suggests? [16:06:16] marktraceur: what I had in mind was not really feedback but a forced opt-in/opt-out choice [16:06:32] not sure if Fabrice meant the same thing [16:10:58] Ah, hm. [16:11:23] "methodology" and "acceptance" suggest not [16:18:12] I think acceptance would just mean deciding with the community what's the thresholds for keeping it on by default [16:18:48] gi11es: Right, but tgr is basically saying putting the current "disable" link above the fold, one time [16:20:12] not really, a yes/no/ask later thing would be much different than the current disable [16:20:40] Right [16:20:52] It was the closest analogy pre-caffeine-has-soaked-in Mark had. [16:21:34] it's an interesting proposal, but I fear the acceptance criteria as defined by Media Viewer's detractors wouldn't be very reasonable [16:22:35] Oh definitely not. [16:23:01] Since the criteria appear to be "There are no criteria" [16:23:17] Alternatively "Spend another 6 months developing features, but don't spend any money on it" [16:39:49] (03PS1) 10Steinsplitter: Adding new licensing template to UploadWizard [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/145358 [16:41:19] (03CR) 10MarkTraceur: [C: 04-1] "> Do not modify this file, instead use localsettings.php and set..." [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/145358 (owner: 10Steinsplitter) [16:42:42] (03PS2) 10Steinsplitter: Adding new licensing template to UploadWizard [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/145358 [16:43:12] (03CR) 10MarkTraceur: [C: 04-1] "Still" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/145358 (owner: 10Steinsplitter) [16:45:03] (03Abandoned) 10Steinsplitter: Adding new licensing template to UploadWizard [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/145358 (owner: 10Steinsplitter) [17:14:46] tgr: What came out of that meeting? [17:14:54] The seeeekrit one. [17:15:10] not much new [17:15:12] Answer probably includes "There will be an email"? [17:16:34] I'm sure there will be emails, yes [17:16:56] but basically the end result was "don't do anything for now, see what happens" [17:17:09] Woohoo. [17:17:31] I mean, Fabrice will post an answer, but it will say we don't do anything [17:17:49] K [17:18:01] Ambivalence is always the solution [17:27:09] marktraceur: right now the dashboard data is generated by limn1 making a mysql connection to analytics-store using a saved password from a config file, right? [17:29:35] tgr: Yeah, I think so [17:29:47] milimetric says we should redo that [17:30:11] so that no important password is stored on any labs instance [17:30:24] No labs instance is involved in generation right now [17:30:28] Oh, wait. [17:30:34] tgr: Not limn1, stat1001 [17:31:04] cool, no problem then [17:31:16] how does the data get from stat1001 to limn1? [17:31:20] rsync? [17:31:36] Naw, HTTP [17:31:43] stat1001 has a public directory [17:31:53] http://stat1001.wikimedia.org/public-datasets/all/multimedia/ [17:35:34] (03PS3) 10Neilk: Remove unnecessary dependencies [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/144751 (owner: 10MarkTraceur) [17:37:18] (03CR) 10Neilk: "At first I thought I couldn't submit patches of my own if I wasn't a module owner. Then I realized I can, so, this last patch addresses th" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/144751 (owner: 10MarkTraceur) [17:39:06] (03CR) 10MarkTraceur: [C: 031] Remove unnecessary dependencies [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/144751 (owner: 10MarkTraceur) [17:40:18] marktraceur: what was the conclusion of the last discussion we had about jquery.hashchange? [17:40:26] Uhhhhgh. [17:40:31] I think I recall that you and gergo discussed alternatives [17:40:36] was there a conclusion? [17:42:33] I can't recall any [17:50:14] Yeah I feel like we just agreed it was all terrible. [17:50:35] gi11es: Probably just using a fork of it that has the issues fixed would be OK [17:51:48] marktraceur: updated the dashboard architecture overview, can you glance at it and see if there are any errors? [17:51:52] https://www.mediawiki.org/wiki/Multimedia/Media_Viewer/Metrics/Architecture [17:53:00] I wonder if we even need hashchange at this point [17:53:26] since we are handling History API compatible browsers manually anyway [17:53:32] and don't support IE6 [17:53:59] Keegan|Away marktraceur tgr Eloquence robla : As discussed, I just responded to the Media Viewer RfC on English Wikipedia, based on everyone’s recommendations: https://en.wikipedia.org/wiki/Wikipedia_talk:Media_Viewer/June_2014_RfC#Response_to_the_Media_Viewer_RfC [17:54:32] I also posted this update on our MediaWiki talk page for Media Viewer: https://www.mediawiki.org/wiki/Talk:Multimedia/About_Media_Viewer#Response_to_the_Media_Viewer_RfC_on_English_Wikipedia [17:54:51] * tgr prepares the popcorn [17:55:57] Should I send the same message to the Multimedia list next? Or wait a few hours? (speaking of which, I will be out of office until 2pm PT, so I will miss the first responses, but hope you enjoy the show :) [17:58:18] well, there wasn't any discussion on that list about the RfC so far, I'm not sure it needs to be sent there [17:59:30] tgr: OK, I’ll wait until this afternoon then. [17:59:51] fabriceflorin: Why send it at all? [17:59:57] This is an enwiki thing. [18:00:19] marktraceur: It’s part of being transparent about this major decision on our part. [18:00:42] It's a major enwiki decision, though, it's not a mediawiki multimedia one. [18:05:40] fabriceflorin: wikimedia-l OTOH does have a discussion about the RfC [18:05:53] you might want to announce it in that thread [18:26:01] (03PS1) 10Gilles: Use the new .bindQueueCallback() instead of .bind() [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/145393 (https://bugzilla.wikimedia.org/65428) [18:26:34] (03CR) 10Gilles: "Depends on: https://gerrit.wikimedia.org/r/#/c/145391/" [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/145393 (https://bugzilla.wikimedia.org/65428) (owner: 10Gilles) [18:27:21] That didn't take long [18:27:39] perhaps the administrators should implement the consensus in a manner similar to how they did with the Visual Editor [18:27:44] From an IP. [18:28:48] 3MediaWiki extensions / 3TimedMediaHandler: MwEmbedSupport and TimedMediaHandler require $wgIncludejQueryMigrate = true; ($.browser) - 10https://bugzilla.wikimedia.org/66359 (10Gilles Dubuc) a:3Gilles Dubuc [18:28:49] 3MediaWiki extensions / 3TimedMediaHandler: Fix "JQMIGRATE: Use of jQuery.fn.data('events') is deprecated" in TimedMediaHandler - 10https://bugzilla.wikimedia.org/65428 (10Gilles Dubuc) a:3Gilles Dubuc [18:30:44] damn the man! [18:33:28] mmm, seems like core's jquery UI is still spewing jqmigrate warnings [18:33:53] has jquery UI not been updated yet to be the latest and greatest? [18:34:04] "greatest", rather [18:35:22] tgr: http://caniuse.com/history are you sure we don't need hashchange for ie8 & 9 as well? [18:37:29] gi11es: nut sure at all, but AFAIK navigation libraries like hashchange do history.pushState on modern browsers, location = '#...' on old-ish IE and crazy iframe magic on really old IE [18:37:35] the last being the hard part [18:38:26] since we already handle the pushState part manually, and don't support IE6 anyway, we could just replace hashchange calls with location=... [18:38:28] the comments in hashchange says ie6/7/8 for that section [18:39:06] // Not only do IE6/7 need the "magical" Iframe treatment, but so does IE8 when running in "IE7 compatibility" mode. [18:39:35] uh [18:39:47] we do need it for IE8 then [18:40:15] which we sort of half-support now I guess? [18:40:38] since the check is $.browser.msie && !supports_onhashchange I'm going to check if checking for IE-ness is even necessary [18:40:52] i.e. would any browser we support report !supports_onhashchange and not need the iframe treatment [18:41:01] the use panel won't work because it's OOJS UI but it's lazy-loaded so won't prevent the main app from loading (?) [18:42:08] I haven't dared try. it depends on whether our browser compatibility test runs after the es5 shim is loaded, though [18:42:19] which I don't think anyone has verified [18:46:30] ffs [18:46:56] the official IE8 browser testing VM published by Microsoft asks for a product key [18:47:03] HAhaha [18:47:10] it does after a while [18:47:18] you need to freeze it in time if you don't want that to happen [18:47:25] i.e. make a snapshot the day you install it [18:47:36] it's what they actually recommend doing [19:02:35] (03PS1) 10Gilles: Remove $.browser call in jquery.hashchange [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/145406 [19:08:44] (03CR) 10Gergő Tisza: "Can you add a comment to the start of the file saying this was forked?" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/145406 (owner: 10Gilles) [19:12:47] (03PS2) 10Gilles: Remove $.browser call in jquery.hashchange [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/145406 [19:15:05] (03CR) 10Gilles: [C: 032] Remove unnecessary dependencies [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/144751 (owner: 10MarkTraceur) [19:15:46] (03Merged) 10jenkins-bot: Remove unnecessary dependencies [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/144751 (owner: 10MarkTraceur) [19:18:02] (03CR) 10Siebrand: Tooltips for loads of things (031 comment) [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/136508 (owner: 10MarkTraceur) [20:00:02] (03PS1) 10Rillke: Adding messages for licensing option of UploadWizard [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/145420 [20:30:03] (03CR) 10Gergő Tisza: [C: 032] "Not sure if this does much good in old non-IE browsers, mixing the navigation history of the iframe and the parent window is a freak IE th" [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/145406 (owner: 10Gilles) [20:31:14] (03Merged) 10jenkins-bot: Remove $.browser call in jquery.hashchange [extensions/MultimediaViewer] - 10https://gerrit.wikimedia.org/r/145406 (owner: 10Gilles) [20:33:24] (03CR) 10Multichill: [C: 031] "This is the right location." [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/145420 (owner: 10Rillke) [20:40:14] (03PS1) 10MarkTraceur: Fix some progress bar issues [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/145428 [20:43:41] eh i would strongly advise disabling mmv on en.wp [20:44:01] history has shown that going against community ends up bad for WMF..... [20:44:10] thedj: Seems a little premature at this point [20:44:29] based on an RFC with <100 participants? not really. [20:44:50] oh boy, this is so going to blow up... [20:45:00] just screw en.wp [20:45:03] ignore them [20:46:28] ;) [20:46:35] I don't think that's a scalable approach. [20:46:45] honestly, these types of site-wide changes should go through codereview like anything else. [20:47:06] that they should [20:47:22] it's not appropriate for admins to tamper with site-wide functionality, certainly not on the basis of small, localized rfcs with highly self-selected participation. [20:47:29] and if we have to put our foot down in this case, we will. [20:47:31] but site changes traditionally follow community [20:48:01] well there are some good points. [20:48:32] which ones? media viewer isn't even the same product since when the RFC started. [20:56:33] fabr I'm mentioning the RFC in my team update on multimedia-l, which should be sufficient. [21:07:51] marktraceur: you around? [21:19:20] 3MediaWiki extensions / 3TimedMediaHandler: MwEmbedSupport and TimedMediaHandler require $wgIncludejQueryMigrate = true; ($.browser) - 10https://bugzilla.wikimedia.org/66359 (10Gilles Dubuc) 5PATC>3RESO/FIX [22:58:46] 3MediaWiki extensions / 3MultimediaViewer: MultimediaViewer should work (or at least fail more gracefully) on wikis where the images have no CORS headers - 10https://bugzilla.wikimedia.org/62469#c9 (10Bryan Hunt) Created attachment 15895 --> https://bugzilla.wikimedia.org/attachment.cgi?id=15895&action=ed... [23:13:20] I noticed that marktraceur used the phabricator topic as a branch name. Is that the usual convention, start something in phabricator, then create a local branch? [23:13:44] neilk_: We use it for bugzilla (bug/nnnnn) and Mingle (card/nnn) as well [23:15:03] marktraceur: ok, but git-review made my local branch "review/marktraceur/T438". What does it look like when you create it, just "T438" ? [23:16:35] marktraceur: and is that a multimedia thing or a mediawiki thing now [23:17:11] Yeah, T438 [23:17:23] ok so let's say I want to do a thing [23:17:28] It's a mediawiki thing for bugs. We sort of use card/ and Tnnn for us [23:17:36] I create a bug? or a mingle card? or a phabricator topic? [23:18:02] I mean, unless there isn't one, you don't need to [23:18:31] 3MediaWiki extensions / 3MultimediaViewer: [Javascript bug]: Some image descriptions (class mv-mmv-author) show "[object Object]" - 10https://bugzilla.wikimedia.org/66564#c2 (10T. Gries) The problem is still present in version 0aab988 (10.07.2014). I am now going to try the api.php?format=jsonfm&action=quer... [23:21:28] marktraceur: okay I created a local branch called "remove-license-from-credits". I can just submit that for review, nothing else required first? [23:21:58] Sure. [23:23:30] (03PS1) 10Neilk: remove license talk from CREDITS [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/145465 [23:25:02] marktraceur: I also wrote a little thing to do credits automatically like you did in the MMV. Will submit later [23:25:12] Cool [23:27:17] 3MediaWiki extensions / 3MultimediaViewer: [Javascript bug]: Some image descriptions (class mv-mmv-author) show "[object Object]" - 10https://bugzilla.wikimedia.org/66564#c3 (10T. Gries) (In reply to Tisza Gergő from comment #1) > Do you have a live test case? No > If not, can you share the response from > a... [23:27:51] (03CR) 10Neilk: "This was prompted by a discussion on the mailing list. Gilles Dubuc noted that the CREDITS file said we it was licensed GPLv2, when every " [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/145465 (owner: 10Neilk)