[00:32:14] 10Multimedia, 10SDC Engineering, 10MW-1.34-notes (1.34.0-wmf.10; 2019-06-18), 10Multimedia-Current-Work, 10SDC-Statements (Depicts-on-training-wheels): Attempting to add a qualifier and then cancelling leaves a "trace" behind - https://phabricator.wikimedia.org/T224639 (10Edtadros) a:05Edtadros→03Cpar... [04:47:57] 10Multimedia, 103D, 10Mobile: Make 3D work on mobile - https://phabricator.wikimedia.org/T167423 (10Aklapper) p:05Normal→03Low [05:00:17] 10Multimedia, 10Commons, 10UploadWizard, 10media-storage: Files occasionally getting uploaded to Commons without file pages. - https://phabricator.wikimedia.org/T179884 (10Ghouston) I still see them occasionally, such as: these two: https://commons.wikimedia.org/wiki/File:Tekija_(Kladovo).JPG https://comm... [05:24:00] 10Multimedia, 10Commons, 10UploadWizard, 10media-storage: Files occasionally getting uploaded to Commons without file pages. - https://phabricator.wikimedia.org/T179884 (10Yann) This is a serious issue. Please fix as soon as possible. [05:33:34] 10Multimedia, 10Commons, 10UploadWizard, 10media-storage: Files occasionally getting uploaded to Commons without file pages. - https://phabricator.wikimedia.org/T179884 (10Marostegui) p:05Triage→03High [06:35:54] (03PS1) 10Matthias Mullie: Grey out statements based on type, not config [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517975 (https://phabricator.wikimedia.org/T224461) [06:37:04] (03PS1) 10Matthias Mullie: Make sure statements always get fed the new data [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517976 (https://phabricator.wikimedia.org/T224639) [06:37:49] (03PS1) 10Matthias Mullie: Wrap qualifier inputs on smaller screens [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517977 (https://phabricator.wikimedia.org/T224624) [06:39:25] (03PS1) 10Matthias Mullie: Don't align leading qualifiers line in the middle [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517978 (https://phabricator.wikimedia.org/T224624) [06:55:47] (03CR) 10jerkins-bot: [V: 04-1] Don't align leading qualifiers line in the middle [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517978 (https://phabricator.wikimedia.org/T224624) (owner: 10Matthias Mullie) [06:58:54] (03CR) 10jerkins-bot: [V: 04-1] Grey out statements based on type, not config [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517975 (https://phabricator.wikimedia.org/T224461) (owner: 10Matthias Mullie) [07:04:10] (03CR) 10jerkins-bot: [V: 04-1] Make sure statements always get fed the new data [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517976 (https://phabricator.wikimedia.org/T224639) (owner: 10Matthias Mullie) [07:04:30] (03CR) 10jerkins-bot: [V: 04-1] Wrap qualifier inputs on smaller screens [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517977 (https://phabricator.wikimedia.org/T224624) (owner: 10Matthias Mullie) [07:08:49] 10Multimedia, 10SDC Engineering, 10MW-1.34-notes (1.34.0-wmf.10; 2019-06-18), 10Multimedia-Current-Work, and 2 others: Attempting to add a qualifier and then cancelling leaves a "trace" behind - https://phabricator.wikimedia.org/T224639 (10matthiasmullie) Looks like testcommons also got rolled back, so the... [07:53:15] (03PS1) 10Matthias Mullie: Update lockfile/versions [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/517984 [08:08:54] (03PS1) 10Matthias Mullie: Re-sync package-lock.json [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517987 [08:39:29] (03CR) 10Cparle: "I'm inclined to agree with Stephen here - I'd like to merge this. We can deal with a combination of widgets and templates when we need to" [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/513706 (owner: 10Eric Gardner) [08:39:49] brion: how are you ? anything low key, yet helpful, i can do for VideoJS ? [09:20:18] (03CR) 10Cparle: [C: 03+2] Update lockfile/versions [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/517984 (owner: 10Matthias Mullie) [09:30:48] (03CR) 10Cparle: [C: 03+2] Add more unit tests and address max-length linter warnings [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/517937 (owner: 10Eric Gardner) [09:42:49] (03Merged) 10jenkins-bot: Update lockfile/versions [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/517984 (owner: 10Matthias Mullie) [09:44:57] (03CR) 10Matthias Mullie: "This patch is/was a POC and not in mergeable state (commit msg explicitly calls out OOUI widgets still need to be hooked up)" [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/513706 (owner: 10Eric Gardner) [09:47:43] (03CR) 10Matthias Mullie: [C: 03+2] Re-sync package-lock.json [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517987 (owner: 10Matthias Mullie) [09:48:55] 10Multimedia, 10Commons, 10MediaWiki-File-management, 10TechCom-RFC, 10Wikimedia-SVG-rendering: Re-evaluate librsvg as SVG renderer on Wikimedia wikis - https://phabricator.wikimedia.org/T40010 (10RazrFalcon) A new version of resvg was released. Now it supports `textPath` ([example](https://ibb.co/z8nyb5... [09:54:12] (03CR) 10jenkins-bot: Update lockfile/versions [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/517984 (owner: 10Matthias Mullie) [09:54:32] (03Merged) 10jenkins-bot: Add more unit tests and address max-length linter warnings [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/517937 (owner: 10Eric Gardner) [09:57:20] (03PS2) 10Matthias Mullie: Wrap qualifier inputs on smaller screens [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517977 (https://phabricator.wikimedia.org/T224624) [09:57:45] (03PS2) 10Matthias Mullie: Make sure statements always get fed the new data [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517976 (https://phabricator.wikimedia.org/T224639) [09:57:50] (03PS2) 10Matthias Mullie: Grey out statements based on type, not config [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517975 (https://phabricator.wikimedia.org/T224461) [09:57:55] (03PS2) 10Matthias Mullie: Don't align leading qualifiers line in the middle [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517978 (https://phabricator.wikimedia.org/T224624) [09:59:55] 10Multimedia, 10Commons, 10MediaWiki-Special-pages, 10Wikimedia-production-error: BadMethodCallException on Commons: /includes/specials/pagers/ImageListPager.php: Call to a member function getUrl() on a non-object (boolean) - https://phabricator.wikimedia.org/T226102 (10Lucas_Werkmeister_WMDE) a:03Lucas_W... [10:00:57] (03CR) 10jenkins-bot: Add more unit tests and address max-length linter warnings [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/517937 (owner: 10Eric Gardner) [10:02:48] 10Multimedia, 10Commons, 10MediaWiki-Special-pages, 10Patch-For-Review, 10Wikimedia-production-error: BadMethodCallException on Commons: /includes/specials/pagers/ImageListPager.php: Call to a member function getUrl() on a non-object (boolean) - https://phabricator.wikimedia.org/T226102 (10Lucas_Werkmeist... [10:04:39] 10Multimedia, 10Commons, 10MediaWiki-Special-pages, 10Patch-For-Review, 10Wikimedia-production-error: BadMethodCallException on Commons: /includes/specials/pagers/ImageListPager.php: Call to a member function getUrl() on a non-object (boolean) - https://phabricator.wikimedia.org/T226102 (10Lucas_Werkmeist... [10:11:45] (03Merged) 10jenkins-bot: Re-sync package-lock.json [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517987 (owner: 10Matthias Mullie) [10:12:07] (03PS3) 10Matthias Mullie: Don't align leading qualifiers line in the middle [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517978 (https://phabricator.wikimedia.org/T224624) [10:12:29] (03PS3) 10Matthias Mullie: Make sure statements always get fed the new data [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517976 (https://phabricator.wikimedia.org/T224639) [10:12:32] (03PS3) 10Matthias Mullie: Grey out statements based on type, not config [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517975 (https://phabricator.wikimedia.org/T224461) [10:12:35] (03PS3) 10Matthias Mullie: Wrap qualifier inputs on smaller screens [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517977 (https://phabricator.wikimedia.org/T224624) [10:13:16] (03CR) 10Matthias Mullie: [V: 03+2 C: 03+2] Grey out statements based on type, not config [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517975 (https://phabricator.wikimedia.org/T224461) (owner: 10Matthias Mullie) [10:13:40] (03PS4) 10Matthias Mullie: Make sure statements always get fed the new data [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517976 (https://phabricator.wikimedia.org/T224639) [10:13:46] (03CR) 10Matthias Mullie: [V: 03+2 C: 03+2] Make sure statements always get fed the new data [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517976 (https://phabricator.wikimedia.org/T224639) (owner: 10Matthias Mullie) [10:14:00] (03PS4) 10Matthias Mullie: Wrap qualifier inputs on smaller screens [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517977 (https://phabricator.wikimedia.org/T224624) [10:14:07] (03CR) 10Matthias Mullie: [V: 03+2 C: 03+2] Wrap qualifier inputs on smaller screens [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517977 (https://phabricator.wikimedia.org/T224624) (owner: 10Matthias Mullie) [10:14:18] (03PS4) 10Matthias Mullie: Don't align leading qualifiers line in the middle [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517978 (https://phabricator.wikimedia.org/T224624) [10:14:23] (03CR) 10Matthias Mullie: [V: 03+2 C: 03+2] Don't align leading qualifiers line in the middle [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517978 (https://phabricator.wikimedia.org/T224624) (owner: 10Matthias Mullie) [10:14:44] (03PS1) 10Matthias Mullie: Focus property when adding new property [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/518000 (https://phabricator.wikimedia.org/T223916) [10:15:18] (03CR) 10Matthias Mullie: [V: 03+2 C: 03+2] Focus property when adding new property [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/518000 (https://phabricator.wikimedia.org/T223916) (owner: 10Matthias Mullie) [10:18:04] (03CR) 10jenkins-bot: Re-sync package-lock.json [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517987 (owner: 10Matthias Mullie) [10:19:01] (03CR) 10jenkins-bot: Grey out statements based on type, not config [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517975 (https://phabricator.wikimedia.org/T224461) (owner: 10Matthias Mullie) [10:19:39] (03CR) 10jenkins-bot: Make sure statements always get fed the new data [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517976 (https://phabricator.wikimedia.org/T224639) (owner: 10Matthias Mullie) [10:20:57] (03CR) 10jenkins-bot: Don't align leading qualifiers line in the middle [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517978 (https://phabricator.wikimedia.org/T224624) (owner: 10Matthias Mullie) [10:23:24] (03CR) 10jenkins-bot: Wrap qualifier inputs on smaller screens [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517977 (https://phabricator.wikimedia.org/T224624) (owner: 10Matthias Mullie) [10:24:17] (03CR) 10jenkins-bot: Grey out statements based on type, not config [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517975 (https://phabricator.wikimedia.org/T224461) (owner: 10Matthias Mullie) [10:25:08] (03CR) 10jenkins-bot: Focus property when adding new property [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/518000 (https://phabricator.wikimedia.org/T223916) (owner: 10Matthias Mullie) [10:30:52] (03CR) 10jenkins-bot: Make sure statements always get fed the new data [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517976 (https://phabricator.wikimedia.org/T224639) (owner: 10Matthias Mullie) [10:39:35] (03CR) 10jenkins-bot: Wrap qualifier inputs on smaller screens [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517977 (https://phabricator.wikimedia.org/T224624) (owner: 10Matthias Mullie) [10:48:09] (03CR) 10jenkins-bot: Don't align leading qualifiers line in the middle [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/517978 (https://phabricator.wikimedia.org/T224624) (owner: 10Matthias Mullie) [10:54:22] (03CR) 10jenkins-bot: Focus property when adding new property [extensions/WikibaseMediaInfo] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/518000 (https://phabricator.wikimedia.org/T223916) (owner: 10Matthias Mullie) [10:54:37] 10Multimedia, 10MediaViewer, 10Browser-Support-Google-Chrome, 10Patch-For-Review: Closing image in MultimediaViewer does not revert back to article title - https://phabricator.wikimedia.org/T225387 (10Aklapper) [11:18:31] <wikibugs> 10Multimedia, 10SDC Engineering, 10UploadWizard, 10Wikidata, 10Multimedia-Current-Work: "Add metadata" in Wikimedia Upload Wizard fails to save - https://phabricator.wikimedia.org/T225148 (10matthiasmullie) The rate limit has been increased, up to 120 edits per 15 minutes for newbie users. This should al... [11:19:27] <wikibugs> 10Multimedia, 10SDC Engineering, 10UploadWizard, 10Wikidata, 10Multimedia-Current-Work: "Add metadata" in Wikimedia Upload Wizard fails to save - https://phabricator.wikimedia.org/T225148 (10matthiasmullie) a:05matthiasmullie→03Edtadros [11:21:04] <wikibugs> 10Multimedia, 10SDC Engineering, 10MW-1.34-notes (1.34.0-wmf.8; 2019-06-04), 10Multimedia-Current-Work, 10SDC-Statements (Depicts-on-training-wheels): Attempting to add a qualifier and then cancelling leaves a "trace" behind - https://phabricator.wikimedia.org/T224639 (10matthiasmullie) 05Open→03Resol... [11:21:26] <wikibugs> (03CR) 10Cparle: [C: 03+2] Fix title max length to match backend, and actually validate it [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/510905 (https://phabricator.wikimedia.org/T208539) (owner: 10Matthias Mullie) [11:24:54] <wikibugs> 10Multimedia, 10SDC Engineering, 10MW-1.34-notes (1.34.0-wmf.11; 2019-06-26), 10Multimedia-Current-Work: Switching properties with different datatype doesn't update input field if it's blank - https://phabricator.wikimedia.org/T226101 (10matthiasmullie) a:05matthiasmullie→03Edtadros [11:31:40] <wikibugs> (03Merged) 10jenkins-bot: Fix title max length to match backend, and actually validate it [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/510905 (https://phabricator.wikimedia.org/T208539) (owner: 10Matthias Mullie) [11:35:28] <wikibugs> (03CR) 10jenkins-bot: Fix title max length to match backend, and actually validate it [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/510905 (https://phabricator.wikimedia.org/T208539) (owner: 10Matthias Mullie) [11:36:37] <wikibugs> 10Multimedia, 10MediaWiki-DjVu, 10MediaWiki-Shell, 10Patch-For-Review: djvu related binaries should have shell restrictions applied - https://phabricator.wikimedia.org/T182743 (10D3r1ck01) a:05D3r1ck01→03None [11:43:49] <wikibugs> (03CR) 10Cparle: [C: 03+1] "LGTM, but not thoroughly tested" [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/339372 (owner: 10Matthias Mullie) [12:05:56] <wikibugs> 10Multimedia, 10Multimedia-Current-Work: Odd behaviour when editing qualifier property type but not value - https://phabricator.wikimedia.org/T226181 (10Cparle) [14:22:53] <wikibugs> (03CR) 10Eric Gardner: [C: 04-2] "I'm glad there's interest in moving in this direction. I'll try to get this ready to merge today/tomorrow and we can go from there." [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/513706 (owner: 10Eric Gardner) [14:31:32] <wikibugs> (03PS1) 10Reedy: Remove defined check for ParserOutput::SUPPORTS_STATELESS_TRANSFORMS [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/518038 [15:23:36] <wikibugs> 10Multimedia, 10Commons, 10MediaWiki-Special-pages, 10MW-1.34-notes (1.34.0-wmf.11; 2019-06-26), and 2 others: BadMethodCallException on Commons: /includes/specials/pagers/ImageListPager.php: Call to a member function getUrl() on a non-object (boolean) - https://phabricator.wikimedia.org/T226102 (10Jdforres... [15:26:26] <wikibugs> 10Multimedia, 10Commons, 10MediaWiki-Special-pages, 10MW-1.34-notes (1.34.0-wmf.11; 2019-06-26), and 2 others: BadMethodCallException on Commons: /includes/specials/pagers/ImageListPager.php: Call to a member function getUrl() on a non-object (boolean) - https://phabricator.wikimedia.org/T226102 (10D3r1ck01... [15:37:48] <wikibugs> 10Multimedia, 10CommonsMetadata, 10Reading-Infrastructure-Team-Backlog, 10Spike: Investigate slowness of imageinfo extended metadata (extmetadata) queries - https://phabricator.wikimedia.org/T225963 (10LGoto) p:05Triage→03Low [15:45:48] <wikibugs> (03CR) 10Jforrester: [C: 03+2] Remove defined check for ParserOutput::SUPPORTS_STATELESS_TRANSFORMS [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/518038 (owner: 10Reedy) [15:56:41] <wikibugs> (03Merged) 10jenkins-bot: Remove defined check for ParserOutput::SUPPORTS_STATELESS_TRANSFORMS [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/518038 (owner: 10Reedy) [15:58:30] <wikibugs> 10Multimedia, 10SDC Engineering, 10UploadWizard, 10Wikidata, 10Multimedia-Current-Work: "Add metadata" in Wikimedia Upload Wizard fails to save - https://phabricator.wikimedia.org/T225148 (10Edtadros) a:05Edtadros→03matthiasmullie @matthiasmullie Which environment should this be tested in? [16:03:36] <wikibugs> (03CR) 10jenkins-bot: Remove defined check for ParserOutput::SUPPORTS_STATELESS_TRANSFORMS [extensions/UploadWizard] - 10https://gerrit.wikimedia.org/r/518038 (owner: 10Reedy) [16:03:38] <wikibugs> 10Multimedia, 10SDC Engineering, 10MW-1.34-notes (1.34.0-wmf.10; 2019-06-18), 10Multimedia-Current-Work, 10SDC-Statements (Depicts-on-training-wheels): On Firefox, quantity qualifier input boxes show a mini increment/decrement UI element - https://phabricator.wikimedia.org/T224625 (10Edtadros) a:05Edtad... [16:03:41] <wikibugs> 10Multimedia, 10SDC Engineering, 10UploadWizard, 10Wikidata, 10Multimedia-Current-Work: "Add metadata" in Wikimedia Upload Wizard fails to save - https://phabricator.wikimedia.org/T225148 (10matthiasmullie) 05Open→03Resolved Production Commons. We probably shouldn't throw test garbage at it - let's a... [16:58:47] <wikibugs> 10Multimedia, 10SDC Engineering, 10Epic, 10SDC-Statements (Campaigns-jogging): Allow campaign admin to suggest statements for display in UploadWizard - https://phabricator.wikimedia.org/T224139 (10Cparle) [16:58:53] <wikibugs> 10Multimedia, 10SDC Engineering, 10MW-1.34-notes (1.34.0-wmf.10; 2019-06-18), 10Multimedia-Current-Work, 10SDC-Statements (Depicts-on-training-wheels): On Firefox, quantity qualifier input boxes show a mini increment/decrement UI element - https://phabricator.wikimedia.org/T224625 (10Volker_E) @Edtadros... [16:59:09] <wikibugs> 10Multimedia, 10SDC Engineering, 10SDC-Statements (Campaigns-jogging): Suggested statements (without values) in UploadWizard configurable via json - https://phabricator.wikimedia.org/T224141 (10Cparle) [16:59:31] <wikibugs> 10Multimedia, 10SDC Engineering, 10SDC-Statements (Campaigns-jogging): Suggested statements (without values) in UploadWizard configurable via query string - https://phabricator.wikimedia.org/T224142 (10Cparle) [16:59:43] <wikibugs> 10Multimedia, 10SDC Engineering, 10Epic, 10SDC-Statements (Campaigns-jogging): Suggested statements WITH values in UploadWizard configurable via json and query string - https://phabricator.wikimedia.org/T224143 (10Cparle) [17:00:11] <wikibugs> 10Multimedia, 10SDC Design, 10SDC Engineering, 10SDC-Statements (Campaigns-jogging): Mechanism to allow a campaign admin to add help text for suggested statements - https://phabricator.wikimedia.org/T224147 (10Cparle) [17:30:52] <wikibugs> 10Multimedia, 10WikibaseMediaInfo, 10Performance-Team (Radar), 10Wikimedia-production-error: Warning: Failed to find "wikibasemediainfo-statements-unsupported-property-type-content" in message cache - https://phabricator.wikimedia.org/T226203 (10Krinkle) [18:20:53] <brion> thedj: I think we're mostly down to testing, checking for regressions, and deciding if we need to make any changes to the UX. (I'm most concerned about the controls on small players, which if it's problematic could lead us to go back to doing a larger popup again) [18:51:29] <wikibugs> 10Multimedia, 10Commons, 10MediaWiki-File-management, 10TechCom-RFC, 10Wikimedia-SVG-rendering: Re-evaluate librsvg as SVG renderer on Wikimedia wikis - https://phabricator.wikimedia.org/T40010 (10Glrx) I've skimmed the comments going back about 2.5 years. It looks like most bases are covered. Commons m... [19:36:47] <wikibugs> (03CR) 10TheDJ: [C: 03+2] Apply LanguageCode::bcp47 for the HTML attribute 'srclang' [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/517182 (owner: 10Fomafix) [19:38:05] <thedj> brion: yeah, i have some concerns about that too. [19:39:16] <thedj> brion: we can adapt the responsive layouting a bit.. it's in videojs core now, but you need to generate your own .css based on their less rules if you really want to make proper use of it. [19:47:51] <wikibugs> (03Merged) 10jenkins-bot: Apply LanguageCode::bcp47 for the HTML attribute 'srclang' [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/517182 (owner: 10Fomafix) [19:48:28] <thedj> brion: also, can this be closed ? https://phabricator.wikimedia.org/T200813 [19:50:01] <brion> Ah nice I'll check out the responsive mode in a bit [19:50:09] <brion> Closed :) [19:50:33] <thedj> https://docs.videojs.com/tutorial-layout.html#responsive-mode [19:51:57] <thedj> i played a bit with it during the hackathon, and concluded it might be better to compile an adapted CSS file, rather than trying to override all those classes ourselves, but it was working. [19:52:11] <brion> Cool [19:56:53] <wikibugs> (03PS1) 10TheDJ: TimedText: VttReader [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/518128 (https://phabricator.wikimedia.org/T116154) [19:59:04] <wikibugs> (03CR) 10jerkins-bot: [V: 04-1] TimedText: VttReader [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/518128 (https://phabricator.wikimedia.org/T116154) (owner: 10TheDJ) [20:29:45] <wikibugs> (03PS2) 10TheDJ: TimedText: Add a audio captioning plugin for video.js [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/511106 (https://phabricator.wikimedia.org/T121387) [20:30:29] <James_F> thedj: Showy. :-) [20:31:07] <James_F> brion: I'll hold back the beta feature unless the train goes out this week, because without the data loading patch wmf.8 is pretty bad. I guess I could back-port that, but… eh. Effort. :-) [20:31:26] <brion> James_F: no worries, we'll wait a bit :D [20:31:37] <wikibugs> (03PS3) 10TheDJ: TimedText: Add a audio captioning plugin for video.js [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/511106 (https://phabricator.wikimedia.org/T121387) [20:31:43] <James_F> But videojs is *so much* nicer. [20:31:45] <brion> :D [20:33:08] <brion> note i'll be in new york next week giving at talk at https://bigapple.video/ on ogv.js and some possible future directions for similar tech, so i'll be busy much of the week and on eastern time :) [20:33:32] <thedj> hmm, we have the SRT AND the VTT subtitles in the html for video.js [20:33:38] <brion> oh nice i'll check out that audi oone later [20:34:11] <James_F> thedj: Oh, we do? Oops. [20:34:16] <brion> thedj: it should only pick out the vtt ones, but we shouldn't actually be advertising them yeah [20:34:24] <thedj> brion: the audio plugin is a a good enough stop gap i think [20:34:42] <brion> thedj: is that a local file or an InstantCommons one? [20:34:44] <thedj> not perfect, but good enough for an initial thing [20:34:55] <thedj> and it's accessible ;) [20:34:59] <brion> iirc i added a hack for file repo to include both sets for compatibility, maybe it's not filtering on the other end [20:35:04] <brion> *api file repo [20:35:20] <thedj> instantcommons thing possibly ? [20:38:26] <thedj> yeah, so both are included, because instantcommons and backwardscompatibilty with current deployed code indeed. [20:38:32] <thedj> and then videojs just lists them all. [20:38:46] <James_F> Ick. [20:40:55] <thedj> i think i can fix this easily. i'll give it a go. [20:41:25] <brion> \o/ yay [20:42:31] <wikibugs> (03CR) 10jerkins-bot: [V: 04-1] TimedText: Add a audio captioning plugin for video.js [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/511106 (https://phabricator.wikimedia.org/T121387) (owner: 10TheDJ) [20:45:23] <brion> James_F: (note that dupe subtitles issue shouldn't block production if it's InstantCommons-only, since the fix will be client-side and we don't use any ForeignAPIRepos in prod) [20:45:46] <James_F> Agreed. [20:53:58] <wikibugs> (03PS4) 10TheDJ: TimedText: Add a audio captioning plugin for video.js [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/511106 (https://phabricator.wikimedia.org/T121387) [20:58:22] <wikibugs> (03PS1) 10TheDJ: Video.js: Remove SRT subtitle files from the player [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/518148 [21:07:07] <wikibugs> (03CR) 10jerkins-bot: [V: 04-1] TimedText: Add a audio captioning plugin for video.js [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/511106 (https://phabricator.wikimedia.org/T121387) (owner: 10TheDJ) [21:07:48] <thedj> we don't allow named colors in less ?? who came up with that idea.... [21:08:43] <MatmaRex> i know, right [21:09:30] <MatmaRex> thedj: i believe the idea is that you're supposed to only use colours from the wikimedia style guide, which are documented… somewhere [21:10:03] <MatmaRex> so it's not a completely stupid limitation, but only kind of ;) [21:10:10] <thedj> cool.. call me when they have colors for subtitles ;) [21:10:42] <MatmaRex> thedj: there are like fifteen named shades of grey [21:10:48] <MatmaRex> (which is also kind of stupid, but oh well) [21:10:53] <James_F> thedj: They have something like five dozen colours. [21:11:07] <James_F> What MatmaRex said. [21:11:10] <MatmaRex> so you can probably pick something [21:11:28] <wikibugs> (03CR) 10Brion VIBBER: "Woohoo! One minor nitpick I noticed in the hide/show logic, comment inline." (031 comment) [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/511106 (https://phabricator.wikimedia.org/T121387) (owner: 10TheDJ) [21:11:34] <MatmaRex> pretty sure there's nothing yellow though [21:11:50] <thedj> well these will be yellow on block and they change them if they want to :) [21:11:52] <MatmaRex> https://design.wikimedia.org/style-guide/visual-style_colors.html [21:12:09] <thedj> also i just googled "color code yellow" [21:12:18] <MatmaRex> :D [21:13:36] <wikibugs> (03CR) 10Brion VIBBER: [C: 03+2] "Does what it says on the tin." [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/518148 (owner: 10TheDJ) [21:26:39] <wikibugs> (03Merged) 10jenkins-bot: Video.js: Remove SRT subtitle files from the player [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/518148 (owner: 10TheDJ) [21:27:19] <brion> wooohoooo [21:47:12] <thedj> bed time. [21:47:13] <wikibugs> (03PS5) 10TheDJ: TimedText: Add a audio captioning plugin for video.js [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/511106 (https://phabricator.wikimedia.org/T121387) [22:00:23] <wikibugs> (03CR) 10jerkins-bot: [V: 04-1] TimedText: Add a audio captioning plugin for video.js [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/511106 (https://phabricator.wikimedia.org/T121387) (owner: 10TheDJ) [22:01:32] <wikibugs> (03PS6) 10TheDJ: TimedText: Add a audio captioning plugin for video.js [extensions/TimedMediaHandler] - 10https://gerrit.wikimedia.org/r/511106 (https://phabricator.wikimedia.org/T121387) [22:29:26] <wikibugs> (03PS1) 10Jforrester: video.js: Update from v7.4.1 to v7.5.4 [extensions/TimedMediaHandler] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/518163 [22:29:39] <wikibugs> (03CR) 10Jforrester: [C: 03+2] video.js: Update from v7.4.1 to v7.5.4 [extensions/TimedMediaHandler] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/518163 (owner: 10Jforrester) [22:42:56] <wikibugs> (03Merged) 10jenkins-bot: video.js: Update from v7.4.1 to v7.5.4 [extensions/TimedMediaHandler] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/518163 (owner: 10Jforrester) [22:43:12] <wikibugs> (03PS1) 10Jforrester: video.js: Local patch for subtitle loading behavior [extensions/TimedMediaHandler] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/518167 (https://phabricator.wikimedia.org/T222763) [22:43:17] <wikibugs> (03CR) 10Jforrester: [C: 03+2] video.js: Local patch for subtitle loading behavior [extensions/TimedMediaHandler] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/518167 (https://phabricator.wikimedia.org/T222763) (owner: 10Jforrester) [22:43:42] <James_F> brion: It turns out that I'm impatient. :-) [22:47:25] <wikibugs> (03PS1) 10Smalyshev: Add more checks for failing test [extensions/WikibaseMediaInfo] - 10https://gerrit.wikimedia.org/r/518168 (https://phabricator.wikimedia.org/T222416) [22:59:35] <wikibugs> (03Merged) 10jenkins-bot: video.js: Local patch for subtitle loading behavior [extensions/TimedMediaHandler] (wmf/1.34.0-wmf.8) - 10https://gerrit.wikimedia.org/r/518167 (https://phabricator.wikimedia.org/T222763) (owner: 10Jforrester) [23:05:52] <wikibugs> 10Multimedia, 10Language-strategy, 10MediaWiki-Internationalization, 10SDC Engineering, and 2 others: Add the Saami languages to Commons - https://phabricator.wikimedia.org/T224293 (10Yupik) [23:07:59] <James_F> brion: It's alive! [23:07:59] <wikibugs> 10Multimedia, 10Language-strategy, 10MediaWiki-Internationalization, 10SDC Engineering, and 2 others: Add the Saami languages to Commons - https://phabricator.wikimedia.org/T224293 (10Yupik) It was already possibly to use Southern and Northern Saami for captions, and now Inari and Skolt Saami can be too. T... [23:50:41] <brion> \o/