[00:59:58] marktraceur: you're going to like what's coming now. [01:00:47] (03PS5) 10Bartosz Dziewoński: Make mw.DestinationChecker more reusable [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/250485 [01:00:49] (03PS2) 10Bartosz Dziewoński: Use String#trim rather than $.trim everywhere [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/250891 [01:00:51] (03PS1) 10Bartosz Dziewoński: Split off uw.DateDetailsWidget from UploadWizardDetails (introduce uw.DetailsWidget) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251179 (https://phabricator.wikimedia.org/T96917) [01:00:53] (03PS1) 10Bartosz Dziewoński: Split off uw.CategoriesDetailsWidget from UploadWizardDetails (improve warnings) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251180 (https://phabricator.wikimedia.org/T96917) [01:00:55] (03PS1) 10Bartosz Dziewoński: Split off uw.DescriptionsDetailsWidget from UploadWizardDetails (and OOUI-ize) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251181 (https://phabricator.wikimedia.org/T96917) [01:00:57] (03PS1) 10Bartosz Dziewoński: Split off uw.TitleDetailsWidget from UploadWizardDetails (and OOUI-ize) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251182 (https://phabricator.wikimedia.org/T96917) [01:01:04] i really hope it won't get a ton of -1s now because that'd ruin the effect. [01:01:49] :-) [01:01:57] MatmaRex: Show-off. [01:04:06] (03CR) 10jenkins-bot: [V: 04-1] Split off uw.CategoriesDetailsWidget from UploadWizardDetails (improve warnings) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251180 (https://phabricator.wikimedia.org/T96917) (owner: 10Bartosz Dziewoński) [01:04:09] (03CR) 10jenkins-bot: [V: 04-1] Split off uw.DateDetailsWidget from UploadWizardDetails (introduce uw.DetailsWidget) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251179 (https://phabricator.wikimedia.org/T96917) (owner: 10Bartosz Dziewoński) [01:04:19] alas. [01:04:34] (03CR) 10jenkins-bot: [V: 04-1] Split off uw.DescriptionsDetailsWidget from UploadWizardDetails (and OOUI-ize) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251181 (https://phabricator.wikimedia.org/T96917) (owner: 10Bartosz Dziewoński) [01:04:36] (03CR) 10jenkins-bot: [V: 04-1] Split off uw.TitleDetailsWidget from UploadWizardDetails (and OOUI-ize) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251182 (https://phabricator.wikimedia.org/T96917) (owner: 10Bartosz Dziewoński) [01:08:16] 6Multimedia, 10UploadWizard, 5Patch-For-Review: Split up the details class into multiple classes and files - https://phabricator.wikimedia.org/T96917#1784450 (10matmarex) The first two changes, for date and categories, have very little user-visible changes and can be merged whenever. The next two, for descr... [01:23:25] (03PS1) 10Bartosz Dziewoński: mw.UploadWizardUploadInterface: Remove some dead code [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251184 [01:23:27] (03PS1) 10Bartosz Dziewoński: mw.UploadWizardDetails: Remove unused method #error [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251185 [01:30:26] (03PS2) 10Bartosz Dziewoński: Split off uw.TitleDetailsWidget from UploadWizardDetails (and OOUI-ize) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251182 (https://phabricator.wikimedia.org/T96917) [01:30:28] (03PS2) 10Bartosz Dziewoński: Split off uw.CategoriesDetailsWidget from UploadWizardDetails (improve warnings) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251180 (https://phabricator.wikimedia.org/T96917) [01:30:31] (03PS2) 10Bartosz Dziewoński: Split off uw.DescriptionsDetailsWidget from UploadWizardDetails (and OOUI-ize) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251181 (https://phabricator.wikimedia.org/T96917) [01:30:32] (03PS2) 10Bartosz Dziewoński: Split off uw.DateDetailsWidget from UploadWizardDetails (introduce uw.DetailsWidget) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251179 (https://phabricator.wikimedia.org/T96917) [01:30:48] i think one of these failures is bogus. [01:32:34] (03CR) 10jenkins-bot: [V: 04-1] Split off uw.TitleDetailsWidget from UploadWizardDetails (and OOUI-ize) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251182 (https://phabricator.wikimedia.org/T96917) (owner: 10Bartosz Dziewoński) [01:32:41] (03CR) 10jenkins-bot: [V: 04-1] Split off uw.CategoriesDetailsWidget from UploadWizardDetails (improve warnings) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251180 (https://phabricator.wikimedia.org/T96917) (owner: 10Bartosz Dziewoński) [01:32:47] (03CR) 10jenkins-bot: [V: 04-1] Split off uw.DescriptionsDetailsWidget from UploadWizardDetails (and OOUI-ize) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251181 (https://phabricator.wikimedia.org/T96917) (owner: 10Bartosz Dziewoński) [01:33:00] (03CR) 10jenkins-bot: [V: 04-1] Split off uw.DateDetailsWidget from UploadWizardDetails (introduce uw.DetailsWidget) [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251179 (https://phabricator.wikimedia.org/T96917) (owner: 10Bartosz Dziewoński) [01:38:02] 6Multimedia, 10UploadWizard: Use OOUI 'required' indicator for required fields - https://phabricator.wikimedia.org/T117833#1784508 (10matmarex) 3NEW [01:57:54] Oh god. [01:58:04] MatmaRex: y u do this [01:58:18] I was all excited [01:58:22] "I'm caught up", I thought [01:58:28] "I can prototype this new thing", I thought [01:58:35] But no. :P [03:31:46] (03PS1) 10Bartosz Dziewoński: mw.UploadWizardDetails: Upload TitleBlacklist error handling for T115258 [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251195 [07:43:32] 6Multimedia, 10MediaWiki-extensions-VipsScaler, 5Patch-For-Review, 5WMF-deploy-2015-11-03_(1.27.0-wmf.5): vips not rendering 16-bit greyscale tiffs properly to png - https://phabricator.wikimedia.org/T116947#1784811 (10Bawolff) Just as a point of interest, the only thing left in https://commons.wikimedia.o... [10:25:23] 6Multimedia, 10MediaWiki-General-or-Unknown, 7Design: Surface both search and upload in a better way (not tabs maybe) - https://phabricator.wikimedia.org/T116044#1785071 (10Pginer-WMF) > @Pginer-WMF @Prtksxna what you think? I think that many of the ideas proposed can work really well. Some comments: - The... [12:04:28] 6Multimedia, 10Browser-Tests-Infrastructure, 10UploadWizard: Links broken in UploadWizard-api-commons.wikimedia.beta.wmflabs.org job error report - https://phabricator.wikimedia.org/T66119#1785291 (10zeljkofilipin) The job sends mail to multimedia-alerts@lists.wikimedia.org https://integration.wikimedia.org... [13:05:56] 6Multimedia, 10UploadWizard: UploadWizard strings are skewed towards images (though e.g. also PDFs can be uploaded) - https://phabricator.wikimedia.org/T117795#1785369 (10Aklapper) [13:06:11] 6Multimedia, 10UploadWizard: UploadWizard strings are skewed towards images (though e.g. also PDFs can be uploaded) - https://phabricator.wikimedia.org/T117795#1785373 (10Aklapper) [13:34:20] (03CR) 10Siebrand: [C: 031] "i18n/L10n reviewed." [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251181 (https://phabricator.wikimedia.org/T96917) (owner: 10Bartosz Dziewoński) [14:25:03] *yawn* good morning. [15:01:58] morning marktraceur [15:04:38] o/ [15:05:20] huh, i have 16 open changes for UW :o [15:06:19] https://gerrit.wikimedia.org/r/251195 this is high-priority [15:06:43] https://gerrit.wikimedia.org/r/251185 https://gerrit.wikimedia.org/r/251184 https://gerrit.wikimedia.org/r/250891 https://gerrit.wikimedia.org/r/250715 https://gerrit.wikimedia.org/r/251121 these are trivial [15:07:10] https://gerrit.wikimedia.org/r/251155 this needs James_F|Away [15:07:47] https://gerrit.wikimedia.org/r/248359 this needs brion to stop -1-ing its dependency for silly reasons ;) [15:09:16] https://gerrit.wikimedia.org/r/#/q/status:open+project:mediawiki/extensions/UploadWizard+branch:master+topic:details,n,z these four are a major refactor of the Details step, and i need to talk to marktraceur about them [15:13:30] Argh [15:14:57] This is exactly what I didn't want to do today. [15:15:04] MatmaRex: What do we need to talk about? :) [15:15:07] marktraceur: prtksxna: there are a few bits of Details that still use the old system, i want one of you to choose one of them and convert to the new system, and raise alarm if it's too weird :P [15:16:01] Oh no. This is one of those audience participation conversations. [15:16:02] i've been staring at this code for the past week and i might be overlooking some obvious problems [15:16:05] haha [15:16:20] OK, I'mma take a look quick [15:17:08] I fixed the calendar event, T-45 minutes [15:17:13] all this can wait though. would be nice to get https://gerrit.wikimedia.org/r/251195 merged quickly, and some of the random patches i have, before they all start conflicting with each other [15:17:46] I'll do that first, then. [15:18:00] ooh, nice, thanks. yay calendars [15:39:53] (03CR) 10Jforrester: [C: 031] "Fine by me; qqq should be updated to explain. :-)" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251155 (owner: 10Bartosz Dziewoński) [15:41:57] (03CR) 10Bartosz Dziewoński: "Done in https://translatewiki.net/w/i.php?title=MediaWiki:Prefs-uploads/qqq&diff=6529703&oldid=4979306" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251155 (owner: 10Bartosz Dziewoński) [15:42:20] MatmaRex: In what way does https://gerrit.wikimedia.org/r/#/c/237640/ look worse than what we've got now? See e.g. http://en.wikipedia.beta.wmflabs.org/wiki/Special:Search [15:42:45] (03CR) 10Jforrester: [C: 032] "Tsk. :-)" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251155 (owner: 10Bartosz Dziewoński) [15:44:11] James_F: in a subjective one :) want a screenshot? [15:44:28] (03Merged) 10jenkins-bot: Change label of preferences tab from "Uploads" to "Uploading" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251155 (owner: 10Bartosz Dziewoński) [15:44:50] James_F: oh also, it'll increase the number of bytes of CSS on each page by a jillion. [15:45:32] MatmaRex: Yeah. Maybe I'm not seeing it? [15:45:38] MatmaRex: Only on pages with a help link. [15:45:44] MatmaRex: Which is far from most. [15:45:52] Or is it unconditionally added? [15:46:16] oh, maybe [15:46:58] It's in addHelpLink only, which is not normally called. [15:47:00] * James_F greps. [15:48:50] It's on 23 core special pages, category pages, one action page (delete), and it's optionally available for all Action, Article and Special pages through config, it looks like. [15:49:42] James_F: http://imgur.com/a/WTXxX [15:50:19] MatmaRex: I prefer the new styling. [15:50:32] i think it's nasty and i'm not going to merge it. [15:50:33] * James_F shrugs. [15:50:36] OK. [15:50:50] We should split up what gets loaded from OOUI to be smart before we merge anyway. [15:52:54] MatmaRex: How do-able do you think that is, BTW? [15:53:50] James_F: very. it just needs work. [15:54:06] The implicit loading idea, or the splitting up? [15:54:16] It's attractive but sounds a bit magic. [15:55:23] the splitting [15:58:35] 6Multimedia, 10MediaWiki-General-or-Unknown, 10VisualEditor, 10VisualEditor-MediaWiki-Media: Show determinate progress bar for the image upload in mw.ForeignStructuredUpload.BookletLayout - https://phabricator.wikimedia.org/T115861#1785609 (10Pginer-WMF) > While the thin progress bar does look smart - I th... [16:07:00] 6Multimedia, 10TimedMediaHandler-Player, 7Design, 5Patch-For-Review: Investigate increasing the contrast for arrows in the multimedia video player - https://phabricator.wikimedia.org/T117344#1785635 (10Jdforrester-WMF) p:5Triage>3Normal [16:15:51] MatmaRex: I thought "senselessimagename" was for stuff like DSC_0023 or whatever [16:16:05] Like, "I forgot to rename the camera's default filenames" [16:16:56] Commons probably blacklists those manually in their blacklist, but we have a way to catch it on not-Commons methinks. [16:17:13] And apparently, people who are not Commons actually use it [16:17:26] (UW, that is, no idea on the title blacklist stuff) [16:18:00] marktraceur: 'senselessimagename' was clearly copied from Commons' blacklist, but the blacklist entry apparently got renamed to something else, because 'senselessimagename' is no longer there [16:18:24] Oh, huh. [16:18:24] and yeah, MW core has some list of bad image name prefixes somewhere, which we also handle [16:19:36] Arright, testing the patch [16:21:58] (03CR) 10MarkTraceur: [C: 032] mw.UploadWizardDetails: Upload TitleBlacklist error handling for T115258 [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251195 (owner: 10Bartosz Dziewoński) [16:22:31] MatmaRex: I guess now I will try converting the location fields to The New Style™ unless there's other urgent stuff? [16:23:41] not really [16:25:50] (03Merged) 10jenkins-bot: mw.UploadWizardDetails: Upload TitleBlacklist error handling for T115258 [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251195 (owner: 10Bartosz Dziewoński) [16:35:24] 6Multimedia, 6Commons, 10MediaWiki-File-management, 10MediaWiki-Tarball-Backports, and 7 others: InstantCommons broken by switch to HTTPS - https://phabricator.wikimedia.org/T102566#1785750 (10BBlack) It's been over a week since the email, which ended up going out a bit later after the releases than expect... [16:36:38] MatmaRex: I guess I never really figured out what layout would allow me to produce three text input widgets side-by-side, horizontal layout isn't the answer IIRC [16:37:35] horizontallayout should work? [16:37:49] marktraceur: and most widgets are inline by default. if you just put them into DOM, they should appear side-by-side. [16:37:57] Hm. [16:39:09] although, hmm, they each have a label here. so that might come out weird [16:39:29] Yeah, that's what my issue was [16:39:50] Maybe have two lines, one with the labels and one with the fields? That might come out misaligned though [16:40:00] 6Multimedia, 10MediaWiki-Uploading, 10VisualEditor, 10VisualEditor-EditingTools: Cross-wiki upload from is not adding == {{int:license-header}} == - https://phabricator.wikimedia.org/T117878#1785751 (10Steinsplitter) 3NEW [16:40:31] 6Multimedia, 10MediaWiki-Uploading, 10VisualEditor, 10VisualEditor-EditingTools: Cross-wiki upload from is not adding == {{int:license-header}} == - https://phabricator.wikimedia.org/T117878#1785759 (10Steinsplitter) [16:41:03] marktraceur: eh, i wouldn't bother with layouts and crap, just style them with a bit of CSS [16:41:14] like: [16:41:30] Ah, sure. [16:41:41] marktraceur:
Latitude
... [16:41:43] I think I can pick up what you're layin' down [16:41:51] I have a less file for it anyway. [16:41:57] (not as inline 'style', naturally :) ) [16:42:01] Yeah [16:42:11] I was worried for a second there [17:03:43] 6Multimedia, 10MediaWiki-Uploading, 10VisualEditor, 10VisualEditor-EditingTools: Cross-wiki upload from is not adding == {{int:license-header}} == - https://phabricator.wikimedia.org/T117878#1785798 (10Steinsplitter) [17:08:48] So...there *is* code here for an OSM map. Should I just delete it all? [17:08:54] James_F? [17:10:54] It's not something I can easily reproduce in an OOUI widget, so... [17:11:30] marktraceur: https://gerrit.wikimedia.org/r/#/c/242637/ [17:12:15] NEVERMERGE though. [17:12:46] marktraceur: Well, yeah, but it's an OOUI widget that does show a map. [17:12:56] 6Multimedia, 6Commons, 10MediaWiki-Uploading, 10VisualEditor: Author not linked in files uploaded from inside the visual editor - https://phabricator.wikimedia.org/T117882#1785832 (10Steinsplitter) 3NEW [17:13:12] 6Multimedia, 6Commons, 10MediaWiki-Uploading, 10VisualEditor, 10VisualEditor-EditingTools: Cross-wiki upload from is not adding == {{int:license-header}} == - https://phabricator.wikimedia.org/T117878#1785841 (10Steinsplitter) [17:14:21] *shrug* James_F should I just fix that up and implement it? [17:14:30] marktraceur: I dunno. edsanders? [17:14:40] "just" [17:15:18] it's in the wrong repo [17:15:19] But also, as far as I know, there is no wiki currently using the OSM code. So it might not be a big deal to lose it. [17:15:42] edsanders: We could make it a mwWidget. [17:15:44] it's code for a extension that doesn't exist [17:15:58] And then when never comes along, and edsanders's map widget gets merged, we can write it into UW [17:15:59] marktraceur: Just kill that code. [17:16:04] Sure. [17:16:12] it's not a hard extension to write... [17:16:28] just generates static map image urls from some attributes [17:17:14] The main problem is that Ops won't let people use maps.wikimedia.org without a lot more hardware (== cash) thrown at it [17:17:15] or whatever [17:17:20] sure [17:17:21] So we can't show it to users. [17:17:54] * marktraceur gets his machete out [17:18:31] edsanders: For context, I suggested that setting the geo-position of an uploaded file could be nice to be a pin-drop on a mpa. [17:18:33] Err. Map. [17:19:24] sure [17:19:37] 6Multimedia, 6Commons, 10MediaWiki-Uploading, 10VisualEditor, 10VisualEditor-EditingTools: Cross-wiki upload tool fails to add == {{int:filedesc}} == or == {{int:license-header}} == - https://phabricator.wikimedia.org/T117878#1785878 (10Jdforrester-WMF) [17:19:50] marktraceur: Feel like fixing that quickly? [17:20:40] Guh [17:20:47] Fun, huh? [17:20:51] Yeah, loads [17:21:18] * James_F sighs. [17:24:36] 6Multimedia, 6Commons, 10MediaWiki-File-management, 10MediaWiki-Tarball-Backports, and 7 others: InstantCommons broken by switch to HTTPS - https://phabricator.wikimedia.org/T102566#1785897 (10BBlack) 5Open>3Resolved a:3BBlack [18:02:31] MatmaRex: "layout.setNotices is not a function" maybe an issue with the first patch? uw.FieldLayout? [18:02:57] marktraceur: update your mediawiki/core, it depends on a tweak in OOUI [18:03:23] KK. [18:03:46] Styling done, looking pretty good so far [18:06:52] MatmaRex: Do I have to do something more than add getErrors to a subclass in order to get it to validate? I'm getting no validation presently I think. [18:07:49] marktraceur: there's a list of widgets to validate in UWDetails#checkValidity, #getErrors, #getWarnings [18:08:00] Neato, thanks [18:08:11] marktraceur: (they should be in some array, but this way seemed simpler for now that we're dealing with the old system) [18:08:51] Oh, it's in the details controller. [18:09:23] (UWDetails = mw.UploadWizardDetails, the huge class) [18:09:39] Naw, it's in uw.controller.Details [18:10:09] 6Multimedia, 6Commons, 10MediaWiki-Uploading, 10VisualEditor, 10VisualEditor-EditingTools: Cross-wiki upload tool fails to add == {{int:filedesc}} == or == {{int:license-header}} == - https://phabricator.wikimedia.org/T117878#1786026 (10matmarex) a:3matmarex [18:11:47] Right, pushing [18:12:25] (03PS1) 10MarkTraceur: Convert location fields to OOUI [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251291 [18:12:30] Pretty nifty system I think [18:13:35] MatmaRex: You're fixing the cross-wiki upload headers bug? [18:13:55] So I can maybe go back to my image editing? :D [18:13:56] (03CR) 10jenkins-bot: [V: 04-1] Convert location fields to OOUI [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251291 (owner: 10MarkTraceur) [18:14:51] yeah [18:15:11] marktraceur: :D [18:16:41] Aw man [18:20:55] 6Multimedia, 10MediaWiki-General-or-Unknown, 10VisualEditor: Categories and descriptions are sometimes multiplicated in mw.Upload uploads - https://phabricator.wikimedia.org/T116394#1786069 (10matmarex) p:5Triage>3High a:3matmarex [18:26:11] 6Multimedia, 10TimedMediaHandler, 10TimedMediaHandler-Player: Improve usability and compatibility of audio pronunciation clips - https://phabricator.wikimedia.org/T116213#1786152 (10Jdforrester-WMF) p:5Triage>3Low [18:27:52] 6Multimedia, 6Commons, 10Wikimedia-Video, 7Epic: [epic] Self-contained spaces for complex media (source files, formats, helper programs, context) - https://phabricator.wikimedia.org/T116544#1786172 (10Jdforrester-WMF) p:5Triage>3Low [18:29:15] 6Multimedia, 6Commons, 10MediaWiki-Uploading, 10VisualEditor: In the cross-wiki upload tool, put in "[[User:Foo|own work]]" not just "own work" - https://phabricator.wikimedia.org/T117882#1786182 (10Jdforrester-WMF) p:5High>3Normal [18:29:42] 6Multimedia, 10VisualEditor, 10VisualEditor-MediaWiki-Media: In the cross-wiki upload tool, the upload can timeout causing an unhandled exception - https://phabricator.wikimedia.org/T117545#1786189 (10Jdforrester-WMF) [18:29:56] 6Multimedia, 10UploadWizard, 7Design: UploadWizard strings are skewed towards images (though e.g. also PDFs can be uploaded) - https://phabricator.wikimedia.org/T117795#1786190 (10Jdforrester-WMF) p:5Triage>3Low [18:30:53] 6Multimedia, 6Commons, 10MediaWiki-Uploading, 10VisualEditor: Author not linked in files uploaded from inside the visual editor - https://phabricator.wikimedia.org/T117882#1786200 (10matmarex) [18:31:08] 6Multimedia, 10TimedMediaHandler, 10Wikimedia-Video: [[File:]] keyword for silent, looped autoplay videos (GIF-like behavior) - https://phabricator.wikimedia.org/T116501#1786202 (10Jdforrester-WMF) p:5Triage>3Lowest I'd rather we not introduce new syntax just yet, given we're planning to replace the whol... [18:31:16] 6Multimedia, 6Commons, 10MediaWiki-Uploading, 10VisualEditor: Author not linked in files uploaded from inside the visual editor - https://phabricator.wikimedia.org/T117882#1785832 (10matmarex) (It's just about the "Author" field, I think, not "Source".) [18:32:14] 6Multimedia, 6Commons, 10MediaWiki-Uploading, 10VisualEditor: Author not linked in files uploaded from inside the visual editor - https://phabricator.wikimedia.org/T117882#1786209 (10Steinsplitter) >Jdforrester-WMF changed the title from "Author not linked in files uploaded from inside the visual editor" t... [18:32:21] 6Multimedia, 10UploadWizard: No way to undo overwriting of filenames after 'copy'ing title from first to (2nd...18th...etc) later files - https://phabricator.wikimedia.org/T116513#1786212 (10Jdforrester-WMF) p:5Triage>3Low [18:32:35] 6Multimedia, 6Commons, 10MediaWiki-Uploading, 10VisualEditor: Author not linked in files uploaded from inside the visual editor - https://phabricator.wikimedia.org/T117882#1786214 (10matmarex) a:3matmarex [18:38:08] James_F: https://gerrit.wikimedia.org/r/251294 https://gerrit.wikimedia.org/r/251298 https://gerrit.wikimedia.org/r/251299 - do we want to SWAT any/all? [18:45:14] MatmaRex: All. [18:46:34] yay [18:46:35] MatmaRex: Is https://gerrit.wikimedia.org/r/#/c/251299/1/resources/src/mediawiki/mediawiki.ForeignStructuredUpload.js right? [18:47:08] James_F: i believe it is. which part looks suspicious? [18:47:43] the English names for namespaces work on all different language wikis too. [18:47:45] MatmaRex: Oh, right, read the updates on the task. [18:47:49] Yeah, that bit's fine. [18:48:14] ah, i see. thanks for checking it :) [18:53:20] (03PS2) 10MarkTraceur: Convert location fields to OOUI [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251291 [18:55:27] (03CR) 10jenkins-bot: [V: 04-1] Convert location fields to OOUI [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/251291 (owner: 10MarkTraceur) [19:09:32] (03PS2) 10Jforrester: Add explicit dependency on es5-shim [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/250715 (owner: 10Bartosz Dziewoński) [19:10:24] (03CR) 10Jforrester: [C: 032] Add explicit dependency on es5-shim [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/250715 (owner: 10Bartosz Dziewoński) [19:15:08] (03Merged) 10jenkins-bot: Add explicit dependency on es5-shim [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/250715 (owner: 10Bartosz Dziewoński) [19:15:31] OK so [19:15:36] I cannot think of a good name for this classe [19:15:38] class. [19:15:47] I started with "UploadFromUpload" but that makes no sense [19:16:01] The most accurate description is "UploadFromUploadingLocalFileToUrl" but that's too long. [19:18:53] UploadSlinger