[00:19:45] 10Citoid, 10VisualEditor, 06Community-Tech, 10RESTBase, and 3 others: Consider removing basefields option from citoid API - https://phabricator.wikimedia.org/T149619#3230200 (10kaldari) @Mvolz: In that case, I'm going to remove it from the #community-tech board for now. Please ping me if anything changes. [00:19:50] 10Citoid, 10VisualEditor, 10RESTBase, 10RESTBase-API, and 2 others: Consider removing basefields option from citoid API - https://phabricator.wikimedia.org/T149619#3230201 (10kaldari) [01:14:13] 10Cite, 05MW-1.29-release (WMF-deploy-2017-04-25_(1.29.0-wmf.21)), 07Technical-Debt: Drop the pointless "AllowCiteGroups" config setting - https://phabricator.wikimedia.org/T161144#3230370 (10Jdforrester-WMF) [03:02:19] 10VisualEditor, 06Collaboration-Team-Triage, 10Flow, 10MediaWiki-Interface, and 4 others: content-holding
should only contain the page text - https://phabricator.wikimedia.org/T37247#3230407 (10Krinkle) It's not clear to me why a separate class is needed here. How is `mw-parser-output` semantically d... [06:14:34] 10VisualEditor, 10VisualEditor-EditingTools, 10VisualEditor-MediaWiki, 05MW-1.29-release (WMF-deploy-2017-04-04_(1.29.0-wmf.19)), 15User-Ryasmeen: Pasted ISBNs should trigger the ISBN typing auto-linking - https://phabricator.wikimedia.org/T161152#3230482 (10Jonesey95) This, or something like it (ISBN wi... [08:26:33] 10VisualEditor, 10OOjs-UI, 07Browser-Support-Firefox, 07Regression: Welcome dialog in VE has scrollbar in Firefox - https://phabricator.wikimedia.org/T164056#3230645 (10Schnark) >>! In T164056#3229004, @Jdforrester-WMF wrote: > Is this on a Linux device? Yes [08:44:04] 10VisualEditor, 06Community-Liaisons, 07User-notice: Support WMF communities in run-up to switching EditPage over to OOUI - https://phabricator.wikimedia.org/T162849#3230737 (10Nemo_bis) The red "Cancel" link seems like a red wikilink. It makes me think that nothing will happen if I click on it. [09:22:34] (03PS1) 10Esanders: Remove duplicate lint rule [extensions/TemplateData] - 10https://gerrit.wikimedia.org/r/351601 [09:23:24] (03PS1) 10Esanders: Fix ,->; typo [extensions/TemplateData] - 10https://gerrit.wikimedia.org/r/351600 [10:47:03] 10VisualEditor, 10VisualEditor-MediaWiki, 10VisualEditor-MediaWiki-2017WikitextEditor, 06Collaboration-Team-Triage, and 6 others: Show a different label for the current mode in the editor mode selector (not "switch to…", as it's confusing) - https://phabricator.wikimedia.org/T162864#3177639 (10Trizek-WMF)... [10:53:17] 10VisualEditor, 10VisualEditor-DataModel, 10Parsoid, 13Patch-For-Review: Parsoid is generating [[Foo|Foo]] instead of [[Foo]] for some VE edits - https://phabricator.wikimedia.org/T153107#3231076 (10Trizek-WMF) Go the case this morning [[ https://fr.wikipedia.org/w/index.php?title=Gall%C3%A9co&type=revisio... [14:28:11] (03CR) 10Bartosz Dziewoński: [C: 032] Fix ,->; typo [extensions/TemplateData] - 10https://gerrit.wikimedia.org/r/351600 (owner: 10Esanders) [14:31:56] (03Merged) 10jenkins-bot: Fix ,->; typo [extensions/TemplateData] - 10https://gerrit.wikimedia.org/r/351600 (owner: 10Esanders) [14:42:23] (03CR) 10jenkins-bot: Fix ,->; typo [extensions/TemplateData] - 10https://gerrit.wikimedia.org/r/351600 (owner: 10Esanders) [14:54:57] (03CR) 10Divec: [C: 032] Add logging to the rebase server and client [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/344730 (owner: 10Catrope) [14:57:28] (03Merged) 10jenkins-bot: Add logging to the rebase server and client [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/344730 (owner: 10Catrope) [14:59:49] (03CR) 10jenkins-bot: Add logging to the rebase server and client [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/344730 (owner: 10Catrope) [15:17:54] 10VisualEditor, 06Collaboration-Team-Triage, 10Flow, 10MediaWiki-Interface, and 4 others: content-holding
should only contain the page text - https://phabricator.wikimedia.org/T37247#3231731 (10Anomie) >>! In T37247#3230407, @Krinkle wrote: > It's not clear to me why a separate class is needed here.... [15:31:43] (03CR) 10Jforrester: [C: 032] Remove duplicate lint rule [extensions/TemplateData] - 10https://gerrit.wikimedia.org/r/351601 (owner: 10Esanders) [15:32:46] 10VisualEditor, 10Parsoid, 10Parsoid-DOM, 13Patch-For-Review: Parsoid doesn't recognize interwiki shortcuts in the href attribute - https://phabricator.wikimedia.org/T136653#2343172 (10cscott) Well, it shouldn't be `href="de:Foo"` because that ought to have `de` parsed as a URL protocol. But I could be co... [15:35:04] (03Merged) 10jenkins-bot: Remove duplicate lint rule [extensions/TemplateData] - 10https://gerrit.wikimedia.org/r/351601 (owner: 10Esanders) [15:36:15] (03CR) 10jenkins-bot: Remove duplicate lint rule [extensions/TemplateData] - 10https://gerrit.wikimedia.org/r/351601 (owner: 10Esanders) [16:12:50] (03PS7) 10DLynch: MobileContext: fix button spacing [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/350962 (https://phabricator.wikimedia.org/T163473) [16:14:40] 10VisualEditor, 10Parsoid, 10Parsoid-DOM, 13Patch-For-Review: Parsoid doesn't recognize interwiki shortcuts in the href attribute - https://phabricator.wikimedia.org/T136653#2343172 (10matmarex) >>! In T136653#3231826, @cscott wrote: > you could imagine the PHP side doing a redirect if you actually hit htt... [16:46:16] (03CR) 10Jforrester: "Not tested. Required for child commit to work, but this might do odd things." [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/351352 (owner: 10Jforrester) [16:59:01] (03CR) 10DLynch: [C: 04-1] Specify suggested parent node types for tables (031 comment) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/350609 (https://phabricator.wikimedia.org/T164016) (owner: 10Esanders) [17:01:54] 10VisualEditor, 10VisualEditor-EditingTools, 10VisualEditor-MediaWiki, 05MW-1.29-release (WMF-deploy-2017-04-04_(1.29.0-wmf.19)), 15User-Ryasmeen: Pasted ISBNs should trigger the ISBN typing auto-linking - https://phabricator.wikimedia.org/T161152#3232294 (10matmarex) That's probably a case of {T162291}.... [17:02:13] 10VisualEditor, 10Parsoid, 10Parsoid-DOM, 13Patch-For-Review: Parsoid doesn't recognize interwiki shortcuts in the href attribute - https://phabricator.wikimedia.org/T136653#3232296 (10cscott) In that case, I have no problems. Although we should be careful about precedence, just in case somebody out there... [17:29:38] 10VisualEditor, 10Parsoid, 10Parsoid-DOM, 13Patch-For-Review: Parsoid doesn't recognize interwiki shortcuts in the href attribute - https://phabricator.wikimedia.org/T136653#3232392 (10ssastry) Ok, so, it looks like there are a bunch of things to be resolved here wrt. these shortcuts. There are 3 possibly... [17:32:02] 10VisualEditor, 10WikiEditor, 07Regression: Toolbar collapses while loading, causing page content to jump - https://phabricator.wikimedia.org/T163793#3232410 (10Esanders) Having discussed with @Catrope and @TrevorParscal (the original authors) I think the best approach is to get rid of some of WikiEditor's m... [17:32:18] 10VisualEditor, 10Parsoid, 10Parsoid-DOM, 13Patch-For-Review: Parsoid doesn't recognize interwiki shortcuts in the href attribute - https://phabricator.wikimedia.org/T136653#3232412 (10Jdforrester-WMF) >>! In T136653#3232392, @ssastry wrote: > Ok, so, it looks like there are a bunch of things to be resolve... [17:32:18] (03PS1) 10Esanders: Make toolbar and dialog part of the core module [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/351681 (https://phabricator.wikimedia.org/T163793) [17:36:42] 10VisualEditor, 10VisualEditor-MediaWiki-2017WikitextEditor: You need to double-cursor to move across an inline slug, which makes editing "blank" lines in NWE confusing - https://phabricator.wikimedia.org/T162916#3232430 (10Esanders) We may be able to disable a bunch of the slugging features in source mode as... [17:37:09] (03CR) 10jerkins-bot: [V: 04-1] Make toolbar and dialog part of the core module [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/351681 (https://phabricator.wikimedia.org/T163793) (owner: 10Esanders) [17:39:18] 10VisualEditor, 10VisualEditor-MediaWiki-2017WikitextEditor, 06Community-Tech, 10MediaWiki-extensions-CodeMirror, 07Performance: Investigation: CodeMirror + New WikiText Editor performance issues - https://phabricator.wikimedia.org/T161054#3232437 (10kaldari) [17:44:55] 10VisualEditor, 10VisualEditor-MediaWiki, 10VisualEditor-MediaWiki-2017WikitextEditor, 07Design: Don't label the "oh well" edit summary box with "Describe your changes" when saving a new section - https://phabricator.wikimedia.org/T163026#3232445 (10Esanders) ...or we can just break consistency with the ol... [17:52:57] 10VisualEditor, 10Parsoid: Parsoid does not escape language prefixes in internal link hrefs, causing html2html round-trip failures - https://phabricator.wikimedia.org/T95850#1201686 (10cscott) Related to {T136653} ? [17:54:30] 10VisualEditor, 10Parsoid, 10Parsoid-DOM, 13Patch-For-Review: Parsoid doesn't recognize interwiki shortcuts in the href attribute - https://phabricator.wikimedia.org/T136653#2343172 (10cscott) VE should be escaping colon characters in titles, or else the URL in the href is not valid. Either `./File:Foo.jp... [18:21:34] 10VisualEditor, 10Parsoid, 10Parsoid-DOM, 13Patch-For-Review: Parsoid doesn't recognize interwiki shortcuts in the href attribute - https://phabricator.wikimedia.org/T136653#3232588 (10cscott) cf "not valid" -> step (5)(1) from https://url.spec.whatwg.org/#scheme-state linked from https://html.spec.whatwg.... [18:27:49] 10VisualEditor, 06Community-Liaisons, 07User-notice: Support WMF communities in run-up to switching EditPage over to OOUI - https://phabricator.wikimedia.org/T162849#3232594 (10Niharika) On iOS -Safari Before: {F7894506} After: {F7894507} JS enabled. >>! In T162849#3226289, @Jdforrester-WMF wrote: >> On... [18:35:47] 10VisualEditor, 06Community-Liaisons, 07User-notice: Support WMF communities in run-up to switching EditPage over to OOUI - https://phabricator.wikimedia.org/T162849#3232609 (10Aklapper) [18:39:43] 10VisualEditor, 06Community-Liaisons, 07User-notice: Support WMF communities in run-up to switching EditPage over to OOUI - https://phabricator.wikimedia.org/T162849#3232630 (10Jdforrester-WMF) >>! In T162849#3232594, @Niharika wrote: > Will be happy to, if someone confirms this is a bug and not intentional.... [18:54:30] (03PS2) 10Jforrester: Make toolbar and dialog part of the core module [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/351681 (https://phabricator.wikimedia.org/T163793) (owner: 10Esanders) [18:55:48] (03CR) 10Jforrester: [C: 031] Make toolbar and dialog part of the core module [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/351681 (https://phabricator.wikimedia.org/T163793) (owner: 10Esanders) [21:45:25] 10VisualEditor, 10VisualEditor-VisualDiffs, 15User-Ryasmeen, 07Verified: Visual Diff showing raw HTML after switching back from NWE - https://phabricator.wikimedia.org/T163945#3233461 (10Ryasmeen) [21:48:11] 10VisualEditor, 10VisualEditor-VisualDiffs, 15User-Ryasmeen, 07Verified, 07Verified-in-Phase0: Visual Diff showing raw HTML after switching back from NWE - https://phabricator.wikimedia.org/T163945#3233463 (10Ryasmeen)