[01:40:50] <wikibugs>	 10VisualEditor, 10VisualEditor-DataModel, 5WMF-deploy-2015-02-18_(1.25wmf18), 5§ VisualEditor Q3 Blockers: class="wikitable wikitable" corrupted to class="wikitable" - https://phabricator.wikimedia.org/T88148#1058089 (10Verdy_p) I can't understand why a duplicate class name is acceptable. Of course you mus...
[01:42:41] <wikibugs>	 10VisualEditor, 10VisualEditor-ContentEditable, 7Browser-Support-Firefox, 5WMF-deploy-2015-02-25_(1.25wmf19), 5§ VisualEditor Q3 Blockers: [Regression pre-wmf19] Cursoring is totally broken in Betalabs with Firefox - https://phabricator.wikimedia.org/T90306#1058090 (10Ryasmeen) Still getting this issue i...
[01:42:53] <wikibugs>	 10VisualEditor, 10VisualEditor-ContentEditable, 7Browser-Support-Firefox, 5WMF-deploy-2015-02-25_(1.25wmf19), 5§ VisualEditor Q3 Blockers: [Regression pre-wmf19] Cursoring is totally broken in Betalabs with Firefox - https://phabricator.wikimedia.org/T90306#1058091 (10Ryasmeen) 5Resolved>3Open
[01:53:34] <grrrit-wm>	 (03PS1) 10Ori.livneh: Emit trace.init.{enter,exit} events [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192256 
[02:41:15] <grrrit-wm>	 (03PS1) 10Catrope: Update VE core submodule to master (8434ab8) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192259 
[02:43:11] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Update VE core submodule to master (8434ab8) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192259 (owner: 10Catrope)
[03:19:47] <grrrit-wm>	 (03PS2) 10Catrope: Add clipboard tests and fix transclusion clipboard logic [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192244 (owner: 10Esanders)
[03:21:20] <wikibugs>	 10VisualEditor, 10VisualEditor-ContentEditable, 7Browser-Support-Firefox, 5WMF-deploy-2015-02-25_(1.25wmf19), 5§ VisualEditor Q3 Blockers: [Regression pre-wmf19] Cursoring is totally broken in Betalabs with Firefox - https://phabricator.wikimedia.org/T90306#1058110 (10Catrope) >>! In T90306#1058090, @Rya...
[03:25:59] <grrrit-wm>	 (03PS2) 10Catrope: Update VE core submodule to master (8434ab8) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192259 
[03:26:01] <grrrit-wm>	 (03PS3) 10Catrope: Fix transclusion clipboard logic [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192244 (owner: 10Esanders)
[03:31:51] <grrrit-wm>	 (03CR) 10Catrope: [C: 032] Fix transclusion clipboard logic [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192244 (owner: 10Esanders)
[03:33:12] <wikibugs>	 10VisualEditor, 10VisualEditor-ContentEditable, 7Browser-Support-Firefox, 5WMF-deploy-2015-02-25_(1.25wmf19), 5§ VisualEditor Q3 Blockers: [Regression pre-wmf19] Cursoring is totally broken in Betalabs with Firefox - https://phabricator.wikimedia.org/T90306#1058116 (10Catrope) I've fixed the failing test...
[03:33:39] <grrrit-wm>	 (03Merged) 10jenkins-bot: Fix transclusion clipboard logic [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192244 (owner: 10Esanders)
[04:15:12] <grrrit-wm>	 (03CR) 10Jforrester: [C: 032] Update VE core submodule to master (8434ab8) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192259 (owner: 10Catrope)
[04:17:08] <grrrit-wm>	 (03Merged) 10jenkins-bot: Update VE core submodule to master (8434ab8) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192259 (owner: 10Catrope)
[04:22:10] <grrrit-wm>	 (03CR) 10Catrope: "How is this different from trace.activate.{enter,exit}? activate.enter fires in the activate() method, right after init.enter, and activat" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192256 (owner: 10Ori.livneh)
[04:43:36] <wikibugs>	 10VisualEditor, 10VisualEditor-ContentEditable, 7Browser-Support-Firefox, 5WMF-deploy-2015-02-25_(1.25wmf19), 5§ VisualEditor Q3 Blockers: [Regression pre-wmf19] Cursoring is totally broken in Betalabs with Firefox - https://phabricator.wikimedia.org/T90306#1058139 (10Ryasmeen) 5Open>3Resolved
[04:43:54] <wikibugs>	 10VisualEditor, 10VisualEditor-ContentEditable, 7Browser-Support-Firefox, 5WMF-deploy-2015-02-25_(1.25wmf19), 5§ VisualEditor Q3 Blockers: [Regression pre-wmf19] Cursoring is totally broken in Betalabs with Firefox - https://phabricator.wikimedia.org/T90306#1055804 (10Ryasmeen) Verified the fix in Betalabs
[05:58:21] <grrrit-wm>	 (03PS1) 10Legoktm: composer.json: Add description field [oojs/ui] - 10https://gerrit.wikimedia.org/r/192271 
[05:58:23] <grrrit-wm>	 (03PS1) 10Legoktm: Add "composer test" command to lint PHP files and run phpcs [oojs/ui] - 10https://gerrit.wikimedia.org/r/192272 
[06:26:03] <grrrit-wm>	 (03Abandoned) 10Ori.livneh: Emit trace.init.{enter,exit} events [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192256 (owner: 10Ori.livneh)
[07:21:55] <wikibugs>	 10VisualEditor, 10VisualEditor-Performance: Reconsider OO.getHash - https://phabricator.wikimedia.org/T90413#1058258 (10ori) 3NEW
[08:46:27] <grrrit-wm>	 (03PS10) 10Krinkle: mw.ViewPageTarget: Use CSS for toolbar transition [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192047 (https://phabricator.wikimedia.org/T89543) 
[08:46:29] <grrrit-wm>	 (03PS1) 10Krinkle: mw.ViewPageTarget: Remove JS animations from transformPage() [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192281 (https://phabricator.wikimedia.org/T89543) 
[08:51:24] <grrrit-wm>	 (03PS1) 10Krinkle: mw.ViewPageTarget.init: Add missing 'return target' [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192285 
[08:51:52] <grrrit-wm>	 (03CR) 10Krinkle: "This broke section edit links as the handler did not forward the return value. I had this fixed locally but forgot to include in the patch" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192081 (owner: 10Krinkle)
[08:53:15] <Krinkle>	 Krenair: Could you review quickfix https://gerrit.wikimedia.org/r/192285 ?
[08:53:25] <Krinkle>	 (good morning :) )
[09:08:44] <grrrit-wm>	 (03PS2) 10Krinkle: mw.ViewPageTarget: Remove JS animations from transformPage() [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192281 (https://phabricator.wikimedia.org/T89543) 
[09:08:46] <grrrit-wm>	 (03PS11) 10Krinkle: mw.ViewPageTarget: Use CSS for toolbar transition [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192047 (https://phabricator.wikimedia.org/T89543) 
[09:10:46] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] mw.ViewPageTarget: Use CSS for toolbar transition [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192047 (https://phabricator.wikimedia.org/T89543) (owner: 10Krinkle)
[09:12:13] <grrrit-wm>	 (03CR) 10Krinkle: "recheck" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192047 (https://phabricator.wikimedia.org/T89543) (owner: 10Krinkle)
[09:15:30] <grrrit-wm>	 (03CR) 10Krinkle: "@Roan: The up-then-down seems specific to Firefox and is now fixed I think." [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192047 (https://phabricator.wikimedia.org/T89543) (owner: 10Krinkle)
[09:25:59] <wikibugs>	 10VisualEditor, 10VisualEditor-ContentEditable, 10VisualEditor-MediaWiki: Initiating VisualEditor via "Edit" button Firefox scrolls to bottom of page - https://phabricator.wikimedia.org/T90420#1058373 (10Krinkle)
[10:28:03] <grrrit-wm>	 (03CR) 10Esanders: "The reason I was avoiding this approach is that it uses $.css (i.e. getComputedStyle) a whole bunch of times on every node in every templa" [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192241 (https://phabricator.wikimedia.org/T52036) (owner: 10Bartosz Dziewoński)
[10:36:54] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki-Links, 7Design, 5§ VisualEditor Q3 Blockers: Search interface for link dialogue - https://phabricator.wikimedia.org/T76397#1058485 (10Pginer-WMF) For the option to create a red link, we may consider a language more related to the user action. So instead of "Page doe...
[10:50:05] <grrrit-wm>	 (03PS1) 10Krinkle: mw.ViewPageTarget: Use CSS for toolbar transition (alt) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192292 (https://phabricator.wikimedia.org/T89543) 
[10:53:53] <grrrit-wm>	 (03CR) 10Krinkle: "I've got an alternative patch (I40eec27ea014) that puts the transform on the entire target. That removes the need to keep content and tool" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192047 (https://phabricator.wikimedia.org/T89543) (owner: 10Krinkle)
[11:26:06] <grrrit-wm>	 (03PS1) 10Amire80: [BrowserTest] Remove upload.rb - it's fully replaced with the gem [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192293 
[11:27:49] <grrrit-wm>	 (03CR) 10Zfilipin: [C: 032] [BrowserTest] Remove upload.rb - it's fully replaced with the gem [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192293 (owner: 10Amire80)
[11:27:55] <grrrit-wm>	 (03PS2) 10Amire80: [BrowserTest] Remove upload.rb - it's fully replaced with the gem [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192293 
[11:46:45] <grrrit-wm>	 (03PS2) 10Krinkle: mw.ViewPageTarget: Use CSS for toolbar transition [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192292 (https://phabricator.wikimedia.org/T89543) 
[11:48:01] <grrrit-wm>	 (03Abandoned) 10Krinkle: mw.ViewPageTarget: Use CSS for toolbar transition [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192292 (https://phabricator.wikimedia.org/T89543) (owner: 10Krinkle)
[11:57:57] <grrrit-wm>	 (03CR) 10Amire80: [C: 032] [BrowserTest] Remove upload.rb - it's fully replaced with the gem [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192293 (owner: 10Amire80)
[11:59:54] <grrrit-wm>	 (03Merged) 10jenkins-bot: [BrowserTest] Remove upload.rb - it's fully replaced with the gem [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192293 (owner: 10Amire80)
[12:16:21] <grrrit-wm>	 (03CR) 10Krinkle: "Yeah, but there is one case where they are appropriate: When your DOM order does not reflect the desired stacking order. If our toolbar wo" [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192075 (owner: 10Krinkle)
[12:17:52] <grrrit-wm>	 (03PS2) 10Krinkle: ve.ui.Toolbar: Add z-index to oo-ui-toolbar-bar [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192075 
[12:18:07] <grrrit-wm>	 (03Abandoned) 10Krinkle: ve.ui.Toolbar: Add z-index to oo-ui-toolbar-bar [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192075 (owner: 10Krinkle)
[12:29:31] <grrrit-wm>	 (03PS12) 10Krinkle: mw.ViewPageTarget: Use CSS for toolbar transition [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192047 (https://phabricator.wikimedia.org/T89543) 
[12:32:47] <grrrit-wm>	 (03CR) 10Krinkle: "I've incorporated Ed's overall approach of using overflow/height and fixed the remaining issues with it not transitioning during teardown " [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192047 (https://phabricator.wikimedia.org/T89543) (owner: 10Krinkle)
[12:36:24] <grrrit-wm>	 (03CR) 10Krinkle: "I've settled on using height/overflow (like Ed did in this commit) and addressed the issue with it not transitioning during teardown, firs" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192072 (https://phabricator.wikimedia.org/T76523) (owner: 10Esanders)
[12:49:35] <grrrit-wm>	 (03CR) 10Krinkle: Show the toolbar as soon as the target has been activated (031 comment) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192072 (https://phabricator.wikimedia.org/T76523) (owner: 10Esanders)
[12:55:58] <grrrit-wm>	 (03CR) 10Krinkle: "I've attempted a rebase locally of this on top of I30a7b69b77b and it seems quite smooth. I like how the dummy toolbar contents gets silen" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192072 (https://phabricator.wikimedia.org/T76523) (owner: 10Esanders)
[13:00:25] <wikibugs>	 10VisualEditor, 5Patch-For-Review: VisualEditor language screenshot scenario for page settings is broken - https://phabricator.wikimedia.org/T86634#1058713 (10Amire80) 5Open>3Resolved
[13:00:41] <wikibugs>	 10VisualEditor, 7Browser-test-bug: VisualEditor language screenshot scenario for page settings is broken - https://phabricator.wikimedia.org/T86634#973109 (10Amire80)
[13:13:50] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki: Browser title of editing page in VisualEditor is not correct - https://phabricator.wikimedia.org/T90433#1058722 (10Bugreporter) 3NEW
[13:21:15] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki: Browser title of editing page in VisualEditor is not correct (drops namespace prefix?) - https://phabricator.wikimedia.org/T90433#1058737 (10Aklapper) p:5High>3Triage
[13:44:30] <wikibugs>	 10VisualEditor, 7Browser-test-bug: in language screenshots there must be a way to add more margins around the element that is being cropped - https://phabricator.wikimedia.org/T87786#1058779 (10Amire80) a:3Vikassy
[13:45:33] <wikibugs>	 10VisualEditor, 7Browser-test-bug: in language screenshots there must be a way to add more margins around the element that is being cropped - https://phabricator.wikimedia.org/T87786#998863 (10Amire80) Assigning to @Vikassy, who volunteered on IRC :)  This will probably have to be changed in the screenshot gem.
[14:02:29] <grrrit-wm>	 (03PS1) 10Amire80: [BrowserTest] Update Gemfile.lock [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192310 
[14:02:31] <grrrit-wm>	 (03PS1) 10Amire80: [BrowserTest] Define Rubocop LineLength as 100 [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192311 
[15:16:45] <grrrit-wm>	 (03CR) 10Esanders: Show the toolbar as soon as the target has been activated (031 comment) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192072 (https://phabricator.wikimedia.org/T76523) (owner: 10Esanders)
[15:30:05] <grrrit-wm>	 (03PS4) 10Esanders: Show the toolbar as soon as the target has been activated [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192072 (https://phabricator.wikimedia.org/T76523) 
[15:32:10] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Show the toolbar as soon as the target has been activated [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192072 (https://phabricator.wikimedia.org/T76523) (owner: 10Esanders)
[15:54:51] <grrrit-wm>	 (03CR) 10Catrope: [C: 032] mw.ViewPageTarget.init: Add missing 'return target' [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192285 (owner: 10Krinkle)
[15:56:51] <grrrit-wm>	 (03Merged) 10jenkins-bot: mw.ViewPageTarget.init: Add missing 'return target' [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192285 (owner: 10Krinkle)
[15:58:49] <grrrit-wm>	 (03CR) 10Catrope: [C: 032] mw.ViewPageTarget: Remove JS animations from transformPage() [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192281 (https://phabricator.wikimedia.org/T89543) (owner: 10Krinkle)
[16:00:41] <grrrit-wm>	 (03Merged) 10jenkins-bot: mw.ViewPageTarget: Remove JS animations from transformPage() [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192281 (https://phabricator.wikimedia.org/T89543) (owner: 10Krinkle)
[16:02:00] <wikibugs>	 10VisualEditor, 10VisualEditor-Performance, 5WMF-deploy-2015-02-25_(1.25wmf19), 5§ VisualEditor Q3 Blockers: ve.init.mw.ViewPageTarget#transformPage triggers recalculate style due to $.fn.animate - https://phabricator.wikimedia.org/T89543#1059070 (10Jdforrester-WMF) 5Open>3Resolved
[16:03:09] <grrrit-wm>	 (03PS13) 10Catrope: mw.ViewPageTarget: Use CSS for toolbar transition [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192047 (https://phabricator.wikimedia.org/T89543) (owner: 10Krinkle)
[16:05:58] <edsanders>	 Krinkle|detached, putting the target toolbar class in attach means that the margins transition from 0 to -1em
[16:06:25] <edsanders>	 which means the toolbar contents moves up and left as it is opening
[16:08:27] <edsanders>	 oh wait, I thought your code was merged
[16:11:21] <grrrit-wm>	 (03PS1) 10Catrope: Followup 14a4a4ae1: *actually* fix back compat for this.timeStamp [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192327 
[16:11:27] <codezee>	 MatmaRex: done, but recheck's needed
[16:11:35] <RoanKattouw>	 edsanders: No he split his commit into two parts, and I merged the part not related to the toolbar
[16:13:32] <grrrit-wm>	 (03CR) 10Zfilipin: [C: 031] "Is there a reason for the upgrade? RuboCop?" [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192310 (owner: 10Amire80)
[16:14:08] <grrrit-wm>	 (03PS2) 10Jforrester: Follow-up 14a4a4ae1: *actually* fix back compat for this.timeStamp [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192327 (owner: 10Catrope)
[16:14:10] <grrrit-wm>	 (03CR) 10Zfilipin: [C: 032] [BrowserTest] Define Rubocop LineLength as 100 [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192311 (owner: 10Amire80)
[16:14:18] <grrrit-wm>	 (03CR) 10Jforrester: [C: 032] Follow-up 14a4a4ae1: *actually* fix back compat for this.timeStamp [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192327 (owner: 10Catrope)
[16:14:52] <grrrit-wm>	 (03CR) 10Amire80: "I couldn't run Rubocop without it." [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192310 (owner: 10Amire80)
[16:16:11] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Follow-up 14a4a4ae1: *actually* fix back compat for this.timeStamp [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192327 (owner: 10Catrope)
[16:16:15] <James_F>	 edsanders: If you could merge https://gerrit.wikimedia.org/r/#/c/191557/ and https://gerrit.wikimedia.org/r/#/c/192031/ Roan and I can do yet-another-pullthrough. :-)
[16:17:08] <wikibugs>	 10MediaWiki-ResourceLoader, 10MediaWiki-Vagrant: qunit test broken without explicitly setting $wgResourceLoaderMaxQueryLength - https://phabricator.wikimedia.org/T90453#1059116 (10JanZerebecki) 3NEW
[16:17:18] <James_F>	 edsanders: (And Roan can do CR on Timo's stuff.)
[16:17:42] <grrrit-wm>	 (03CR) 10Esanders: "Do the tests in the dependent commit cover this?" [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/191557 (https://phabricator.wikimedia.org/T89943) (owner: 10Catrope)
[16:18:09] <RoanKattouw>	 edsanders: No, the other commit's commit message even says so
[16:18:12] <edsanders>	 Oh, I see on the message it says not
[16:18:15] <edsanders>	 why not?
[16:18:25] <James_F>	 edsanders: DocumentSets.
[16:18:34] <RoanKattouw>	 Because it was 2am and I couldn't be bothered, mostly
[16:18:42] <RoanKattouw>	 But it'd be a fair amount of work
[16:18:47] <edsanders>	 hmm
[16:18:54] <grrrit-wm>	 (03CR) 10Bartosz Dziewoński: "Yes, I was also wary, but I implemented it, tested it and (per commit message) this doesn't seem to cause a performance hit. Probably the " [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192241 (https://phabricator.wikimedia.org/T52036) (owner: 10Bartosz Dziewoński)
[16:19:02] <RoanKattouw>	 You have to have two different documents with different stores, and then assert that the store merge operation happens correctly
[16:19:19] <RoanKattouw>	 Which as far as I could tell means rewriting the test runner and the test specification format for that test
[16:19:33] <James_F>	 edsanders: As RoanKattouw pointed out to me when I asked, DocumentSets will make this moot anyway, and it's "so close" (ha!).
[16:19:36] <RoanKattouw>	 All for internalList-related functionality, while I'm supposed to be working on DocumentSet so we can kill InternalList
[16:19:53] <ori>	 morning
[16:19:54] <RoanKattouw>	 James_F's skepticism about how close DocumentSet is is warranted :)
[16:19:57] <RoanKattouw>	 It needs more work
[16:20:02] <RoanKattouw>	 Morning ori 
[16:20:03] <James_F>	 Morning, ori.
[16:20:32] <grrrit-wm>	 (03CR) 10Esanders: [C: 032] newFromDocumentInsertion: Also remap store indexes in internalList data [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/191557 (https://phabricator.wikimedia.org/T89943) (owner: 10Catrope)
[16:21:09] <wikibugs>	 10VisualEditor, 10VisualEditor-CopyPaste, 5WMF-deploy-2015-02-25_(1.25wmf19), 5§ VisualEditor Q3 Blockers: Editor freezes after copypasting a reference and trying to insert a references list - https://phabricator.wikimedia.org/T89943#1059129 (10Jdforrester-WMF) 5Open>3Resolved
[16:21:22] <grrrit-wm>	 (03CR) 10Jforrester: [C: 032] Follow-up 14a4a4ae1: *actually* fix back compat for this.timeStamp [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192327 (owner: 10Catrope)
[16:22:35] <grrrit-wm>	 (03Merged) 10jenkins-bot: newFromDocumentInsertion: Also remap store indexes in internalList data [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/191557 (https://phabricator.wikimedia.org/T89943) (owner: 10Catrope)
[16:22:37] <grrrit-wm>	 (03CR) 10Esanders: "getClientRects is a native API that avoids getComputedStyles, that's why we switched to it. This would undo all that optimisation. Profili" [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192241 (https://phabricator.wikimedia.org/T52036) (owner: 10Bartosz Dziewoński)
[16:22:50] <James_F>	 RoanKattouw: https://gerrit.wikimedia.org/r/#/c/190630/ https://gerrit.wikimedia.org/r/#/c/190022/
[16:23:19] <grrrit-wm>	 (03Merged) 10jenkins-bot: Transaction#newFromDocumentInsertion: Add some more tests [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/191561 (owner: 10Catrope)
[16:23:22] <grrrit-wm>	 (03Merged) 10jenkins-bot: Follow-up 14a4a4ae1: *actually* fix back compat for this.timeStamp [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192327 (owner: 10Catrope)
[16:23:51] <grrrit-wm>	 (03CR) 10Catrope: [C: 04-1] mw.ViewPageTarget: Use CSS for toolbar transition (034 comments) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192047 (https://phabricator.wikimedia.org/T89543) (owner: 10Krinkle)
[16:24:41] <grrrit-wm>	 (03CR) 10Zfilipin: [C: 032] "Strange. Did you confuse "bundle install" and "bundle update"? ;)" [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192310 (owner: 10Amire80)
[16:26:14] <grrrit-wm>	 (03PS3) 10Mvolz: Make zotero.js an object [services/citoid] - 10https://gerrit.wikimedia.org/r/192197 (https://phabricator.wikimedia.org/T78389) 
[16:27:19] <grrrit-wm>	 (03CR) 10Catrope: "I still see something strange at the start of the opening transition, where the bottom border of the toolbar is initially inside the margi" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192047 (https://phabricator.wikimedia.org/T89543) (owner: 10Krinkle)
[16:29:10] <grrrit-wm>	 (03PS6) 10Esanders: [DEPRECATING CHANGE] Rename setPosition to setLabelPosition [oojs/ui] - 10https://gerrit.wikimedia.org/r/190436 
[16:29:57] <grrrit-wm>	 (03CR) 10Jforrester: [C: 032] [DEPRECATING CHANGE] Rename setPosition to setLabelPosition [oojs/ui] - 10https://gerrit.wikimedia.org/r/190436 (owner: 10Esanders)
[16:34:03] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] [BrowserTest] Update Gemfile.lock [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192310 (owner: 10Amire80)
[16:34:05] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] [BrowserTest] Define Rubocop LineLength as 100 [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192311 (owner: 10Amire80)
[16:35:14] <grrrit-wm>	 (03CR) 10Amire80: "recheck" [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192310 (owner: 10Amire80)
[16:38:11] <grrrit-wm>	 (03Merged) 10jenkins-bot: [DEPRECATING CHANGE] Rename setPosition to setLabelPosition [oojs/ui] - 10https://gerrit.wikimedia.org/r/190436 (owner: 10Esanders)
[16:42:49] <wikibugs>	 10VisualEditor, 10VisualEditor-Performance: Load and display the welcome dialog straight after the toolbar, without waiting for the content area - https://phabricator.wikimedia.org/T90454#1059169 (10Jdforrester-WMF) 3NEW
[16:45:46] <grrrit-wm>	 (03PS1) 10Jforrester: Revert "Localisation updates from https://translatewiki.net." [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192334 
[16:46:12] <grrrit-wm>	 (03CR) 10Catrope: [C: 032] Revert "Localisation updates from https://translatewiki.net." [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192334 (owner: 10Jforrester)
[16:46:24] <grrrit-wm>	 (03CR) 10Bartosz Dziewoński: "I am entirely unable to reproduce the performance problems with this approach using Blink-based Opera browser (Mozilla/5.0 (Windows NT 6.1" [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192241 (https://phabricator.wikimedia.org/T52036) (owner: 10Bartosz Dziewoński)
[16:48:43] <grrrit-wm>	 (03CR) 10Catrope: [C: 032] mw.ViewPageTarget: Remove workaround for cascading defaultUserOptions [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/190630 (owner: 10Krinkle)
[16:49:12] <grrrit-wm>	 (03Merged) 10jenkins-bot: Revert "Localisation updates from https://translatewiki.net." [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192334 (owner: 10Jforrester)
[16:51:16] <grrrit-wm>	 (03Merged) 10jenkins-bot: mw.ViewPageTarget: Remove workaround for cascading defaultUserOptions [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/190630 (owner: 10Krinkle)
[16:52:11] <grrrit-wm>	 (03CR) 10Esanders: [C: 032] "I think we worked around most of those issues by lazy-evaluating these highlights on first focus. I can't think of another away around thi" [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192241 (https://phabricator.wikimedia.org/T52036) (owner: 10Bartosz Dziewoński)
[16:54:36] <wikibugs>	 10VisualEditor, 10VisualEditor-ContentEditable: VisualEditor: enwiki template {{columns-list}} looks particularly-broken at high zoom/low width - https://phabricator.wikimedia.org/T52269#1059222 (10Jdforrester-WMF)
[16:54:37] <wikibugs>	 10VisualEditor, 10VisualEditor-ContentEditable, 7Browser-Support-Google-Chrome, 7Upstream, and 2 others: Focus highlights for elements using CSS column-count are too tall in Chrome - https://phabricator.wikimedia.org/T52036#1059219 (10Jdforrester-WMF) 5Open>3Resolved a:5Esanders>3matmarex
[16:54:54] <grrrit-wm>	 (03Merged) 10jenkins-bot: ve.ce.FocusableNode: Ignore children of elements using CSS column-count in Chrome [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192241 (https://phabricator.wikimedia.org/T52036) (owner: 10Bartosz Dziewoński)
[16:55:32] <MatmaRex>	 argh. one of the links in commit message is http D:
[16:56:01] <grrrit-wm>	 (03PS1) 10Catrope: Use prefixed page title for browser title bar [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192338 (https://phabricator.wikimedia.org/T90433) 
[17:00:42] <James_F>	 edsanders: You about to +2 https://gerrit.wikimedia.org/r/#/c/192031/ or should I not wait to do the pull-through?
[17:01:12] <grrrit-wm>	 (03CR) 10Esanders: [C: 032] "Interestingly if you select across a focusable it still gets highlighted. user-select: none doesn't do this when ce=false, so probably a s" [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192031 (https://phabricator.wikimedia.org/T70537) (owner: 10Catrope)
[17:01:20] <James_F>	 Well answered. :-)
[17:02:10] <wikibugs>	 10VisualEditor, 10VisualEditor-EditingTools, 7Browser-Support-Firefox, 5WMF-deploy-2015-02-25_(1.25wmf19), 5§ VisualEditor Q3 Blockers: In production, sometimes page scrolling stops working and in the console getting "Error: offset was inside a handlesOwnC... - https://phabricator.wikimedia.org/T70537#1059252
[17:03:01] <edsanders>	 RoanKattouw, see this in FF: http://jsfiddle.net/8ea7aff2/4/
[17:03:09] <grrrit-wm>	 (03CR) 10Alex Monk: [C: 032] Use prefixed page title for browser title bar [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192338 (https://phabricator.wikimedia.org/T90433) (owner: 10Catrope)
[17:03:10] <edsanders>	 select from Line 1 to 3 in each example
[17:03:19] <RoanKattouw>	 Oh God
[17:03:28] <RoanKattouw>	 I read the docs for user-select and they seemed to suggest this wouldn't happen
[17:03:33] <edsanders>	 I was expecting your commit to be broken for this reason
[17:03:52] <edsanders>	 at the moment this bug works in our favour
[17:04:04] <RoanKattouw>	 Yeah but eventually it won't
[17:04:27] <RoanKattouw>	 We can also use user-select: all
[17:05:04] <RoanKattouw>	 GRRRAH
[17:05:05] <RoanKattouw>	 Also:
[17:05:07] <RoanKattouw>	 Note: One of the differences between browser implementations is inheritance. In Firefox, -moz-user-select is not inherited by absolutely positioned elements, but in Safari and Chrome, -webkit-user-select is inherited by those elements.
[17:05:26] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki, 5WMF-deploy-2015-02-25_(1.25wmf19): Browser title of editing page in VisualEditor is not correct (drops namespace prefix?) - https://phabricator.wikimedia.org/T90433#1059286 (10Jdforrester-WMF) 5Open>3Resolved p:5Triage>3Normal a:3Catrope
[17:05:27] <RoanKattouw>	 That suggests we might need .ve-ce-focusableNode * { -moz-user-select: none/all; }
[17:08:18] <edsanders>	 yum, stars selectors
[17:11:42] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Work around Firefox bug allowing cursor in cE=false [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192031 (https://phabricator.wikimedia.org/T70537) (owner: 10Catrope)
[17:12:21] <grrrit-wm>	 (03Merged) 10jenkins-bot: Use prefixed page title for browser title bar [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192338 (https://phabricator.wikimedia.org/T90433) (owner: 10Catrope)
[17:15:58] <grrrit-wm>	 (03CR) 10Krinkle: mw.ViewPageTarget: Use CSS for toolbar transition (032 comments) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192047 (https://phabricator.wikimedia.org/T89543) (owner: 10Krinkle)
[17:18:02] <grrrit-wm>	 (03PS1) 10Krinkle: mw.ViewPageTarget: Remove obsolete transitions for margin [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192342 
[17:18:07] <Krinkle>	 RoanKattouw: ^
[17:18:41] <grrrit-wm>	 (03CR) 10Esanders: [C: 04-1] "Does this need a star selector?" [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192031 (https://phabricator.wikimedia.org/T70537) (owner: 10Catrope)
[17:19:14] <grrrit-wm>	 (03CR) 10Bartosz Dziewoński: [C: 032] "Oh yes, this was bugging me. Thanks." [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192342 (owner: 10Krinkle)
[17:19:22] <Krinkle>	 Thanks :)
[17:19:33] <Krinkle>	 Thank you for removing it from Vector
[17:19:47] * Krinkle is in a christmas mindset.
[17:19:52] <Krinkle>	 Just watched the Dr Who ep, finally
[17:20:27] <grrrit-wm>	 (03CR) 10Catrope: mw.ViewPageTarget: Use CSS for toolbar transition (031 comment) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192047 (https://phabricator.wikimedia.org/T89543) (owner: 10Krinkle)
[17:21:13] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] mw.ViewPageTarget: Remove obsolete transitions for margin [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192342 (owner: 10Krinkle)
[17:25:50] <Krinkle>	 RoanKattouw: Hm.. There's a tricky bit with the css state managing. Which is why I'm unsure of calling initialize() async
[17:25:54] <edsanders>	 MatmaRex, Krinkle what is wrong with resize transitions?
[17:26:31] <Krinkle>	 RoanKattouw: ve-active is added when surface is ready, but unlike teardown, setup does not have a promise to wait for
[17:27:24] <RoanKattouw>	 How is the teardown promise done?
[17:27:31] <RoanKattouw>	 A timeout? transitionend event?
[17:27:35] <Krinkle>	 RoanKattouw: Yeah
[17:27:44] <Krinkle>	 But there's nowhere for the promise to go to rightnow
[17:27:47] <Krinkle>	 we'd need a change in ve-core
[17:28:04] <Krinkle>	 but actually, we probably shouldn't be blocking on that
[17:28:16] <MatmaRex>	 edsanders: not much, but they're annoying to support, and their usefulness was disputed on https://phabricator.wikimedia.org/T85614
[17:28:26] <RoanKattouw>	 Krinkle: Wait, hold on
[17:28:29] <Krinkle>	 the toolbar is added and transitions but it's done as far as the state is concerned
[17:28:35] <RoanKattouw>	 How does VE core have a teardown promise for the toolbar?
[17:28:40] <RoanKattouw>	 The transition stuff isn't in VE core
[17:28:45] <Krinkle>	 RoanKattouw: tearDownToolbar returns promise
[17:29:09] <RoanKattouw>	 Yes, and that's in VE-MW
[17:29:20] <RoanKattouw>	 The promise it returns comes from $.fn.slideUp()
[17:29:22] <Krinkle>	 Ah, right
[17:29:23] <Krinkle>	 Yeah
[17:29:27] <Krinkle>	 Well, yeah, in the current codee
[17:29:37] <Krinkle>	 but we wait for tearDownToolbar in tearDownSurface
[17:29:58] <Krinkle>	 which cancel() waits for
[17:30:00] <Krinkle>	 OK, we can do this
[17:30:05] <Krinkle>	 using activatingDeferred
[17:30:23] <Krinkle>	 RoanKattouw: So the question is, is it okay for the transition-in of toolbar to not start until surface is ready
[17:30:26] <Krinkle>	 or should it start asap
[17:30:36] <wm-ve-needcheck>	 Corruption alert: visualeditor-needcheck on frwiki: https://fr.wikipedia.org/?diff=112117815
[17:30:36] <wm-ve-needcheck>	 Corruption alert: visualeditor-needcheck on frwiki: https://fr.wikipedia.org/?diff=112121781
[17:30:36] <wm-ve-needcheck>	 Corruption alert: visualeditor-needcheck on zhwiki: https://zh.wikipedia.org/?diff=34406572
[17:30:36] <wm-ve-needcheck>	 Corruption alert: visualeditor-needcheck on zhwiki: https://zh.wikipedia.org/?diff=34406580
[17:30:36] <wm-ve-needcheck>	 Corruption alert: visualeditor-needcheck on zhwiki: https://zh.wikipedia.org/?diff=34407304
[17:30:37] <wm-ve-needcheck>	 Corruption alert: visualeditor-needcheck on zhwiki: https://zh.wikipedia.org/?diff=34416301
[17:30:37] <wm-ve-needcheck>	 Corruption alert: visualeditor-needcheck on zhwiki: https://zh.wikipedia.org/?diff=34416493
[17:30:38] <wm-ve-needcheck>	 Corruption alert: visualeditor-needcheck on ruwiki: https://ru.wikipedia.org/?diff=68774786
[17:30:38] <wm-ve-needcheck>	 Corruption alert: visualeditor-needcheck on ruwiki: https://ru.wikipedia.org/?diff=68781623
[17:30:39] <wm-ve-needcheck>	 Corruption alert: visualeditor-needcheck on ruwiki: https://ru.wikipedia.org/?diff=68795099
[17:30:39] <wm-ve-needcheck>	 Corruption alert: visualeditor-needcheck on ptwiki: https://pt.wikipedia.org/?diff=41402974
[17:30:40] <wm-ve-needcheck>	 Corruption alert: visualeditor-needcheck on dewiki: https://de.wikipedia.org/?diff=139120589
[17:30:40] <wm-ve-needcheck>	 Corruption alert: visualeditor-needcheck on itwiki: https://it.wikipedia.org/?diff=70994294
[17:30:43] <RoanKattouw>	 Krinkle: Well, Ed's change moves the timing of when the toolbar is attached
[17:30:49] <Krinkle>	 Yeah
[17:30:52] <RoanKattouw>	 To well before the real surface is ready
[17:30:58] <Krinkle>	 RoanKattouw: I'd like to get that in without the transition change
[17:30:59] <RoanKattouw>	 You guys should really rebase your changes onto each other
[17:31:06] <RoanKattouw>	 Or one onto the other, or something
[17:31:08] <RoanKattouw>	 Yeah me too
[17:31:24] <RoanKattouw>	 It's funny, I asked Ed to split his change, and instead what happened was you split your change :)
[17:31:29] <Krinkle>	 I'm not rebasing it onto mine though, I want the dummy toolbar to land first
[17:31:39] <RoanKattouw>	 OK maybe you can rebase your change onto his then?
[17:31:48] <Krinkle>	 Yeah, but stripping transition change from his change
[17:32:04] <RoanKattouw>	 Sure
[17:32:40] <Krinkle>	 I removed like 25 screenshots from my VE folder this weekend. Tech debt I haven't bother even writing a task for. Jut things I noticed and captured. All fixed now
[17:32:43] <Krinkle>	 Weeee
[17:34:04] <grrrit-wm>	 (03CR) 10Krinkle: [C: 032] "esprima is getting really annoying. Re-submit" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192342 (owner: 10Krinkle)
[17:36:00] <grrrit-wm>	 (03Merged) 10jenkins-bot: mw.ViewPageTarget: Remove obsolete transitions for margin [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192342 (owner: 10Krinkle)
[17:38:38] <James_F>	 Krinkle: Is esprima one of the tickets in Phabricator?
[17:38:46] <Krinkle>	 Nope
[17:38:50] <James_F>	 Should it be?
[17:39:17] <Krinkle>	 https://github.com/jscs-dev/node-jscs/issues/787
[17:39:18] <Krinkle>	 https://github.com/npm/npm/issues/7130#issuecomment-75508043
[17:39:22] <Krinkle>	 I guess, yeah
[17:39:28] <Krinkle>	 I thought it was gone with update to npm 2.4
[17:39:30] <Krinkle>	 it was gone for a while
[17:39:32] <Krinkle>	 but it's back...
[17:39:38] <Krinkle>	 has to do with concurrent writers to the central cache
[17:39:44] <Krinkle>	 not with workspace cleaning, that was fixed
[17:39:52] <James_F>	 Helpful.
[17:41:22] <RoanKattouw>	 edsanders: Ugh, using user-select: all doesn't fix the bug
[17:41:36] <RoanKattouw>	 So I'll just stick with none
[17:41:57] <RoanKattouw>	 And we'll just not tell Firefox about the bug where you can select us:none stuff in cE
[17:42:13] <RoanKattouw>	 Knowing how little they work on cE support, if we don't tell them they'll never fix it
[17:42:55] <grrrit-wm>	 (03PS1) 10Kmenger: IconElement: Add description for config options [oojs/ui] - 10https://gerrit.wikimedia.org/r/192348 
[17:43:42] <James_F>	 RoanKattouw: Optimism bias. :_)
[17:44:10] <grrrit-wm>	 (03PS2) 10Catrope: Work around Firefox bug allowing cursor in cE=false [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192031 (https://phabricator.wikimedia.org/T70537) 
[17:48:30] <grrrit-wm>	 (03PS5) 10Catrope: Test for class attribute duplication bug [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/191037 (https://phabricator.wikimedia.org/T88148) (owner: 10Esanders)
[17:48:45] <grrrit-wm>	 (03PS5) 10Krinkle: Show the toolbar as soon as the target has been activated [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192072 (https://phabricator.wikimedia.org/T76523) (owner: 10Esanders)
[17:48:47] <grrrit-wm>	 (03CR) 10Jforrester: [C: 032] IconElement: Add description for config options [oojs/ui] - 10https://gerrit.wikimedia.org/r/192348 (owner: 10Kmenger)
[17:50:16] <grrrit-wm>	 (03Merged) 10jenkins-bot: IconElement: Add description for config options [oojs/ui] - 10https://gerrit.wikimedia.org/r/192348 (owner: 10Kmenger)
[17:50:22] <grrrit-wm>	 (03PS6) 10Krinkle: Show the toolbar as soon as the target has been activated [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192072 (https://phabricator.wikimedia.org/T76523) (owner: 10Esanders)
[17:50:27] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Test for class attribute duplication bug [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/191037 (https://phabricator.wikimedia.org/T88148) (owner: 10Esanders)
[17:57:23] <wikibugs>	 10VisualEditor, 10Citoid: Citoid / citoid integration with VE is emitting data-mw for a template arg that is not compliant with the Parsoid DOMspec - https://phabricator.wikimedia.org/T90463#1059517 (10ssastry) 3NEW a:3Mvolz
[17:58:12] <grrrit-wm>	 (03CR) 10Catrope: [C: 031] Simplify slug behaviour [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/190022 (https://phabricator.wikimedia.org/T89192) (owner: 10Esanders)
[17:58:55] <wikibugs>	 10VisualEditor, 10Citoid: Citoid / citoid integration with VE is emitting data-mw for a template arg that is not compliant with the Parsoid DOMspec - https://phabricator.wikimedia.org/T90463#1059536 (10ssastry)
[17:58:58] <Eloquence>	 are there known major issues with firefox right now?
[17:59:43] <wikibugs>	 10VisualEditor, 10Citoid: Citoid / citoid integration with VE is emitting data-mw for a template arg that is not compliant with the Parsoid DOMspec - https://phabricator.wikimedia.org/T90463#1059517 (10ssastry)
[18:00:09] <Eloquence>	 (nm, seems just one user's issue)
[18:01:48] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Work around Firefox bug allowing cursor in cE=false [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192031 (https://phabricator.wikimedia.org/T70537) (owner: 10Catrope)
[18:03:23] <wikibugs>	 10OOjs-UI, 10VisualEditor, 7Design: Button labels are invisible while the button is clicked - https://phabricator.wikimedia.org/T90465#1059546 (10Eloquence) 3NEW
[18:10:53] <wikibugs>	 10OOjs-UI, 10VisualEditor, 7Design: Button labels are invisible while the button is clicked - https://phabricator.wikimedia.org/T90465#1059593 (10matmarex) This is already fixed (try it on labs), but not yet deployed.
[18:11:01] <wikibugs>	 10OOjs-UI: Primary buttons change text colour when pressed - https://phabricator.wikimedia.org/T88948#1059595 (10matmarex)
[18:11:02] <wikibugs>	 10OOjs-UI, 10VisualEditor, 7Design: Button labels are invisible while the button is clicked - https://phabricator.wikimedia.org/T90465#1059594 (10matmarex)
[18:11:50] <wikibugs>	 10OOjs-UI, 10VisualEditor, 7Design: Button labels are invisible while the button is clicked - https://phabricator.wikimedia.org/T90465#1059598 (10Eloquence) Thanks Bartosz!
[18:13:43] <wikibugs>	 10VisualEditor, 10VisualEditor-EditingTools, 10VisualEditor-Performance: VisualEditor: Editing a transclusion with many templates hangs the browser - https://phabricator.wikimedia.org/T71799#1059604 (10Etonkovidova) p:5Triage>3Normal
[18:14:23] <James_F>	 Eloquence: No major issues that I know of, no.
[18:15:48] <wikibugs>	 10VisualEditor, 10VisualEditor-DataModel, 10VisualEditor-Performance: Re-consider using OO.getHash in convertModelFromDom - https://phabricator.wikimedia.org/T90413#1059609 (10Jdforrester-WMF) p:5Triage>3Normal
[18:16:13] <wikibugs>	 10VisualEditor, 10VisualEditor-ContentEditable, 10VisualEditor-Performance, 5§ VisualEditor Q3 Blockers: Incrementally build CE tree/DOM - https://phabricator.wikimedia.org/T90376#1059616 (10Jdforrester-WMF) p:5Triage>3High
[18:16:25] <wikibugs>	 10VisualEditor, 10VisualEditor-Performance, 5§ VisualEditor Q3 Blockers: Retrieve page HTML directly from RESTbase - https://phabricator.wikimedia.org/T90374#1059618 (10Jdforrester-WMF) p:5Triage>3High
[18:16:41] <wikibugs>	 10VisualEditor, 10VisualEditor-Performance: Benchmark hot cache load times - https://phabricator.wikimedia.org/T90373#1059620 (10Jdforrester-WMF) p:5Triage>3Normal
[18:17:35] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki, 10VisualEditor-Performance, 5§ VisualEditor Q3 Blockers: Send Parsoid API request immediately when user clicks edit - https://phabricator.wikimedia.org/T90372#1059632 (10Jdforrester-WMF) p:5Triage>3Unbreak!
[18:18:57] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki-Links: insert reference confusing enough to produce broken link - https://phabricator.wikimedia.org/T90379#1059635 (10Jdforrester-WMF) 5Open>3declined a:3Jdforrester-WMF
[18:19:38] <Krinkle>	 James_F: Are there browsers on here http://caniuse.com/#search=requestAnimationFrame in red or green+yellow that we need to support, ever?
[18:19:48] <Krinkle>	 (in ve-mw)
[18:20:44] <James_F>	 Krinkle: We're meant to eventually work in Android 3+; if we need to bump that to 4.4+ that's a serious reduction in device coverage. Also, IE9 is meant to be eventually supported but meh.
[18:21:02] <Krinkle>	 OK. I'll do a fallback. no worries
[18:21:12] <James_F>	 Krinkle: Cool.
[18:21:15] <Krinkle>	 It's one line of code.
[18:21:19] <Krinkle>	 But just checking
[18:21:39] * James_F nods.
[18:24:55] <wikibugs>	 10VisualEditor: tiny windows with scrollbars should be increased - https://phabricator.wikimedia.org/T90377#1059677 (10Jdforrester-WMF) In your screenshots, the diff window is already 900px wide; standard width, as used for the other windows, is 700px wide. I don't think going full-screen is a good outcome for u...
[18:25:35] <grrrit-wm>	 (03PS1) 10Kmenger: IconElement: Add description of methods [oojs/ui] - 10https://gerrit.wikimedia.org/r/192361 
[18:27:47] <Krinkle>	 Oh bollocks. WE're loading jQuery UI also before the surface is considerd ready
[18:27:55] <Krinkle>	 because we depend on mw.feedback
[18:27:56] <Krinkle>	 :(
[18:28:08] <Krinkle>	 There's a couple repaints and stuff that it does onload
[18:28:08] <James_F>	 Krinkle: Yeah, there's a ticket to convert it.
[18:28:14] <Krinkle>	 Can we lazy load it?
[18:28:21] <James_F>	 Maybe mooeypoo should convert it sooner rather than later?
[18:28:24] <Krinkle>	 It's in its own request either way
[18:28:27] <RoanKattouw>	 Yeah I noticed that the other day
[18:28:32] <RoanKattouw>	 JUI does some stupid things on load
[18:28:36] <James_F>	 (*coughs*)
[18:28:43] <James_F>	 RoanKattouw: Like, exist?
[18:28:46] <RoanKattouw>	 Can we load mw.feedback on demand or something?
[18:28:48] <RoanKattouw>	 James_F: That too
[18:28:51] <Krinkle>	 jQuery core does that on-demand now (like Sizzle). jQuery UI, too, but we're using an old version
[18:28:56] <grrrit-wm>	 (03PS10) 10Mooeypoo: [WIP] Refactor Citoid extension as an inspector [extensions/Citoid] - 10https://gerrit.wikimedia.org/r/190973 (https://phabricator.wikimedia.org/T88152) 
[18:30:18] <Krinkle>	 Ha, so if I use requestAnimationFrame instead of setTimeout to set the height to scrollHeight, there is no transition, and the toolbar initially renders without the negative margin from mwViewPAgeTarget.vector.js because... because ResourceLoader sets the styles *after* the js starts executing.
[18:30:21] <Krinkle>	 Debug mode disabled
[18:30:33] <James_F>	 Krinkle: Fun.
[18:30:33] <Krinkle>	 vector.css*
[18:31:08] <Krinkle>	 RoanKattouw: I put a debugger in the top of mw.vpt.js and the <style> for ve-mw was not attached yet
[18:31:19] <Krinkle>	 That's not good :)
[18:34:52] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] IconElement: Add description of methods [oojs/ui] - 10https://gerrit.wikimedia.org/r/192361 (owner: 10Kmenger)
[18:35:39] <James_F>	 RoanKattouw: https://gerrit.wikimedia.org/r/192031 is still V-1'ed.
[18:35:52] <James_F>	 s/still/stupidly/
[18:36:01] <grrrit-wm>	 (03CR) 10Jforrester: "recheck" [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192031 (https://phabricator.wikimedia.org/T70537) (owner: 10Catrope)
[18:37:12] <wikibugs>	 10VisualEditor, 10MediaWiki-General-or-Unknown, 10UI-Standardization: mediawiki.feedback should use OOjs UI dialog - https://phabricator.wikimedia.org/T89878#1059708 (10Jdforrester-WMF)
[18:37:53] <wikibugs>	 10VisualEditor, 10MediaWiki-General-or-Unknown, 10UI-Standardization, 5§ VisualEditor Q3 Blockers: mediawiki.feedback should use OOjs UI dialog - https://phabricator.wikimedia.org/T89878#1047843 (10Jdforrester-WMF)
[18:38:50] <wikibugs>	 10VisualEditor, 10Flow, 10§Collaboration-Team-Sprint-Q-2015-02-25: Q2. Define and estimate work for Minimum Viable Product of VisualEditor in Flow (spike) (Catalan, Translatewiki) - https://phabricator.wikimedia.org/T88182#1059713 (10DannyH) a:3Mattflaschen
[18:39:12] <James_F>	 mooeypoo: https://gerrit.wikimedia.org/r/#/c/32700/
[18:39:24] <James_F>	 mooeypoo: But let me rebase it for you first. ;-)
[18:44:20] <grrrit-wm>	 (03CR) 10Catrope: "recheck" [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192031 (https://phabricator.wikimedia.org/T70537) (owner: 10Catrope)
[18:44:36] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki, 10VisualEditor-Performance, 10Parsoid, and 2 others: Log Parsoid server-side save performance - https://phabricator.wikimedia.org/T66171#1059746 (10ssastry)
[18:44:41] <RoanKattouw>	 Krinkle: That's bad, and I'm guessing that's why I saw big style recomputations at the end
[18:48:59] <grrrit-wm>	 (03PS1) 10Kmenger: IndicatorElement: Add description for configs and static properties [oojs/ui] - 10https://gerrit.wikimedia.org/r/192369 
[18:50:40] <grrrit-wm>	 (03CR) 10Kmenger: "recheck" [oojs/ui] - 10https://gerrit.wikimedia.org/r/192361 (owner: 10Kmenger)
[18:53:01] <Krinkle>	 RoanKattouw: Hmpf, on a short page I can see both at activate and deactivate a brief moment where mw-content and surface are both visible underneath each other (long page)
[18:53:08] <Krinkle>	 unrelated, but also causing side effects
[18:53:26] <Krinkle>	 there is native replaceChild(), but we can't use that because of the target-nesting changes
[18:53:50] <RoanKattouw>	 Hmm
[18:53:52] <Krinkle>	 We should be able to hide the other via css though
[18:53:57] <RoanKattouw>	 Yeah
[18:54:01] <wikibugs>	 10VisualEditor, 10VisualEditor-ContentEditable: Annotated text: typing,deleting and then again typing it,  does not remove the annotation. - https://phabricator.wikimedia.org/T90155#1059794 (10Swainr) Issue is not resolved.Found in test2 & beta- chrome and FF.
[18:54:06] <RoanKattouw>	 That duplication is going to hurt on e.g. Obama
[18:54:09] <Krinkle>	 But that might break ce computation thinking its visibnle
[18:54:14] <Krinkle>	 It's not new though
[18:54:20] <Krinkle>	 it's already there now in master and everywhere
[18:56:43] <wikibugs>	 10VisualEditor, 10VisualEditor-Performance, 5§ VisualEditor Q3 Blockers: Retrieve page HTML directly from RESTbase - https://phabricator.wikimedia.org/T90374#1059805 (10GWicke) @krenair, see https://rest.wikimedia.org/en.wikipedia.org/v1/?doc. This went alpha-live on Friday & needs a little bit more testing,...
[18:59:25] <grrrit-wm>	 (03PS14) 10Krinkle: mw.ViewPageTarget: Use CSS for toolbar transition [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192047 (https://phabricator.wikimedia.org/T89543) 
[19:01:51] <Krinkle>	 RoanKattouw: Hm.. I removed a lot of jank but it's still a bit strange. I can't quite pinpoint what it is
[19:02:48] <Krinkle>	 I guess it's a result of transitioning a layout property (height). But there isn't really another way short of re-arranging how target tree is composed (again)
[19:02:59] <grrrit-wm>	 (03CR) 10Esanders: Unify data transfer handlers for paste and drop (035 comments) [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/188029 (owner: 10Esanders)
[19:03:37] <grrrit-wm>	 (03PS10) 10Esanders: Unify data transfer handlers for paste and drop [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/188029 
[19:03:41] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Unify data transfer handlers for paste and drop [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/188029 (owner: 10Esanders)
[19:04:06] <grrrit-wm>	 (03PS11) 10Esanders: Unify data transfer handlers for paste and drop [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/188029 
[19:07:44] <RoanKattouw>	 Krinkle: You didn't address my "isn't this a no-op" comment
[19:07:59] <wikibugs>	 10VisualEditor, 10VisualEditor-ContentEditable: Annotated text: typing,deleting and then again typing it,  does not remove the annotation. - https://phabricator.wikimedia.org/T90155#1059861 (10Etonkovidova) Re-opening the issue - the behavior in production is different: once **all of the formatted text **is de...
[19:08:01] <RoanKattouw>	 For this.toolbar.$element.css( 'height', this.toolbar.$bar.outerHeight() );  in teardownToolbar
[19:08:43] <RoanKattouw>	 I don't understand why that's there
[19:09:00] <Krinkle>	 RoanKattouw: You can't transition to 0 without a height being set
[19:09:05] <wikibugs>	 10VisualEditor, 10VisualEditor-ContentEditable: [Regression-pre19]: Annotated text: typing,deleting and then again typing it,  does not remove the annotation. - https://phabricator.wikimedia.org/T90155#1059869 (10Etonkovidova) p:5Triage>3Normal
[19:09:10] <grrrit-wm>	 (03CR) 10Amire80: [C: 032] [BrowserTest] Update Gemfile.lock [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192310 (owner: 10Amire80)
[19:09:12] <wikibugs>	 10VisualEditor, 10VisualEditor-ContentEditable: [Regression-pre19]: Annotated text: typing,deleting and then again typing it,  does not remove the annotation. - https://phabricator.wikimedia.org/T90155#1053504 (10Etonkovidova) 5Resolved>3Open
[19:09:21] <RoanKattouw>	 Krinkle: Right, so it's needed for the transition, makes sense
[19:09:23] <Krinkle>	 RoanKattouw: And we remove the one we set in setup so that you can resize the window and allow toolbar to take up more height
[19:09:48] <Krinkle>	 I'll change that one to scrollHeigiht for consistency though
[19:10:05] <Krinkle>	 RoanKattouw: So, the transition can be on either toolbar or toolbar-bar
[19:10:39] <Krinkle>	 if we do it on toolbar, the grey border will appear last because it's naturally cropped by the parent
[19:10:54] <Krinkle>	 however on toolbar the transition is smoother without stutter on start/end
[19:10:59] <RoanKattouw>	 OK
[19:11:11] <RoanKattouw>	 So should we introduce another wrapper for the grey border then?
[19:11:20] <Krinkle>	 if we do it on toolbar-bar there is a stutter at start/end when it gets detached as height:0 still has the border there
[19:11:44] <Krinkle>	 We already have that grey wrapper
[19:11:51] <Krinkle>	 effectively anyway
[19:11:57] <RoanKattouw>	 Oh wow it's super smooth now
[19:12:02] <RoanKattouw>	 But I see what you mean about the border
[19:12:13] <Krinkle>	 There used to be a sort of scan animation
[19:12:17] <Krinkle>	 the border line being sticky
[19:12:32] <Krinkle>	 We might be able to bring it back by deducting 1px of that think border and move it to the wrapper
[19:14:29] <grrrit-wm>	 (03PS15) 10Krinkle: mw.ViewPageTarget: Use CSS for toolbar transition [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192047 (https://phabricator.wikimedia.org/T89543) 
[19:14:47] <Krinkle>	 RoanKattouw: ^ even smoother now. scrollHeight caused a wrong effect, that is 5px short due to the border. so there was an even bigger stutter
[19:14:58] <Krinkle>	 it both appeared last and all at once
[19:15:06] <Krinkle>	 moving the 1px now to see how that looks
[19:15:36] <RoanKattouw>	 Krinkle: How does that even work? Didn't you set height: 0?
[19:15:52] <Krinkle>	 RoanKattouw: On the parent yet
[19:16:04] <Krinkle>	 but the children are still fully rendered in overflow
[19:16:07] <Krinkle>	 yes*
[19:16:11] <RoanKattouw>	 Oh of course
[19:16:11] <grrrit-wm>	 (03Merged) 10jenkins-bot: [BrowserTest] Update Gemfile.lock [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192310 (owner: 10Amire80)
[19:16:13] <grrrit-wm>	 (03Merged) 10jenkins-bot: [BrowserTest] Define Rubocop LineLength as 100 [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192311 (owner: 10Amire80)
[19:16:52] <Krinkle>	 RoanKattouw: OK. So check out that latest patch and in the console, make oo-ui-toolbar-bar 3px instead of 1px and add border-bottom: 1px solid rgba(0,0,0,0.15) to oo-ui-toolbar
[19:17:00] <Krinkle>	 then toggle back to read and to edit again
[19:17:12] <Krinkle>	 we'll need to do that in oo-ui though and propagate
[19:17:13] <Krinkle>	 but I think its worth it
[19:17:29] <Krinkle>	 instead of 4px*
[19:21:44] <grrrit-wm>	 (03PS16) 10Krinkle: mw.ViewPageTarget: Use CSS for toolbar transition [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192047 (https://phabricator.wikimedia.org/T89543) 
[19:21:45] <Krinkle>	 ^ baked in for now
[19:22:12] <wikibugs>	 10VisualEditor: TypeError "Cannot read property 'length' of undefined" appears while trying to select the same reference from Use Existing Reference dialog - https://phabricator.wikimedia.org/T90476#1059914 (10Ryasmeen) 3NEW
[19:22:26] <Krinkle>	 It goes all wrong when you cancel loading by clicking on Read too soon though
[19:22:29] <Krinkle>	 ugh..
[19:23:05] <RoanKattouw>	 wow
[19:23:07] <RoanKattouw>	 Yeah that's very broken
[19:23:14] <RoanKattouw>	 Probably a bug in Ed's load-the-toolbar-early change
[19:23:30] <Krinkle>	 it's leaving the toolbar outside ve-init-target
[19:23:49] <RoanKattouw>	 It's also not unhiding the page content
[19:23:54] <Krinkle>	 and leaving the surface in
[19:24:08] <Krinkle>	 I hate thit
[19:24:17] <Krinkle>	 Too much style logic in the javascript
[19:24:31] <Krinkle>	 but css can't cope with just class switches
[19:24:57] <Krinkle>	 Wait, we have a toolbar inside the toolbar
[19:25:05] <Krinkle>	 the question mark and editnotice
[19:25:14] <Krinkle>	 Those two icons are an entire toolbar?
[19:25:30] <RoanKattouw>	 Three icons, rihgt?
[19:25:33] <Krinkle>	 I guess that's why the css uses >
[19:25:34] <Krinkle>	 Yeah
[19:25:55] <RoanKattouw>	 The hamburger menu is there too
[19:26:00] <RoanKattouw>	 And contains a bunch of tools
[19:26:03] <grrrit-wm>	 (03PS17) 10Krinkle: mw.ViewPageTarget: Use CSS for toolbar transition [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192047 (https://phabricator.wikimedia.org/T89543) 
[19:26:26] <grrrit-wm>	 (03CR) 10Krinkle: "Fixes border showing up on ve-init-mw-viewPageTarget-toolbar-utilities." [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192047 (https://phabricator.wikimedia.org/T89543) (owner: 10Krinkle)
[19:26:42] <Krinkle>	 Yeah, but why can't those be in the main toolbar
[19:26:53] <Krinkle>	 registered too late?
[19:27:02] <RoanKattouw>	 No, I think it's because they need to float right? Not sure
[19:27:17] <Krinkle>	 Well, having two groups of icons would be a possible feature to add to toolbar
[19:27:31] <wikibugs>	 10VisualEditor, 10VisualEditor-EditingTools, 7Design, 7Epic, 5§ VisualEditor Q3 Blockers: Cite: Auto-filled references based on destination URL / ISBN / etc. - https://phabricator.wikimedia.org/T62768#1059944 (10KHammerstein)
[19:27:38] <Krinkle>	 Unless we separate the layout from the logic and consider a toolbar a single row of icons
[19:27:48] <Krinkle>	 But that's not the case either right now
[19:27:49] <Krinkle>	 Oh well
[19:27:50] <Krinkle>	 fixed
[19:28:16] <grrrit-wm>	 (03PS7) 10Esanders: Move special character inserter to toolbar dialog manager [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/188280 
[19:30:08] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] mw.ViewPageTarget: Use CSS for toolbar transition [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192047 (https://phabricator.wikimedia.org/T89543) (owner: 10Krinkle)
[19:31:17] <Krinkle>	 wait what
[19:31:18] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Move special character inserter to toolbar dialog manager [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/188280 (owner: 10Esanders)
[19:31:23] <Krinkle>	 that's a phantomjs job
[19:31:27] <Krinkle>	 why is that running in labs
[19:36:20] <wikibugs>	 10VisualEditor, 10VisualEditor-EditingTools, 7Design, 7Epic, 5§ VisualEditor Q3 Blockers: Cite: Auto-filled references based on destination URL / ISBN / etc. - https://phabricator.wikimedia.org/T62768#1059961 (10KHammerstein)
[19:42:01] <grrrit-wm>	 (03CR) 10Krinkle: "recheck" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192047 (https://phabricator.wikimedia.org/T89543) (owner: 10Krinkle)
[19:42:19] <grrrit-wm>	 (03PS1) 10Catrope: Load mw.feedback on demand [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192378 
[19:42:32] <RoanKattouw>	 ori: ---^^ No longer loading jQuery UI during VE startup
[19:42:59] <ori>	 so that was an actual thing
[19:43:02] <ori>	 good lord
[19:43:03] <ori>	 :P
[19:43:16] <wikibugs>	 10VisualEditor, 10VisualEditor-EditingTools, 7Design, 7Epic, 5§ VisualEditor Q3 Blockers: Cite: Auto-filled references based on destination URL / ISBN / etc. - https://phabricator.wikimedia.org/T62768#1059985 (10KHammerstein)
[19:47:35] <Krinkle>	 RoanKattouw: What happens if .launch() is called multiples times?
[19:47:45] <wikibugs>	 10VisualEditor: [Regression wmf 18]  Error: Offset could not be translated to a DOM element and offset appears while deleting the text inside a bullet/numbered item and then cursor starts behaving weirdly while typing text - https://phabricator.wikimedia.org/T90478#1060020 (10Ryasmeen)
[19:47:49] <Krinkle>	 as being async, I can open ? aagain and re-click it
[19:47:56] <Krinkle>	 queueing two launch calls :P
[19:48:13] <RoanKattouw>	 Good question
[19:48:33] <grrrit-wm>	 (03PS1) 10Kmenger: DraggableElement: Mark private methods and add description to events [oojs/ui] - 10https://gerrit.wikimedia.org/r/192379 
[19:48:38] <wikibugs>	 10VisualEditor: [Regression wmf 18]  Error: Offset could not be translated to a DOM element and offset appears while deleting the text inside a bullet/numbered item and then cursor starts behaving weirdly while typing text - https://phabricator.wikimedia.org/T90478#1060014 (10Ryasmeen) Happens in both Chrome and...
[19:49:17] <wikibugs>	 10VisualEditor: [Regression wmf 18]  "Error: Offset could not be translated to a DOM element and offset" appears while deleting the text inside a bullet/numbered item and then cursor starts behaving weirdly while typing text  - https://phabricator.wikimedia.org/T90478#1060031 (10Ryasmeen)
[19:49:36] <RoanKattouw>	 Krinkle: It appears to be idempotent
[19:49:42] <RoanKattouw>	 From running it multiple times in the console
[19:49:44] <RoanKattouw>	 Let me look at the code
[19:51:53] <Krinkle>	 programming front-end within php is *so* easy
[19:52:05] <Krinkle>	 There's nothing to worry about
[19:53:00] <Krinkle>	 I'd like to see what VE would look like without javascirpt. Using ASP. You know, every click a refresh, but using flash/ajax to replace segments of the page. And even a local reverse proxy to not actually go over the wire.
[19:53:28] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki-References, 10Citoid, 5WMF-deploy-2015-02-25_(1.25wmf19): Don't let the Citoid tool be added to the "catchall" toolbar group, because it makes for a confusing/broken experience - https://phabricator.wikimedia.org/T89555#1060057 (10Etonkovidova) Verified the fix in b...
[19:53:55] <RoanKattouw>	 Krinkle: Bah, it's not idempotent
[19:54:04] <Krinkle>	 brb
[19:54:05] <RoanKattouw>	 It blanks the input fields in the dialog and moves the focus
[19:54:13] <RoanKattouw>	 It's just that jQuery UI Dialog .open() is idempotent
[19:56:04] <RoanKattouw>	 Krinkle|detached: But I'm going to leave that unaddressed if you don't mind, because clicking the feedback tool twice within the loading time is quite a feat
[19:56:40] <RoanKattouw>	 And the two launch operations will take place right after each other so the user won't really notice
[20:03:30] <James_F>	 RoanKattouw: https://gerrit.wikimedia.org/r/188029
[20:05:34] <wikibugs>	 10VisualEditor, 10Citoid: Citoid / citoid integration with VE is emitting data-mw for a template arg that is not compliant with the Parsoid DOMspec - https://phabricator.wikimedia.org/T90463#1060150 (10ssastry)
[20:06:58] <grrrit-wm>	 (03CR) 10Jforrester: [C: 031] Work around Firefox bug allowing cursor in cE=false [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192031 (https://phabricator.wikimedia.org/T70537) (owner: 10Catrope)
[20:07:40] <James_F>	 TrevorParscal: https://gerrit.wikimedia.org/r/#/c/192031/
[20:08:11] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki-References, 10Citoid: Citoid: Reopening the Web Reference in media shows the details of the first entry(valid/ invalid-URL /DOI; whatever was entered first in the "Autofill from URL dialog upon intiliazation) - https://phabricator.wikimedia.org/T90480#1060162 (10Swainr)
[20:08:46] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki-References, 10Citoid: Citoid: Reopening the Web Reference in media shows the details of the first entry(valid/ invalid-URL /DOI; whatever was entered first in the "Autofill from URL" dialog upon intiliazation) - https://phabricator.wikimedia.org/T90480#1060166 (10Swain...
[20:08:56] <MatmaRex>	 James_F: is there a reason why https://phabricator.wikimedia.org/T73085 is not "done" or resolved? waiting for QA?
[20:09:04] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki-References, 10Citoid: Citoid: Reopening the Web Reference in media shows the details of the first entry(valid/ invalid-URL /DOI; whatever was entered first in the "Autofill from URL" dialog upon initialization) - https://phabricator.wikimedia.org/T90480#1060156 (10Swai...
[20:09:08] <James_F>	 MatmaRex: Yes.
[20:09:08] <MatmaRex>	 our bug management practices change so often, i'm losing track
[20:09:24] <James_F>	 MatmaRex: I wanted Rummana and Elena to look at it.
[20:09:29] <James_F>	 MatmaRex: Will speak to them now,
[20:09:44] <MatmaRex>	 alright, so this is a special-case? okay
[20:09:50] <James_F>	 MatmaRex: Normally I'd have marked it as resolved, yes.
[20:09:55] <James_F>	 MatmaRex: Good to remind me. :-)
[20:10:57] <MatmaRex>	 James_F: also. it looks like kirstenMA has done / is doing most of https://phabricator.wikimedia.org/T88628 and https://phabricator.wikimedia.org/T88629 , which are assigned to me. i'd reassign, but she's not on phab?
[20:10:59] <MatmaRex>	 kirstenMA: ^
[20:11:36] <kirstenMA>	 I've never used phabricator. Do I need to sign up somewhere?
[20:12:36] <James_F>	 MatmaRex: Sure.
[20:12:39] <James_F>	 kirstenMA: Yes.
[20:12:45] <MatmaRex>	 https://phabricator.wikimedia.org/auth/start/
[20:12:51] <James_F>	 https://phabricator.wikimedia.org/auth/start/?next=%2F
[20:12:53] <James_F>	 Ha.
[20:13:03] <James_F>	 And click "Login|MediaWiki" at the bottom.
[20:13:11] <grrrit-wm>	 (03CR) 10Catrope: [C: 032] Unify data transfer handlers for paste and drop [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/188029 (owner: 10Esanders)
[20:13:12] <MatmaRex>	 you can connect both your mediawiki.org account and gerrit/LDAP account later, in preferences
[20:13:15] <kirstenMA>	 Thanks. I'll try that
[20:13:42] <James_F>	 Cool. :-)
[20:16:54] <Pickle-san>	 I have been trying to sort through a VisualEditor issue on the VisualEditor Extension:Talk page.  The information is under  "Selecting 'Edit' button produces no observable result."  My gut feeling is that a simple configuration change will fix it.  Could I ask for some help in reviewing my set-up?
[20:17:33] <grrrit-wm>	 (03Merged) 10jenkins-bot: Unify data transfer handlers for paste and drop [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/188029 (owner: 10Esanders)
[20:20:15] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki-References, 10Citoid: Citoid: Reopening the Web Reference in media shows the details of the first entry(valid/ invalid-URL /DOI; whatever was entered first in the "Autofill from URL" dialog upon initialization) - https://phabricator.wikimedia.org/T90480#1060190 (10Swai...
[20:20:31] <kirstenMA>	 I created KMenger on phabricator 
[20:22:01] <MatmaRex>	 yay
[20:22:26] <grrrit-wm>	 (03CR) 10Trevor Parscal: [C: 032] Work around Firefox bug allowing cursor in cE=false [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192031 (https://phabricator.wikimedia.org/T70537) (owner: 10Catrope)
[20:22:33] <James_F>	 Yay.
[20:22:37] * James_F assigned.
[20:22:38] <MatmaRex>	 kirstenMA: are you working on these tasks? i'm not sure what's included in your documentation work :)
[20:22:38] <James_F>	 Err.
[20:22:41] * James_F assigns.
[20:23:00] <James_F>	 mooeypoo: https://phabricator.wikimedia.org/T89880 in general.
[20:23:14] <wikibugs>	 10OOjs-UI: Audit all methods and add @private and @protected where appropriate - https://phabricator.wikimedia.org/T88628#1060201 (10Jdforrester-WMF) a:5matmarex>3KMenger
[20:23:22] <wikibugs>	 10OOjs-UI: Document public properties - https://phabricator.wikimedia.org/T88629#1060204 (10Jdforrester-WMF) a:5matmarex>3KMenger
[20:23:33] <James_F>	 etonkovidova: https://phabricator.wikimedia.org/T73085
[20:23:35] <kirstenMA>	 I've been doing that.
[20:24:59] <MatmaRex>	 alright. thanks!
[20:25:36] <kirstenMA>	 sure :)
[20:26:15] <grrrit-wm>	 (03Merged) 10jenkins-bot: Work around Firefox bug allowing cursor in cE=false [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/192031 (https://phabricator.wikimedia.org/T70537) (owner: 10Catrope)
[20:26:20] <wikibugs>	 10VisualEditor: [Regression wmf 18]  "Error: Offset could not be translated to a DOM element and offset" appears while deleting the text inside a bullet/numbered item and then cursor starts behaving weirdly while typing text - https://phabricator.wikimedia.org/T90478#1060217 (10Ryasmeen) p:5Triage>3High
[20:36:58] <grrrit-wm>	 (03PS1) 10Kmenger: Element: Add description for configs and static property [oojs/ui] - 10https://gerrit.wikimedia.org/r/192390 
[20:40:57] <grrrit-wm>	 (03PS1) 10Jforrester: Update VE core submodule to master (2eaf5c9) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192391 
[20:43:09] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Update VE core submodule to master (2eaf5c9) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192391 (owner: 10Jforrester)
[20:44:28] <grrrit-wm>	 (03PS2) 10Jforrester: Update VE core submodule to master (2eaf5c9) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192391 
[20:47:02] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Update VE core submodule to master (2eaf5c9) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192391 (owner: 10Jforrester)
[20:53:41] <grrrit-wm>	 (03CR) 10Trevor Parscal: [C: 032] Simplify slug behaviour [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/190022 (https://phabricator.wikimedia.org/T89192) (owner: 10Esanders)
[20:54:40] <grrrit-wm>	 (03PS3) 10Jforrester: Update VE core submodule to master (2eaf5c9) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192391 
[20:57:57] <wikibugs>	 10VisualEditor, 10VisualEditor-DataModel, 5Patch-For-Review, 5§ VisualEditor Q3 Blockers: VisualEditor: Comments in "unsafe" content locations are not displayed - https://phabricator.wikimedia.org/T73085#1060305 (10Ryasmeen) Comment added after citation not is not getting added , nothing in review changes...
[21:00:47] <grrrit-wm>	 (03Merged) 10jenkins-bot: Simplify slug behaviour [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/190022 (https://phabricator.wikimedia.org/T89192) (owner: 10Esanders)
[21:09:24] <wikibugs>	 10VisualEditor, 10VisualEditor-EditingTools, 7Design, 7Epic, 5§ VisualEditor Q3 Blockers: Cite: Auto-filled references based on destination URL / ISBN / etc. - https://phabricator.wikimedia.org/T62768#1060342 (10KHammerstein)
[21:12:48] <wikibugs>	 10VisualEditor, 10VisualEditor-DataModel, 5Patch-For-Review, 5§ VisualEditor Q3 Blockers: VisualEditor: Comments in "unsafe" content locations are not displayed - https://phabricator.wikimedia.org/T73085#1060357 (10Ryasmeen) Also , comment is not gettiing added after template
[21:20:11] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki, 10VisualEditor-Performance, 10Parsoid, and 2 others: Log Parsoid server-side save performance - https://phabricator.wikimedia.org/T66171#1060393 (10ssastry) 5Open>3Resolved a:3ssastry Now deployed and live in production.
[21:20:39] <wikibugs>	 10VisualEditor, 10VisualEditor-Performance, 10Flow, 10Parsoid, and 4 others: Move data-parsoid and possibly data-mw out of the DOM, add uids - https://phabricator.wikimedia.org/T54936#1060397 (10ssastry)
[21:20:40] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki, 10VisualEditor-Performance, 10Parsoid, and 2 others: Log Parsoid server-side save performance - https://phabricator.wikimedia.org/T66171#1060396 (10ssastry)
[21:21:19] <wikibugs>	 10VisualEditor, 10Parsoid, 7Blocked-on-Parsoid, 7Parsoid-DOM: [Regression] Automatically added reference lists do not have unique about attributes - https://phabricator.wikimedia.org/T90309#1060402 (10ssastry) 5Open>3Resolved
[21:25:19] <Krinkle>	 RoanKattouw: OK
[21:25:31] <Krinkle>	 RoanKattouw: As long as it handles error, dbl clicking is fine
[21:25:36] <James_F>	 TrevorParscal: https://gerrit.wikimedia.org/r/#/c/192391/
[21:26:07] <grrrit-wm>	 (03CR) 10Trevor Parscal: [C: 032] Update VE core submodule to master (2eaf5c9) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192391 (owner: 10Jforrester)
[21:26:29] <wikibugs>	 10MediaWiki-Page-editing, 7JavaScript: LivePreview should give feedback when failing - https://phabricator.wikimedia.org/T90490#1060438 (10TheDJ) 3NEW
[21:27:52] <wikibugs>	 10MediaWiki-Page-editing, 7JavaScript: LivePreview should give feedback when failing - https://phabricator.wikimedia.org/T90490#1060459 (10TheDJ)
[21:27:53] <wikibugs>	 10MediaWiki-Page-editing, 10MediaWiki-RfCs, 7JavaScript, 7Tracking: Live preview (requires JavaScript) feature bugs (tracking) - https://phabricator.wikimedia.org/T41272#1060458 (10TheDJ)
[21:27:56] <grrrit-wm>	 (03PS1) 10Kmenger: LabelElement: Add description, config description, static property description [oojs/ui] - 10https://gerrit.wikimedia.org/r/192455 
[21:29:11] <grrrit-wm>	 (03CR) 10Jforrester: [C: 032] IndicatorElement: Add description for configs and static properties [oojs/ui] - 10https://gerrit.wikimedia.org/r/192369 (owner: 10Kmenger)
[21:29:27] <grrrit-wm>	 (03CR) 10Jforrester: [C: 032] IconElement: Add description of methods [oojs/ui] - 10https://gerrit.wikimedia.org/r/192361 (owner: 10Kmenger)
[21:29:31] <Krenair>	 James_F, who needs to do what to fix the Edit Schema situation?
[21:36:50] <grrrit-wm>	 (03PS11) 10Mooeypoo: Refactor Citoid extension as an inspector [extensions/Citoid] - 10https://gerrit.wikimedia.org/r/190973 (https://phabricator.wikimedia.org/T88152) 
[21:38:54] <Krenair>	 Krinkle, hey
[21:39:00] <Krenair>	 I was looking at https://phabricator.wikimedia.org/T89435
[21:39:17] <Krenair>	 Special:JavaScriptTest/qunit/plain?module=ve.dm.TransactionProcessor passes for me
[21:39:46] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Update VE core submodule to master (2eaf5c9) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192391 (owner: 10Jforrester)
[21:40:07] <Krinkle>	 Krenair: Let me heck
[21:42:22] <Krenair>	 Was this failing in your browser?
[21:42:45] <Krenair>	 looks like it was, based on your comment
[21:42:54] <milimetric>	 could someone point me at where in the VE code EL instrumentation lives?
[21:44:10] <Krenair>	 hmm... you might be looking for https://github.com/wikimedia/mediawiki-extensions-VisualEditor/blob/master/modules/ve-mw/init/ve.init.mw.trackSubscriber.js ?
[21:44:25] * milimetric looks
[21:45:27] <Krenair>	 see also "grep mwedit modules/ve-mw/* -R --exclude-dir=i18n" from the extension directory
[21:45:31] <milimetric>	 that looks good, thanks Krenair.  And answers my other question: "Is it hard to figure out if instrumentation or data is broken": "Yes, yes it is"
[21:47:18] <Krinkle>	 Krenair: Hm.. I
[21:47:26] <wikibugs>	 10VisualEditor, 10VisualEditor-Performance, 6Analytics-Engineering, 6Analytics-Kanban, 5§ VisualEditor Q3 Blockers: Report on the central tendency for length of pages which are edited for VisualEditor performance benchmarking - https://phabricator.wikimedia.org/T89788#1060551 (10Milimetric) The fact that...
[21:47:26] <Krinkle>	 Krenair: Hm.. I'm not seeing that error. I am seeing a differnet error though
[21:47:30] <Krinkle>	 Uncaught TypeError: Cannot read property '0' of undefined
[21:47:34] <Krenair>	 :/
[21:47:35] <Krinkle>	 ve.dm.SurfaceFragment: wrapAllNodes
[21:47:50] <Krinkle>	 and: afterEach failed on getModelFromDom: Unfinished AJAX requests: 1
[21:47:54] <Krinkle>	 from ve.dm.Converter (MW): getModelFromDom
[21:47:57] <Krinkle>	 and two more failures
[21:48:33] <Krinkle>	 I think Roan might've fixed the dependency issue a few days ago
[21:50:23] <grrrit-wm>	 (03Merged) 10jenkins-bot: IndicatorElement: Add description for configs and static properties [oojs/ui] - 10https://gerrit.wikimedia.org/r/192369 (owner: 10Kmenger)
[21:50:25] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] IconElement: Add description of methods [oojs/ui] - 10https://gerrit.wikimedia.org/r/192361 (owner: 10Kmenger)
[21:57:47] <mooeypoo>	 mvolz, you around?
[22:03:22] <Krenair>	 Krinkle, I wonder why I don't get those errors...
[22:03:52] <Krenair>	 Krinkle, oh, SurfaceFragment? I was just running TransactionProcessor tests?
[22:04:01] <Krinkle>	 Krenair: don't filter
[22:04:05] <Krenair>	 ok
[22:04:06] <Krinkle>	 changes things
[22:04:13] <Krinkle>	 separate issue :-(
[22:04:39] <Krinkle>	 Krenair: While I can't reproduce it, Jenkins can
[22:04:44] <Krenair>	 sigh.
[22:04:46] <Krinkle>	 build from a few minutes ago
[22:04:46] <Krinkle>	 https://integration.wikimedia.org/ci/job/mwext-VisualEditor-qunit-karma/419/console
[22:04:50] <Krinkle>	 .constructor undefined
[22:04:51] <Krenair>	 Of course jenkins can.
[22:04:52] <Krinkle>	 the same error I reported
[22:06:01] <Krenair>	 Uncaught TypeError: Cannot read property 'constructor' of undefined
[22:06:02] <Krenair>	 right
[22:06:48] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki-References, 10Citoid: Citoid: Reopening the Web Reference in media shows the details of the first entry(valid/ invalid-URL /DOI; whatever was entered first in the "Autofill from URL" dialog upon initialization) - https://phabricator.wikimedia.org/T90480#1060633 (10Swai...
[22:07:46] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki-References, 10Citoid: Citoid: Reopening the Web Reference in media shows the details of the first entry(valid/ invalid-URL /DOI; whatever was entered first in the "Autofill from URL" dialog upon initialization) - https://phabricator.wikimedia.org/T90480#1060156 (10Swai...
[22:13:23] <Krenair>	 Krinkle, https://gerrit.wikimedia.org/r/#/c/186548/ probably fixes this..
[22:13:37] <Krenair>	 Can't we rely on ve.init.target then?
[22:15:57] <Krinkle>	 Krenair: I was looking at that patch earlier as well
[22:16:01] <Krinkle>	 Can you rebase it one more time?
[22:16:22] <Krenair>	 ok
[22:16:23] <Krinkle>	 I'll check one more time if the load error is handled (some aspect of this changed earlier this week) - and then merge it
[22:18:04] <grrrit-wm>	 (03CR) 10Jforrester: [C: 032] "Dear esprina, you suck. Ktnxbai." [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192391 (owner: 10Jforrester)
[22:19:25] <James_F>	 Krenair: What's broken about Schema:Edit?
[22:20:06] <grrrit-wm>	 (03PS12) 10Alex Monk: Use mw.Api to run requests rather than our own stuff in ve.init.mw.Target [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/186548 (https://phabricator.wikimedia.org/T58659) 
[22:20:23] <Krenair>	 James_F, The issues I brought up with some of the properties
[22:20:24] <grrrit-wm>	 (03Merged) 10jenkins-bot: Update VE core submodule to master (2eaf5c9) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192391 (owner: 10Jforrester)
[22:20:56] <Krenair>	 and their possible values
[22:21:21] <grrrit-wm>	 (03CR) 10Jforrester: Add "composer test" command to lint PHP files and run phpcs (031 comment) [oojs/ui] - 10https://gerrit.wikimedia.org/r/192272 (owner: 10Legoktm)
[22:21:42] <James_F>	 Krenair: Did you write a Phabricator ticket?
[22:21:57] <grrrit-wm>	 (03CR) 10Jforrester: [C: 032] composer.json: Add description field [oojs/ui] - 10https://gerrit.wikimedia.org/r/192271 (owner: 10Legoktm)
[22:22:08] <Krenair>	 I wrote about one of them in an email with analytics
[22:22:20] <grrrit-wm>	 (03CR) 10Legoktm: Add "composer test" command to lint PHP files and run phpcs (031 comment) [oojs/ui] - 10https://gerrit.wikimedia.org/r/192272 (owner: 10Legoktm)
[22:22:24] <grrrit-wm>	 (03CR) 10Jforrester: "Should have a CI test added before this is merged to demonstrate that it works." [oojs/ui] - 10https://gerrit.wikimedia.org/r/192272 (owner: 10Legoktm)
[22:23:14] <grrrit-wm>	 (03CR) 10Jforrester: Add "composer test" command to lint PHP files and run phpcs (031 comment) [oojs/ui] - 10https://gerrit.wikimedia.org/r/192272 (owner: 10Legoktm)
[22:23:35] <grrrit-wm>	 (03Merged) 10jenkins-bot: composer.json: Add description field [oojs/ui] - 10https://gerrit.wikimedia.org/r/192271 (owner: 10Legoktm)
[22:24:59] <grrrit-wm>	 (03CR) 10Legoktm: Add "composer test" command to lint PHP files and run phpcs (031 comment) [oojs/ui] - 10https://gerrit.wikimedia.org/r/192272 (owner: 10Legoktm)
[22:26:22] <Krenair>	 Krinkle, hm, I don't think it's quite handling errors correctly yet
[22:29:05] <Krinkle>	 Krenair: I'm debugging something ele, but let me know if it's ready for review :)
[22:30:34] <Krenair>	 ok
[22:31:39] <Krenair>	 {"error":{"code":"0","info":"parsoidserver-http: HTTP 0","*":"See http://mediawiki.krenair.dev/mediawiki_dev/w/api.php for API usage"}}
[22:31:56] <Krenair>	 very helpful :/
[22:42:27] <grrrit-wm>	 (03PS13) 10Alex Monk: Use mw.Api to run requests rather than our own stuff in ve.init.mw.Target [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/186548 (https://phabricator.wikimedia.org/T58659) 
[22:52:41] <wikibugs>	 10VisualEditor, 10WikiEditor, 10Flow, 6§Collaboration-Team, 7Design: Editing toolbar(s) for Flow (VE and/or wikitext toolbar) - https://phabricator.wikimedia.org/T78346#1060761 (10Quiddity) >>! In T78346#1038829, @GOIII wrote: > So until we either actively poll for as much or implement & advertise those...
[22:58:36] <grrrit-wm>	 (03PS1) 10Kmenger: TitledElement: Add description and config and static descriptions [oojs/ui] - 10https://gerrit.wikimedia.org/r/192469 
[23:03:08] <wikibugs>	 10TemplateData, 5WMF-deploy-2015-02-25_(1.25wmf19): TemplateData: Display a useful message instead of an empty table for templates that take no parameters - https://phabricator.wikimedia.org/T53429#1060845 (10Etonkovidova) The fix in beta:  {F45817}  vs test2:   {F45819}
[23:06:50] <grrrit-wm>	 (03CR) 10Ori.livneh: "* Do you really need to have 'feedbackPromise' stored? Multiple mw.loader.using( 'mediawiki.feedback', ... ) won't result in multiple netw" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192378 (owner: 10Catrope)
[23:07:55] <ori>	 RoanKattouw: ^
[23:22:24] <grrrit-wm>	 (03CR) 10Catrope: "- No, I can dispense with that, good point" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/192378 (owner: 10Catrope)
[23:22:43] <RoanKattouw>	 Krenair: IME HTTP 0 means it failed to connect to the Parsoid server
[23:22:53] <Krenair>	 yes
[23:25:02] <grrrit-wm>	 (03CR) 10Catrope: [C: 04-1] Refactor Citoid extension as an inspector (031 comment) [extensions/Citoid] - 10https://gerrit.wikimedia.org/r/190973 (https://phabricator.wikimedia.org/T88152) (owner: 10Mooeypoo)
[23:32:13] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki, 7Technical-Debt: Test failure: Circular dependency between mw.Target and mw.ImageInfoCache - https://phabricator.wikimedia.org/T89435#1060937 (10Krenair) https://gerrit.wikimedia.org/r/#/c/186548/ would probably solve this.
[23:49:04] <wikibugs>	 10VisualEditor, 10WikiEditor, 10Flow, 6§Collaboration-Team, 7Design: Editing toolbar(s) for Flow (VE and/or wikitext toolbar) - https://phabricator.wikimedia.org/T78346#1060956 (10GOIII) >>! In T78346#1060761, @Quiddity wrote: >>>! In T78346#1038829, @GOIII wrote: >> So until we either actively poll for...
[23:50:21] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki-References, 10Citoid: Citoid: Reopening the Web Reference in media shows the details of the first entry(valid/ invalid-URL /DOI; whatever was entered first in the "Autofill from URL" dialog upon initialization) - https://phabricator.wikimedia.org/T90480#1060958 (10Eton...
[23:50:37] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki-References, 10Citoid: Citoid: Reopening the Web Reference in media shows the details of the first entry(valid/ invalid-URL /DOI; whatever was entered first in the "Autofill from URL" dialog upon initialization) - https://phabricator.wikimedia.org/T90480#1060959 (10Eton...
[23:52:46] <grrrit-wm>	 (03CR) 10Jagori79: "Hi Zeljko," [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/122400 (owner: 10Jagori79)