[06:24:05] (03CR) 10DannyS712: Introduce Jest test suite for Vue files (031 comment) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/616605 (https://phabricator.wikimedia.org/T258610) (owner: 10Eric Gardner) [07:39:34] (03Merged) 10jenkins-bot: Adjust for signature change in TermFallbackLanguageChain constructor [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/615483 (https://phabricator.wikimedia.org/T250930) (owner: 10Michael Große) [08:14:24] (03CR) 10Jforrester: [C: 04-1] "Hot-loading off Commons? This is absolutely not the right way to use the built-in icons." [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/615435 (https://phabricator.wikimedia.org/T258584) (owner: 10Iniquity) [10:03:51] (03CR) 10DCausse: Normalize statements & fulltext scores relative-ish to eachother (031 comment) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/617678 (owner: 10Matthias Mullie) [14:00:43] 10Multimedia, 10Commons, 10MediaWiki-DjVu, 10MediaWiki-File-management, and 2 others: Some DjVu files have too much metadata to fit in their database column - https://phabricator.wikimedia.org/T192866 (10Xover) @Aklapper Thanks, but as @TheDJ notes, going by [[ https://www.mediawiki.org/wiki/Developers/Mai... [14:03:34] 10Multimedia, 10Commons, 10MediaWiki-DjVu, 10MediaWiki-File-management, and 5 others: Some DjVu files have too much metadata to fit in their database column - https://phabricator.wikimedia.org/T192866 (10Xover) [15:25:19] (03CR) 10Eric Gardner: "Thanks for the comments. I will address most of them today." (036 comments) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/616918 (https://phabricator.wikimedia.org/T258833) (owner: 10Eric Gardner) [15:27:23] (03CR) 10Eric Gardner: Introduce Jest test suite for Vue files (031 comment) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/616605 (https://phabricator.wikimedia.org/T258610) (owner: 10Eric Gardner) [16:09:27] 10Multimedia, 10Commons, 10MediaWiki-DjVu, 10MediaWiki-File-management, and 5 others: Some DjVu files have too much metadata to fit in their database column - https://phabricator.wikimedia.org/T192866 (10MarkTraceur) The maintenance of Formerly-Multimedia™ components is an ongoing issue for the Structured... [16:45:29] 10Multimedia, 10VideoJS player, 10Wikimedia-Site-requests, 10Wikimedia-Video, and 3 others: Roll out videojs as the only video/audio player on all Wikimedia wikis - https://phabricator.wikimedia.org/T248418 (10Iniquity) Wow, stop, you have a bunch of bugs not fixed, what deployment to Group 1? [17:11:53] 10Multimedia, 10VideoJS player, 10Wikimedia-Site-requests, 10Wikimedia-Video, and 3 others: Roll out videojs as the only video/audio player on all Wikimedia wikis - https://phabricator.wikimedia.org/T248418 (10brion) I might actually prefer to delay until the remember-subtitles-setting fix is merged. I'm h... [17:13:14] 10Multimedia, 10VideoJS player, 10Wikimedia-Site-requests, 10Wikimedia-Video, and 3 others: Roll out videojs as the only video/audio player on all Wikimedia wikis - https://phabricator.wikimedia.org/T248418 (10brion) I think I lost my work patch for the subtitles >:( Will have to recreate it shortly [17:21:27] 10Multimedia, 10VideoJS player, 10Wikimedia-Site-requests, 10Wikimedia-Video, and 3 others: Roll out videojs as the only video/audio player on all Wikimedia wikis - https://phabricator.wikimedia.org/T248418 (10Iniquity) Blockers, I think: * T258578 * T258644 * T258570 * T258622 * T249204 * T258638 [17:27:30] 10Multimedia, 10VideoJS player, 10Wikimedia-Site-requests, 10Wikimedia-Video, and 3 others: Roll out videojs as the only video/audio player on all Wikimedia wikis - https://phabricator.wikimedia.org/T248418 (10Iniquity) And what will happen after the deployment with {T246035}? [17:33:47] 10Multimedia, 10VideoJS player, 10Wikimedia-Site-requests, 10Wikimedia-Video, and 3 others: Roll out videojs as the only video/audio player on all Wikimedia wikis - https://phabricator.wikimedia.org/T248418 (10brion) >>! In T248418#6357093, @Iniquity wrote: > And what will happen after the deployment with... [17:44:24] brion: Unless we rush some fixes out tonight, they won't ship this week. :-( [17:44:48] James_F: s'okay! we put the fear of the Elder Gods in people and they told us which bugs are the highest priority for them [17:44:50] this is a win ;) [17:45:09] i'll fix bugs the rest of this wekk and we'll push next monday :) [17:45:09] Well, we've already triaged all those tasks as non-blockers, I think. [17:45:22] well i'd maybe let half of them actually block [17:45:24] lemme look over [17:45:29] After the push to group1 is when the /real/ complaining will begin. [17:45:34] Let alone after group2. [17:46:02] def fix the subtitles minor issues & not-remembering. [17:46:06] and if i fix the rest, well great [17:46:33] the hardest question is the audio player behavior differently. i mean it just does behavior differently because we want to provide UI for it that's biggest than a postage stamp [17:46:36] But e.g. gallery / Special:NewFiles rendering isn't really a huge priority. [17:46:53] I think the current videojs behaviour is good enough. [17:46:54] heh [17:47:06] Once deployed, no doubt lots of people will have ideas. [17:47:10] :D :D [17:47:16] they'll never stop [17:51:26] Oh, indeed. [18:06:20] (03CR) 10Iniquity: "> Patch Set 5: Code-Review-1" [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/615435 (https://phabricator.wikimedia.org/T258584) (owner: 10Iniquity) [18:20:33] (03CR) 10Brion VIBBER: [C: 04-1] "Doesn't show any play icon at all when I test it in Chrome. Hotlinks a file from Commons (but it doesn't seem to load...?) which would hav" (031 comment) [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/615435 (https://phabricator.wikimedia.org/T258584) (owner: 10Iniquity) [18:22:10] (03CR) 10Iniquity: "> Patch Set 5: Code-Review-1" (031 comment) [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/615435 (https://phabricator.wikimedia.org/T258584) (owner: 10Iniquity) [18:28:08] * brion fights with docker and with ipv6, separately [18:31:58] (03PS6) 10Iniquity: Change icon style of embed video and use OOUI play icon [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/615435 (https://phabricator.wikimedia.org/T258584) [18:32:09] That sounds entirely fun. ;-) [18:48:47] (03PS4) 10Anne Tomasevich: Recreate autocomplete search input [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/617796 (https://phabricator.wikimedia.org/T256673) [19:23:24] (03PS5) 10Anne Tomasevich: Recreate autocomplete search input [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/617796 (https://phabricator.wikimedia.org/T256673) [19:30:04] (03CR) 10Anne Tomasevich: "This is ready for review, although I'm still doing some accessibility testing locally." [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/617796 (https://phabricator.wikimedia.org/T256673) (owner: 10Anne Tomasevich) [19:30:38] (03PS6) 10Anne Tomasevich: Recreate autocomplete search input [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/617796 (https://phabricator.wikimedia.org/T256673) [19:50:09] 10Multimedia, 10Beta-Cluster-Infrastructure, 10Commons, 10MediaWiki-File-management, 10Wikimedia-SVG-rendering: Test resvg on Beta Cluster - https://phabricator.wikimedia.org/T243893 (10AntiCompositeNumber) The beta cluster still uses production Thumbor for thumbnail rendering. Testing resvg via MediaWik... [21:18:03] (03CR) 10Eric Gardner: "This is a huge amount of work and I think it's 80% of the way there, nicely done!" (034 comments) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/617796 (https://phabricator.wikimedia.org/T256673) (owner: 10Anne Tomasevich) [21:26:16] (03CR) 10Kaldari: [C: 03+2] Moving stray license definition from CommonSettings.php [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/615303 (owner: 10Kaldari) [21:34:38] (03Merged) 10jenkins-bot: Moving stray license definition from CommonSettings.php [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/615303 (owner: 10Kaldari) [22:49:28] (03CR) 10DannyS712: Introduce Jest test suite for Vue files (031 comment) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/616605 (https://phabricator.wikimedia.org/T258610) (owner: 10Eric Gardner) [22:53:30] (03CR) 10VulpesVulpes825: "> Also, this moves the play button back to the centre, reversing the decision to move it to the top-left (top-right in RTL)." [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/615435 (https://phabricator.wikimedia.org/T258584) (owner: 10Iniquity) [23:03:02] (03PS7) 10Anne Tomasevich: Recreate autocomplete search input [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/617796 (https://phabricator.wikimedia.org/T256673) [23:05:31] (03CR) 10Anne Tomasevich: Recreate autocomplete search input (033 comments) [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/617796 (https://phabricator.wikimedia.org/T256673) (owner: 10Anne Tomasevich) [23:53:18] (03PS8) 10Anne Tomasevich: Recreate autocomplete search input [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/617796 (https://phabricator.wikimedia.org/T256673)