[00:01:51] (03PS1) 10Esanders: Move generic VE desktop styles into a separate module [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268922 [00:13:28] (03CR) 10jenkins-bot: [V: 04-1] Move generic VE desktop styles into a separate module [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268922 (owner: 10Esanders) [01:01:01] 10VisualEditor, 10VisualEditor-EditingTools: Reconsider whether a multi-surface target should only have one toolbar that moves to the active surface, or if this should be configurable - https://phabricator.wikimedia.org/T126144#2005843 (10Jdforrester-WMF) 3NEW [01:21:52] 10VisualEditor, 10VisualEditor-MediaWiki, 6Editing-Department, 10MediaWiki-Page-editing: When CustomEditor is used, the EditPage robot policy isn't loaded - https://phabricator.wikimedia.org/T126145#2005861 (10Jdforrester-WMF) 3NEW a:3Krenair [01:22:44] 10VisualEditor, 10VisualEditor-MediaWiki, 6Editing-Department, 10MediaWiki-Page-editing, 5Patch-For-Review: When CustomEditor is used, the EditPage robot policy isn't loaded - https://phabricator.wikimedia.org/T126145#2005877 (10Jdforrester-WMF) [01:22:48] 10VisualEditor, 10VisualEditor-MediaWiki, 7Design, 7Epic: Change VE integration to only use one edit tab, to avoid user choice paralysis - https://phabricator.wikimedia.org/T58337#2005876 (10Jdforrester-WMF) [01:22:56] 10VisualEditor, 10VisualEditor-MediaWiki, 5Patch-For-Review: VisualEditor SET doesn't set the element - https://phabricator.wikimedia.org/T125670#2005878 (10Jdforrester-WMF) [01:24:33] <wikibugs> 10VisualEditor, 10VisualEditor-MediaWiki, 7Design, 7Epic: Change VE integration to only use one edit tab, to avoid user choice paralysis - https://phabricator.wikimedia.org/T58337#604549 (10Jdforrester-WMF) [01:24:37] <wikibugs> 10VisualEditor, 10VisualEditor-MediaWiki, 5WMF-deploy-2016-02-09_(1.27.0-wmf.13): VisualEditor SET doesn't set the <title> element - https://phabricator.wikimedia.org/T125670#2005888 (10Jdforrester-WMF) 5Open>3Resolved a:3Krenair [01:24:53] <wikibugs> 10VisualEditor, 10VisualEditor-MediaWiki, 5WMF-deploy-2016-02-09_(1.27.0-wmf.13): VisualEditor SET doesn't set the <title> element - https://phabricator.wikimedia.org/T125670#1994207 (10Jdforrester-WMF) Split the second part into {T126145}. [01:25:41] <wikibugs> 10VisualEditor, 10VisualEditor-MediaWiki, 6Editing-Department, 10MediaWiki-Page-editing, 5Patch-For-Review: When CustomEditor is used, the EditPage robot policy isn't loaded - https://phabricator.wikimedia.org/T126145#2005902 (10Jdforrester-WMF) p:5Triage>3Normal [01:28:04] <wikibugs> 10VisualEditor, 10VisualEditor-MediaWiki: Use different preferences for the seen state of each of the educational pop-ups - https://phabricator.wikimedia.org/T117638#2005910 (10Jdforrester-WMF) p:5Normal>3Low [01:28:37] <wikibugs> 10VisualEditor, 10VisualEditor-MediaWiki, 10MediaWiki-extensions-UniversalLanguageSelector: Integrate jQuery IME control with the VE toolbar - https://phabricator.wikimedia.org/T126125#2005912 (10Jdforrester-WMF) p:5Triage>3Low [01:44:13] <grrrit-wm> (03PS1) 10Jforrester: [BREAKING CHANGE] Drop MW pre-1.25 PHP API documentation compatibility [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268924 [01:51:39] <wikibugs> 10Cite, 6TCB-Team, 7German-Community-Wishlist, 5Patch-For-Review: Users should be (more) aware that references in section preview aren't a part of final rendered section - https://phabricator.wikimedia.org/T125981#2005923 (10Izno) >>! In T125981#2005213, @Cenarium wrote: > {F3316401} > Is this better ? > I... [03:05:52] <grrrit-wm> (03CR) 10Jforrester: [C: 032] DesktopArticleTarget: Cleanup styles [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268916 (owner: 10Esanders) [03:06:52] <wikibugs> 10VisualEditor, 10VisualEditor-MediaWiki, 6Collaboration-Team-Backlog, 10Flow, 5Patch-For-Review: VE dialog not positioned above personal items in Flow - https://phabricator.wikimedia.org/T126135#2005960 (10Jdforrester-WMF) [03:07:03] <wikibugs> 10VisualEditor, 10VisualEditor-MediaWiki, 6Collaboration-Team-Backlog, 10Flow, 5Patch-For-Review: VE dialog not positioned above personal items in Flow - https://phabricator.wikimedia.org/T126135#2005661 (10Jdforrester-WMF) [03:20:38] <grrrit-wm> (03CR) 10jenkins-bot: [V: 04-1] DesktopArticleTarget: Cleanup styles [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268916 (owner: 10Esanders) [03:21:29] <James_F> Perplexing. [03:24:27] <grrrit-wm> (03PS1) 10Jforrester: jsduck: Add `Text` type to external known styles [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268929 [03:25:12] <grrrit-wm> (03PS2) 10Jforrester: DesktopArticleTarget: Cleanup styles [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268916 (owner: 10Esanders) [03:25:44] <grrrit-wm> (03CR) 10Jforrester: "PS2: Rebase onto patch fixing the repo." [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268916 (owner: 10Esanders) [03:26:03] <grrrit-wm> (03PS2) 10Jforrester: Remove global overlay hack now fixed in Vector [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268917 (owner: 10Esanders) [03:26:08] <grrrit-wm> (03CR) 10Jforrester: [C: 032] Remove global overlay hack now fixed in Vector [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268917 (owner: 10Esanders) [03:31:52] <grrrit-wm> (03PS2) 10Jforrester: Follow-up I7e1edcf05c: Remove monobook z-index hack [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268918 (owner: 10Esanders) [03:32:17] <grrrit-wm> (03CR) 10Jforrester: "This was definitely needed when we wrote it. What changed? Did MonoBook's structure get redone?" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268918 (owner: 10Esanders) [03:32:25] <grrrit-wm> (03PS2) 10Jforrester: Move generic VE desktop styles into a separate module [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268922 (owner: 10Esanders) [03:33:28] <grrrit-wm> (03PS2) 10Jforrester: MWImageMode: Use clearDefaultDimensions [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268900 (https://phabricator.wikimedia.org/T126078) (owner: 10Esanders) [03:34:04] <wikibugs> 10VisualEditor, 7I18n: [[MediaWiki:Apihelp-visualeditor-param-pst/eo]] i18n issue - https://phabricator.wikimedia.org/T125798#2005971 (10Jdforrester-WMF) The implicit part of the comment is "//If this is set, do a// Pre-save transform //on the// wikitext before sending //the resultant pre-save transformed wiki... [03:34:37] <wikibugs> 10VisualEditor, 10VisualEditor-MediaWiki, 6Collaboration-Team-Backlog, 10Flow, 5Patch-For-Review: VE dialog not positioned above personal items in Flow - https://phabricator.wikimedia.org/T126135#2005973 (10Jdforrester-WMF) p:5Triage>3Normal [03:34:56] <wikibugs> 10VisualEditor, 10VisualEditor-Media, 10VisualEditor-MediaWiki, 5Patch-For-Review, 7Technical-Debt: ve.dm.MWImageModel.prototype.resetDefaultDimensions: Calling `this.scalable.setDefaultDimensions( {} );` is a no-op - https://phabricator.wikimedia.org/T126078#2005976 (10Jdforrester-WMF) p:5Triage>3Low... [03:36:13] <grrrit-wm> (03CR) 10Jforrester: "PS2: Rebase onto patch fixing the repo." [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268900 (https://phabricator.wikimedia.org/T126078) (owner: 10Esanders) [03:37:22] <wikibugs> 10VisualEditor, 10VisualEditor-MediaWiki, 6Collaboration-Team-Backlog, 10Flow, 5WMF-deploy-2016-02-09_(1.27.0-wmf.13): VE dialog not positioned above personal items in Flow - https://phabricator.wikimedia.org/T126135#2005979 (10Jdforrester-WMF) 5Open>3Resolved [06:00:03] <grrrit-wm> (03CR) 10Alex Monk: [C: 032] jsduck: Add `Text` type to external known styles [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268929 (owner: 10Jforrester) [06:17:32] <grrrit-wm> (03Merged) 10jenkins-bot: jsduck: Add `Text` type to external known styles [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268929 (owner: 10Jforrester) [06:28:04] <grrrit-wm> (03Merged) 10jenkins-bot: DesktopArticleTarget: Cleanup styles [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268916 (owner: 10Esanders) [06:28:06] <grrrit-wm> (03Merged) 10jenkins-bot: Remove global overlay hack now fixed in Vector [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268917 (owner: 10Esanders) [07:01:17] <grrrit-wm> (03PS1) 10Alex Monk: Empty special character page before adding more characters to it [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268935 (https://phabricator.wikimedia.org/T126138) [07:02:08] <wikibugs> 10WikiEditor, 5Patch-For-Review: Every click after the initial selection of a character set in WikiEditor contiguously loads the entire group at the end of the previous group - https://phabricator.wikimedia.org/T126138#2006041 (10Krenair) a:3Krenair [09:00:21] <wikibugs> 10VisualEditor, 10Parsoid: Inserting the English Wikipedia {{won}} template into a table cell using VisualEditor shows unnecessary code - https://phabricator.wikimedia.org/T124934#2006068 (10Amire80) >>! In T124934#2005718, @ssastry wrote: > @esanders, The template [[https://en.wikipedia.org/w/index.php?title=... [09:01:15] <wikibugs> 10VisualEditor, 10Parsoid: Inserting the English Wikipedia {{won}} template into a table cell using VisualEditor shows unnecessary code - https://phabricator.wikimedia.org/T124934#2006069 (10Amire80) [09:20:13] <grrrit-wm> (03CR) 10Cenarium: [C: 031] "Yeah, that was my rationale for making them null in the first place, but I forgot about it. Looks good now." [extensions/TemplateData] - 10https://gerrit.wikimedia.org/r/266661 (owner: 10Cenarium) [09:45:19] <grrrit-wm> (03CR) 10Fomafix: [C: 031] "Tested." [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268935 (https://phabricator.wikimedia.org/T126138) (owner: 10Alex Monk) [10:10:40] <grrrit-wm> (03CR) 10Esanders: "Styles?" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268929 (owner: 10Jforrester) [10:14:38] <grrrit-wm> (03CR) 10Esanders: [C: 032] DesktopArticleTarget#editSource: Use actionFactory to create a window with the surface [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268857 (https://phabricator.wikimedia.org/T125479) (owner: 10Alex Monk) [10:30:08] <grrrit-wm> (03Merged) 10jenkins-bot: DesktopArticleTarget#editSource: Use actionFactory to create a window with the surface [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268857 (https://phabricator.wikimedia.org/T125479) (owner: 10Alex Monk) [12:18:23] <grrrit-wm> (03PS13) 10Paladox: Allow the signature wikitext to be overridden locally [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/258783 (https://phabricator.wikimedia.org/T26191) [12:18:43] <grrrit-wm> (03PS14) 10Paladox: Allow the signature wikitext to be overridden locally [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/258783 (https://phabricator.wikimedia.org/T26191) [13:41:16] <wikibugs> 10VisualEditor, 10VisualEditor-EditingTools, 10VisualEditor-MediaWiki, 5Patch-For-Review: Back out pending nodes/etc. on switching editor so that e.g. half-made references aren't created - https://phabricator.wikimedia.org/T125479#2006143 (10Krenair) 5Open>3Resolved [14:22:56] <grrrit-wm> (03PS1) 10Gerrit Patch Uploader: Use .prop() instead of .attr()/.removeAttr() for boolean HTML attributes [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268944 [14:22:59] <grrrit-wm> (03CR) 10Gerrit Patch Uploader: "This commit was uploaded using the Gerrit Patch Uploader [1]." [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268944 (owner: 10Gerrit Patch Uploader) [14:47:32] <grrrit-wm> (03CR) 10Fomafix: [C: 04-1] Allow the signature wikitext to be overridden locally (031 comment) [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/258783 (https://phabricator.wikimedia.org/T26191) (owner: 10Paladox) [15:25:07] <wikibugs> 10VisualEditor, 10Parsoid: Inserting the English Wikipedia {{won}} template into a table cell using VisualEditor shows unnecessary code - https://phabricator.wikimedia.org/T124934#2006234 (10ssastry) I now see what is going on. ``` [subbu@earth tools] echo '{{won}}' | parse.js ... <p about="#mwt1" typeof="mw:... [16:07:47] <wikibugs> 10VisualEditor, 10MediaWiki-extensions-ContentTranslation, 7Community-Wishlist-Survey: Merge the Content Translation tool and VisualEditor - https://phabricator.wikimedia.org/T121475#2006300 (10Amire80) I have a bit of a //déjà vu//: T120476 :) [16:10:27] <wikibugs> 10Cite, 5Patch-For-Review, 3Reading-Web-Sprint-65-Game of Phones: Save references in page_props and cache - https://phabricator.wikimedia.org/T125329#2006316 (10jcrespo) 4 Million rows in a 26 million table does not worry me too much. The average row length now, however, is 79 bytes. I do not know if it will... [16:56:15] <grrrit-wm> (03CR) 10Jforrester: "s/styles/classes/; apparently I shouldn't make commits at 19:30 on a Saturday. :-)" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268929 (owner: 10Jforrester) [16:57:40] <yurik> edsanders|away, ping [16:57:45] <yurik> ah, you are away :) [16:58:50] <grrrit-wm> (03CR) 10Alex Monk: [C: 032] [BREAKING CHANGE] Drop MW pre-1.25 PHP API documentation compatibility [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268924 (owner: 10Jforrester) [17:04:28] <grrrit-wm> (03CR) 10GOIII: [C: 031] Empty special character page before adding more characters to it [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268935 (https://phabricator.wikimedia.org/T126138) (owner: 10Alex Monk) [17:15:38] <grrrit-wm> (03Merged) 10jenkins-bot: [BREAKING CHANGE] Drop MW pre-1.25 PHP API documentation compatibility [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268924 (owner: 10Jforrester) [17:16:55] <grrrit-wm> (03PS1) 10Gerrit Patch Uploader: JavaScript: Use single quotes instead of double quotes for strings [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268955 [17:16:58] <grrrit-wm> (03CR) 10Gerrit Patch Uploader: "This commit was uploaded using the Gerrit Patch Uploader [1]." [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268955 (owner: 10Gerrit Patch Uploader) [17:34:28] <grrrit-wm> (03PS2) 10Esanders: ResizableNode: Remove assumption about 'width'/'height' attributes [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/268059 (https://phabricator.wikimedia.org/T126092) [17:46:30] <wikibugs> 10VisualEditor, 6Discovery, 10Kartographer, 10Maps: Copied adjacent extension objects collapsed on save - https://phabricator.wikimedia.org/T126114#2006486 (10Esanders) a:3Esanders [17:47:30] <wikibugs> 10VisualEditor, 6Discovery, 10Kartographer, 10Maps: Copied adjacent extension objects collapsed on save - https://phabricator.wikimedia.org/T126114#2005242 (10Esanders) This is because the 'about' attribute is also copied. Parsoid uses this to ignore siblings which are just part of the rendering. We fixed... [17:55:59] <grrrit-wm> (03CR) 10TheDJ: [C: 032] Use .prop() instead of .attr()/.removeAttr() for boolean HTML attributes [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268944 (owner: 10Gerrit Patch Uploader) [17:58:42] <grrrit-wm> (03Merged) 10jenkins-bot: Use .prop() instead of .attr()/.removeAttr() for boolean HTML attributes [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268944 (owner: 10Gerrit Patch Uploader) [18:01:26] <grrrit-wm> (03CR) 10TheDJ: [C: 04-1] "Please fix it everywhere and then flip the switch in the .jscrc file, so that we can verify this going forward with 'npm test'" [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268955 (owner: 10Gerrit Patch Uploader) [18:07:23] <grrrit-wm> (03CR) 10TheDJ: "Although this works, it probably doesn't fix the deeper problem. Namely that reallybuildpage, apparently always rebuilds page, which I don" [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268935 (https://phabricator.wikimedia.org/T126138) (owner: 10Alex Monk) [18:17:54] <grrrit-wm> (03CR) 10Esanders: Update VE core submodule to master (7d2a68e) (031 comment) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/190963 (owner: 10Catrope) [18:22:07] <grrrit-wm> (03PS2) 10Gerrit Patch Uploader: JavaScript: Use single quotes instead of double quotes for strings [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268955 [18:22:10] <grrrit-wm> (03CR) 10Gerrit Patch Uploader: "This commit was uploaded using the Gerrit Patch Uploader [1]." [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268955 (owner: 10Gerrit Patch Uploader) [18:24:22] <wikibugs> 10VisualEditor: [regression] Adjacent copied inline template is destroyed by Parsoid - https://phabricator.wikimedia.org/T126169#2006518 (10Esanders) 3NEW a:3Esanders [18:25:55] <grrrit-wm> (03PS1) 10Esanders: cloneElements: Use the correct node type when cloning [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/268963 (https://phabricator.wikimedia.org/T126169) [18:31:53] <grrrit-wm> (03PS1) 10Esanders: Fix transclusion node cloneElement method [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268966 (https://phabricator.wikimedia.org/T126169) [18:33:53] <grrrit-wm> (03PS2) 10Esanders: Fix transclusion node cloneElement method [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/268966 (https://phabricator.wikimedia.org/T126169) [18:46:16] <wikibugs> 10VisualEditor, 6Discovery, 10Kartographer, 10Maps, 5Patch-For-Review: Copied adjacent extension objects collapsed on save - https://phabricator.wikimedia.org/T126114#2006548 (10Esanders) or at least we //did// fix it for templates. Turns our you found a one year old regression: T126169 [18:53:37] <grrrit-wm> (03CR) 10Fomafix: "Correct. The reason for the rebuilding should be solved." [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268935 (https://phabricator.wikimedia.org/T126138) (owner: 10Alex Monk) [19:05:12] <grrrit-wm> (03CR) 10GOIII: "At this point, I take whatever improvements I can get - but yes, that would appear to be a better solution in a perfect world." [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268935 (https://phabricator.wikimedia.org/T126138) (owner: 10Alex Monk) [19:11:49] <grrrit-wm> (03CR) 10GOIII: "I'm not 100% sure deferLoad is not being used by some other extension that adds to the toolbar in some way so why not keep the code and ju" [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268935 (https://phabricator.wikimedia.org/T126138) (owner: 10Alex Monk) [19:14:18] <grrrit-wm> (03PS1) 10Gerrit Patch Uploader: Remove deferLoad of the toolbar [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268968 (https://phabricator.wikimedia.org/T126138) [19:14:21] <grrrit-wm> (03CR) 10Gerrit Patch Uploader: "This commit was uploaded using the Gerrit Patch Uploader [1]." [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268968 (https://phabricator.wikimedia.org/T126138) (owner: 10Gerrit Patch Uploader) [19:25:17] <grrrit-wm> (03CR) 10Fomafix: "Made a change for removing the deferLoad in I49909ef35b87ce9091cf9897f278adcc420c14aa." [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268935 (https://phabricator.wikimedia.org/T126138) (owner: 10Alex Monk) [19:36:11] <grrrit-wm> (03CR) 10GOIII: [C: 04-1] "Forget about my concerns; go with the patch that removes all bits relating to deferLoad instead of this one" [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268935 (https://phabricator.wikimedia.org/T126138) (owner: 10Alex Monk) [19:38:11] <grrrit-wm> (03CR) 10GOIII: [C: 031] "Much better solution than the one in" [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268968 (https://phabricator.wikimedia.org/T126138) (owner: 10Gerrit Patch Uploader) [19:44:18] <grrrit-wm> (03PS2) 10Gerrit Patch Uploader: Remove deferLoad of the toolbar [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268968 (https://phabricator.wikimedia.org/T126138) [19:44:21] <grrrit-wm> (03CR) 10Gerrit Patch Uploader: "This commit was uploaded using the Gerrit Patch Uploader [1]." [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268968 (https://phabricator.wikimedia.org/T126138) (owner: 10Gerrit Patch Uploader) [20:20:07] <grrrit-wm> (03PS3) 10Gerrit Patch Uploader: Remove deferLoad of the toolbar [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268968 (https://phabricator.wikimedia.org/T126138) [20:20:10] <grrrit-wm> (03CR) 10Gerrit Patch Uploader: "This commit was uploaded using the Gerrit Patch Uploader [1]." [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268968 (https://phabricator.wikimedia.org/T126138) (owner: 10Gerrit Patch Uploader) [21:16:04] <grrrit-wm> (03CR) 10Alex Monk: [C: 032] ResizableNode: Remove assumption about 'width'/'height' attributes [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/268059 (https://phabricator.wikimedia.org/T126092) (owner: 10Esanders) [21:23:14] <grrrit-wm> (03Merged) 10jenkins-bot: ResizableNode: Remove assumption about 'width'/'height' attributes [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/268059 (https://phabricator.wikimedia.org/T126092) (owner: 10Esanders) [21:27:22] <grrrit-wm> (03CR) 10Paladox: "recheck" [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268968 (https://phabricator.wikimedia.org/T126138) (owner: 10Gerrit Patch Uploader) [23:26:29] <grrrit-wm> (03CR) 10TheDJ: [C: 032] JavaScript: Use single quotes instead of double quotes for strings [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268955 (owner: 10Gerrit Patch Uploader) [23:28:42] <grrrit-wm> (03Merged) 10jenkins-bot: JavaScript: Use single quotes instead of double quotes for strings [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/268955 (owner: 10Gerrit Patch Uploader) [23:55:18] <wikibugs> 10Cite, 5Patch-For-Review, 3Reading-Web-Sprint-65-Game of Phones: Save references in page_props and cache - https://phabricator.wikimedia.org/T125329#2006973 (10Cenarium) >>! In T125329#2006316, @jcrespo wrote: > 4 Million rows in a 26 million table does not worry me too much. The average row length now, how...