[05:12:05] 06Multimedia, 06Commons, 10UploadWizard, 10Wikimedia-Site-requests, 13Patch-For-Review: Set $wgUploadWizardConfig['blacklistIssuesPage'] = ''; for commonswiki - https://phabricator.wikimedia.org/T146417#2666116 (10Josve05a) [11:25:02] marktraceur: I can't open that bug (T146659) [12:13:50] (03PS1) 10Prtksxna: [wip] Handle API failures when editing [extensions/FileAnnotations] - 10https://gerrit.wikimedia.org/r/312823 (https://phabricator.wikimedia.org/T145455) [12:15:34] (03CR) 10Prtksxna: "Didn't we do some clever stuff in the upload dialog with error messages? Does that need to be replicated here, or is this a simpler use ca" [extensions/FileAnnotations] - 10https://gerrit.wikimedia.org/r/312823 (https://phabricator.wikimedia.org/T145455) (owner: 10Prtksxna) [12:47:11] (03CR) 10TheDJ: "I think it is a bit embarrassing if we cannot handle this... We have little video to begin with and metadata fetching considerably speeds " [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312713 (owner: 10Brion VIBBER) [13:11:31] (03CR) 10TheDJ: [C: 04-2] "Does not take into account parser output caching. Either make the user option vary the parser cache, or make sure both TMH and videoJS mod" [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) (owner: 10Paladox) [13:36:56] (03Abandoned) 10TheDJ: Trigger transcode job for missing transcodes [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/284269 (owner: 10TheDJ) [14:15:32] (03CR) 10Brion VIBBER: "On mobile we're now delaying loading of images until they appear in viewport, so there is some sensitivity to extra bandwidth usage of thi" [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312713 (owner: 10Brion VIBBER) [14:33:13] (03CR) 10TheDJ: ""On mobile we're now delaying loading of images"" [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312713 (owner: 10Brion VIBBER) [14:33:47] (03CR) 10Paladox: "Hi, I thought I did, since it works." [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) (owner: 10Paladox) [14:49:29] Ugh travel [14:53:39] (03CR) 10Brion VIBBER: "Even on desktop with native playback I'm extremely reluctant to apply preload="auto" which per spec (and confirmed in Firefox) can trigger" [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312713 (owner: 10Brion VIBBER) [14:59:12] ugh indeed. [14:59:18] oh, meeting time. [15:11:12] (03CR) 10MarkTraceur: "* I believe there are several other options (including in core):" [extensions/FileAnnotations] - 10https://gerrit.wikimedia.org/r/312823 (https://phabricator.wikimedia.org/T145455) (owner: 10Prtksxna) [15:26:03] (03PS3) 10Brion VIBBER: Don't preload entire video files in video.js mode [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312713 [15:29:54] (03CR) 10jenkins-bot: [V: 04-1] Don't preload entire video files in video.js mode [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312713 (owner: 10Brion VIBBER) [15:30:40] (03PS4) 10Brion VIBBER: Don't preload entire video files in video.js mode [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312713 [15:34:58] (03CR) 10jenkins-bot: [V: 04-1] Don't preload entire video files in video.js mode [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312713 (owner: 10Brion VIBBER) [15:46:09] (03PS5) 10Brion VIBBER: Don't preload entire video files in video.js mode [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312713 [15:52:24] (03CR) 10Paladox: "> Does not take into account parser output caching. Either make the" [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) (owner: 10Paladox) [15:55:12] (03CR) 10Brion VIBBER: "It's probably working when switching on due to the existing support for invalidating cached entries that were saved with the mwembed mode;" [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) (owner: 10Paladox) [15:57:38] (03CR) 10Paladox: "Would I be able to have help with that please?" [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) (owner: 10Paladox) [16:00:57] 06Multimedia, 06Commons, 10UploadWizard, 13Patch-For-Review: Verify license tags for custom licenses ("Another reason not mentioned above") - https://phabricator.wikimedia.org/T140903#2667600 (10matmarex) >>! In T140903#2480363, @matmarex wrote: > We used to have some form of this, but IIRC it was broken i... [16:44:56] (03CR) 10Bartosz Dziewoński: [C: 032] Verify license tags for custom licenses [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/309384 (https://phabricator.wikimedia.org/T140903) (owner: 10Matthias Mullie) [16:46:49] (03Merged) 10jenkins-bot: Verify license tags for custom licenses [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/309384 (https://phabricator.wikimedia.org/T140903) (owner: 10Matthias Mullie) [16:49:20] 06Multimedia, 06Commons, 10UploadWizard: Verify license tags for custom licenses ("Another reason not mentioned above") - https://phabricator.wikimedia.org/T140903#2667863 (10matmarex) Note that this is not done yet, a small configuration change is required. Without it, the patch only checks that any templat... [16:51:03] (03CR) 10Paladox: "Would something like" [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) (owner: 10Paladox) [17:15:49] (03PS13) 10Paladox: Add a new beta feature to enable Videojs [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) [17:16:00] (03CR) 10Paladox: "> Does not take into account parser output caching. Either make the" [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) (owner: 10Paladox) [17:16:09] (03PS14) 10Paladox: Add a new beta feature to enable Videojs [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) [17:19:50] (03CR) 10jenkins-bot: [V: 04-1] Add a new beta feature to enable Videojs [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) (owner: 10Paladox) [17:21:32] (03PS15) 10Paladox: Add a new beta feature to enable Videojs [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) [17:22:37] (03CR) 10Paladox: "> It's probably working when switching on due to the existing support" [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) (owner: 10Paladox) [17:44:38] (03PS1) 10Bartosz Dziewoński: mw.UploadWizardLicenseInput: Parse license templates as if they were used on a file page [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/312854 (https://phabricator.wikimedia.org/T140903) [17:47:02] 06Multimedia, 06Commons, 10UploadWizard, 13Patch-For-Review, 05WMF-deploy-2016-10-04_(1.28.0-wmf.21): Verify license tags for custom licenses ("Another reason not mentioned above") - https://phabricator.wikimedia.org/T140903#2668177 (10matmarex) I tried to compare the differences in behavior between the... [18:37:54] 06Multimedia, 10UploadWizard: Error text links on step Describe for the title input field are broken - https://phabricator.wikimedia.org/T146532#2668322 (10matmarex) @Herzi.Pinki Okay, sorry for misunderstanding. The popups definitely work for me, though (I just verified this again using Firefox 50 on Windows... [18:51:12] (03CR) 10Paladox: "@Brion VIBBER or @TheDJ could you re review please?" [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) (owner: 10Paladox) [18:51:44] (03CR) 10TheDJ: [C: 032] Don't preload entire video files in video.js mode [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312713 (owner: 10Brion VIBBER) [18:54:12] (03CR) 10TheDJ: [C: 032] Upgrade ogv.js to 1.2.1 [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312712 (owner: 10Brion VIBBER) [18:54:46] 06Multimedia, 10MediaWiki-Uploading, 10VisualEditor, 10VisualEditor-MediaWiki-Media, 07Design: Provide a way during the upload stage for the user to change their mind about what file to upload once they've started it uploading - https://phabricator.wikimedia.org/T114938#2668365 (10matmarex) 05Open>03R... [18:56:46] (03CR) 10Paladox: "@TheDJ could you merge please?" [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312633 (owner: 10Paladox) [19:00:50] (03Merged) 10jenkins-bot: Upgrade ogv.js to 1.2.1 [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312712 (owner: 10Brion VIBBER) [19:01:24] (03Merged) 10jenkins-bot: Don't preload entire video files in video.js mode [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312713 (owner: 10Brion VIBBER) [19:18:47] (03PS16) 10Paladox: Add a new beta feature to enable Videojs [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) [19:25:33] 06Multimedia, 10UploadWizard: Error text links on step Describe for the title input field are broken - https://phabricator.wikimedia.org/T146532#2668438 (10Herzi.Pinki) Matmarex, sorry for being such an idiot. grmmmmm. Of course I tried to open it in another tab. It works like a charm on left click. From the... [19:26:57] (03CR) 10TheDJ: [C: 032] Fix VideoJS resolution plugin on Internet Explorer 9 [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312633 (owner: 10Paladox) [19:33:43] (03Merged) 10jenkins-bot: Fix VideoJS resolution plugin on Internet Explorer 9 [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312633 (owner: 10Paladox) [19:38:52] (03CR) 10Paladox: "Thanks." [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312633 (owner: 10Paladox) [19:40:22] (03PS17) 10Paladox: Add a new beta feature to enable Videojs [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) [19:40:27] (03CR) 10Paladox: "Just rebasing." [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) (owner: 10Paladox) [19:41:35] (03CR) 10Paladox: "@TheDJ I think the cache should work now :)" [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) (owner: 10Paladox) [19:43:47] 06Multimedia, 10UploadWizard: Error text links on step Describe for the title input field are broken - https://phabricator.wikimedia.org/T146532#2668502 (10matmarex) Haha, whoops. You make a fair point though, these look like links (even though they really aren't) and users are likely to try to open them in a... [19:44:01] 06Multimedia, 10UploadWizard: Error text links on step Describe for the title input field do not open popups on middle-click / Ctrl-click etc. - https://phabricator.wikimedia.org/T146532#2668503 (10matmarex) [20:25:04] (03CR) 10TheDJ: [C: 04-1] "A few comments." (037 comments) [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/312327 (https://phabricator.wikimedia.org/T100106) (owner: 10Paladox) [20:34:29] 06Multimedia, 10MediaWiki-General-or-Unknown, 10UploadWizard, 13Patch-For-Review: Error text links on step Describe for the title input field do not open popups on middle-click / Ctrl-click etc. - https://phabricator.wikimedia.org/T146532#2668658 (10matmarex) p:05Triage>03Normal a:03matmarex [20:38:41] God the Gerrit upgrade is like a murder/suicide pact for lines longer than 80 characters [20:39:01] good night folks [20:39:18] thedj: Night! [20:57:40] (03CR) 10MarkTraceur: [C: 032] mw.UploadWizardLicenseInput: Parse license templates as if they were used on a file page [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/312854 (https://phabricator.wikimedia.org/T140903) (owner: 10Bartosz Dziewoński) [20:59:27] (03Merged) 10jenkins-bot: mw.UploadWizardLicenseInput: Parse license templates as if they were used on a file page [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/312854 (https://phabricator.wikimedia.org/T140903) (owner: 10Bartosz Dziewoński)