[00:42:30] <legoktm>	 eeek https://www.mediawiki.org/w/index.php?title=Wikibugs&diff=prev&oldid=2096741
[00:47:43] <wikibugs>	 10VisualEditor: VisualEditor saves section as page text (deleting all other sections) - https://phabricator.wikimedia.org/T132309#2194062 (10Legoktm)
[09:35:29] <Josve05a>	 again, pretty important discussions about enabling VE on different namespaces on svwp. Input from devs. wanted... https://sv.wikipedia.org/wiki/Wikipedia:VisualEditor/%C3%85terkoppling#Bybrunnsdiskussion_om_.C3.B6nskem.C3.A5l_om_VE_p.C3.A5_diskussionssidor_och_andra_namnrymder
[11:53:09] <grrrit-wm>	 (03PS1) 10Esanders: Move toolbar positioning logic to PositionedTargetToolbar [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282671 
[11:53:11] <grrrit-wm>	 (03PS1) 10Esanders: Add support for placing a toolbar dialog in the side margin [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282672 
[11:53:13] <grrrit-wm>	 (03PS1) 10Esanders: [NEVER MERGE] Move special character dialog to the sidebar [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282673 
[11:56:42] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Move toolbar positioning logic to PositionedTargetToolbar [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282671 (owner: 10Esanders)
[11:56:52] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Add support for placing a toolbar dialog in the side margin [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282672 (owner: 10Esanders)
[11:57:55] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] [NEVER MERGE] Move special character dialog to the sidebar [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282673 (owner: 10Esanders)
[12:22:02] <grrrit-wm>	 (03PS2) 10Esanders: Add support for placing a toolbar dialog in the side margin [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282672 
[12:22:04] <grrrit-wm>	 (03PS2) 10Esanders: [NEVER MERGE] Move special character dialog to the sidebar [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282673 
[12:22:06] <grrrit-wm>	 (03PS1) 10Esanders: Adjust DOM and CSS so UI surface runs from top to bottom of page [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/282678 
[12:22:08] <grrrit-wm>	 (03PS1) 10Esanders: [PULL THROUGH] Positioned toolbar class [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/282679 
[12:25:17] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Add support for placing a toolbar dialog in the side margin [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282672 (owner: 10Esanders)
[12:26:35] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] [NEVER MERGE] Move special character dialog to the sidebar [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282673 (owner: 10Esanders)
[12:27:16] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] [PULL THROUGH] Positioned toolbar class [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/282679 (owner: 10Esanders)
[12:27:18] <grrrit-wm>	 (03PS9) 10Esanders: [NEVER MERGE] Link suggestion gadget [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/257040 (https://phabricator.wikimedia.org/T119477) 
[13:12:23] <grrrit-wm>	 (03PS3) 10Esanders: Add support for placing a toolbar dialog in the side margin [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282672 
[13:12:25] <grrrit-wm>	 (03PS3) 10Esanders: [NEVER MERGE] Move special character dialog to the sidebar [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282673 
[13:12:27] <grrrit-wm>	 (03PS1) 10Esanders: Put border around whole target [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282687 
[13:13:20] <grrrit-wm>	 (03PS1) 10Esanders: Protect against null surface in renderFragments [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282688 
[13:13:47] <MatmaRex>	 :o
[13:16:14] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Put border around whole target [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282687 (owner: 10Esanders)
[13:16:59] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Add support for placing a toolbar dialog in the side margin [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282672 (owner: 10Esanders)
[13:17:12] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] [NEVER MERGE] Move special character dialog to the sidebar [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282673 (owner: 10Esanders)
[13:33:52] <grrrit-wm>	 (03CR) 10Jforrester: [C: 032] "Works fine with current RB release too. Nice to get this unblocked." [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/282448 (https://phabricator.wikimedia.org/T130757) (owner: 10Alex Monk)
[13:37:56] <grrrit-wm>	 (03CR) 10Jforrester: [C: 032] Rename ve-ce-sectionNode-focused to active [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282462 (owner: 10Esanders)
[13:39:25] <grrrit-wm>	 (03CR) 10Jforrester: [C: 032] Implement ve.dm.Selection#getCoveringRange [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282463 (owner: 10Esanders)
[13:40:26] <grrrit-wm>	 (03CR) 10Jforrester: Prevent selection from spanning an active node (031 comment) [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282464 (owner: 10Esanders)
[13:41:41] <grrrit-wm>	 (03PS3) 10Jforrester: Prevent selection from spanning an active node [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282464 (owner: 10Esanders)
[13:42:00] <grrrit-wm>	 (03PS4) 10Jforrester: ce.Surface: Prevent selection from spanning an active node [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282464 (owner: 10Esanders)
[13:42:19] <grrrit-wm>	 (03CR) 10Jforrester: [C: 032] "PS2: Drop old comment, no longer relevant; PS3, adjust title." [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282464 (owner: 10Esanders)
[13:44:23] <grrrit-wm>	 (03Merged) 10jenkins-bot: Add ?redirect=false to VE's RB /page/html/ requests [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/282448 (https://phabricator.wikimedia.org/T130757) (owner: 10Alex Monk)
[13:44:26] <grrrit-wm>	 (03Merged) 10jenkins-bot: Rename ve-ce-sectionNode-focused to active [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282462 (owner: 10Esanders)
[13:44:59] <grrrit-wm>	 (03Merged) 10jenkins-bot: Implement ve.dm.Selection#getCoveringRange [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282463 (owner: 10Esanders)
[13:45:03] <grrrit-wm>	 (03PS2) 10Jforrester: Move toolbar positioning logic to PositionedTargetToolbar [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282671 (owner: 10Esanders)
[13:45:23] <grrrit-wm>	 (03CR) 10Jforrester: "PS2: Fixed jsduck error (absent from manifest)." [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282671 (owner: 10Esanders)
[13:47:20] <grrrit-wm>	 (03CR) 10Jforrester: [C: 032] Adjust DOM and CSS so UI surface runs from top to bottom of page [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/282678 (owner: 10Esanders)
[13:47:41] <grrrit-wm>	 (03CR) 10Jforrester: [C: 032] Move toolbar positioning logic to PositionedTargetToolbar [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282671 (owner: 10Esanders)
[13:48:02] <grrrit-wm>	 (03Merged) 10jenkins-bot: ce.Surface: Prevent selection from spanning an active node [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282464 (owner: 10Esanders)
[13:49:17] <grrrit-wm>	 (03PS2) 10Jforrester: demo: Put border around whole target [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282687 (owner: 10Esanders)
[13:49:21] <grrrit-wm>	 (03PS3) 10Jforrester: demo: Put border around whole target [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282687 (owner: 10Esanders)
[13:49:26] <grrrit-wm>	 (03CR) 10Jforrester: [C: 032] demo: Put border around whole target [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282687 (owner: 10Esanders)
[13:59:45] <grrrit-wm>	 (03CR) 10Jforrester: [C: 032] More minor/watch checkbox state computation server-side [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/281312 (owner: 10Catrope)
[14:03:31] <grrrit-wm>	 (03Merged) 10jenkins-bot: Adjust DOM and CSS so UI surface runs from top to bottom of page [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/282678 (owner: 10Esanders)
[14:03:33] <grrrit-wm>	 (03Merged) 10jenkins-bot: Move toolbar positioning logic to PositionedTargetToolbar [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282671 (owner: 10Esanders)
[14:03:35] <grrrit-wm>	 (03Merged) 10jenkins-bot: demo: Put border around whole target [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282687 (owner: 10Esanders)
[14:06:20] <grrrit-wm>	 (03PS2) 10Jforrester: Update VE core submodule to master (f8be8c3) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/282679 (owner: 10Esanders)
[14:06:32] <grrrit-wm>	 (03CR) 10Jforrester: [C: 032] Update VE core submodule to master (f8be8c3) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/282679 (owner: 10Esanders)
[14:09:46] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki, 10Beta-Cluster-Infrastructure: Beta Cluster threw a 'internal_api_error_DBConnectionError' at me - https://phabricator.wikimedia.org/T129192#2194879 (10Jdforrester-WMF) >>! In T129192#2101951, @AlexMonk-WMF wrote: > I don't think there's anything for VE to do here....
[14:09:50] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki, 10Beta-Cluster-Infrastructure: Beta Cluster threw a 'internal_api_error_DBConnectionError' at me - https://phabricator.wikimedia.org/T129192#2194880 (10Jdforrester-WMF) a:5Jdforrester-WMF>3None
[14:10:39] <grrrit-wm>	 (03PS4) 10Esanders: Add support for placing a toolbar dialog in the side margin [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282672 
[14:10:41] <grrrit-wm>	 (03PS4) 10Esanders: [NEVER MERGE] Move special character dialog to the sidebar [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282673 
[14:10:45] <grrrit-wm>	 (03PS2) 10Jforrester: FindAndReplaceDialog#renderFragments: Protect against null surface [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282688 (owner: 10Esanders)
[14:10:50] <grrrit-wm>	 (03CR) 10Jforrester: [C: 032] FindAndReplaceDialog#renderFragments: Protect against null surface [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282688 (owner: 10Esanders)
[14:13:38] <grrrit-wm>	 (03CR) 10Jforrester: "Interesting. We'd probably want to shrink the height of the menu a fair bit first for VE-core, though it looks rather nice in VE-MW." [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282673 (owner: 10Esanders)
[14:13:52] <grrrit-wm>	 (03CR) 10Jforrester: [C: 032] Add support for placing a toolbar dialog in the side margin [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282672 (owner: 10Esanders)
[14:15:30] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] [NEVER MERGE] Move special character dialog to the sidebar [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282673 (owner: 10Esanders)
[14:17:15] <James_F>	 edsanders: Review on https://gerrit.wikimedia.org/r/#/c/281456/ greatly appreciated.
[14:18:58] <grrrit-wm>	 (03Merged) 10jenkins-bot: More minor/watch checkbox state computation server-side [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/281312 (owner: 10Catrope)
[14:24:24] <grrrit-wm>	 (03Merged) 10jenkins-bot: Update VE core submodule to master (f8be8c3) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/282679 (owner: 10Esanders)
[14:24:27] <grrrit-wm>	 (03Merged) 10jenkins-bot: FindAndReplaceDialog#renderFragments: Protect against null surface [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282688 (owner: 10Esanders)
[14:24:29] <grrrit-wm>	 (03Merged) 10jenkins-bot: Add support for placing a toolbar dialog in the side margin [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/282672 (owner: 10Esanders)
[14:27:56] <grrrit-wm>	 (03CR) 10Esanders: "So does this mean a link goes from inline to inline-block when you cursor into it? If so that would potentially change the layout of the p" [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/281456 (https://phabricator.wikimedia.org/T126487) (owner: 10DLynch)
[14:29:28] <grrrit-wm>	 (03PS7) 10Jforrester: ve.dm.MetaItem: Add removable flag, split from internal state [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/205291 (https://phabricator.wikimedia.org/T96528) (owner: 10Esanders)
[14:30:12] <James_F>	 edsanders: Just to check that my change here https://gerrit.wikimedia.org/r/#/c/205291/6..7/tests/dm/ve.dm.MetaList.test.js which makes it pass is OK from your perspective, rather than a sign that the tests aren't quite right?
[14:30:19] <grrrit-wm>	 (03PS8) 10Jforrester: ve.dm.MetaItem: Add removable flag, split from internal state [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/205291 (https://phabricator.wikimedia.org/T96528) (owner: 10Esanders)
[14:30:31] <grrrit-wm>	 (03PS3) 10Jforrester: [WIP] Failing test case for removable meta items [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/206846 (owner: 10Catrope)
[14:30:47] * James_F breakfasts.
[14:32:06] <edsanders>	 Roan seemed to think there was a genuine bug in there somewhere
[14:38:25] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] [WIP] Failing test case for removable meta items [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/206846 (owner: 10Catrope)
[14:40:17] <James_F>	 edsanders: Well, his test case doesn't pass. :-)
[14:42:39] <grrrit-wm>	 (03CR) 10Jforrester: [C: 031] ve.dm.MetaItem: Add removable flag, split from internal state [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/205291 (https://phabricator.wikimedia.org/T96528) (owner: 10Esanders)
[14:54:16] <grrrit-wm>	 (03CR) 10DLynch: "Yeah, it does if the active-link crosses lines. I'll see if I can find another way that lets the cursor escape the link without having to " [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/281456 (https://phabricator.wikimedia.org/T126487) (owner: 10DLynch)
[15:16:57] <wikibugs>	 10VisualEditor, 10Wikimedia-Site-requests, 5WMF-deploy-2016-04-12_(1.27.0-wmf.21): Please enable VisualEditor in htwiki's "Wikipedya:" (=Project:) namespace - https://phabricator.wikimedia.org/T130177#2195103 (10Jdforrester-WMF) 5Open>3Resolved
[15:37:05] <wikibugs>	 10VisualEditor, 10WikiEditor, 6Editing-Department, 6Research-and-Data-Backlog, 13Patch-For-Review: Add support for hashtags (URL parameter -> edit summary) in VisualEditor, WikiEditor, and ?? other editors - https://phabricator.wikimedia.org/T123529#2195172 (10Sadads) @Ricordisamoa Can I ask why you gave...
[16:16:50] <wikibugs>	 10Cite, 7Easy, 7Technical-Debt: <references/> list item must not wrap the text in <span> - https://phabricator.wikimedia.org/T49544#2195385 (10Izno)
[16:17:45] <wikibugs>	 10Cite, 7Easy, 7Technical-Debt: <references/> list item must not wrap the text in <span> - https://phabricator.wikimedia.org/T49544#511015 (10Izno)
[16:17:47] <wikibugs>	 10Cite, 7Easy, 13Patch-For-Review: Cite allows block content inside a span - https://phabricator.wikimedia.org/T128718#2195397 (10Izno)
[16:25:06] <wikibugs>	 10Cite: Allow readers to mark a sentence/selection as needing a citation - https://phabricator.wikimedia.org/T89578#1039697 (10Izno) {T20231} seems related.
[17:26:13] <wikibugs>	 10Cite, 10MobileFrontend, 3Reading-Web-Sprint-71-m: Create Special:Citations fallback for non-JavaScript/Resourceloader unsupported users - https://phabricator.wikimedia.org/T125897#2195839 (10dr0ptp4kt)
[19:32:47] <James_F>	 RoanKattouw: Can I beg you to explain on https://gerrit.wikimedia.org/r/#/c/205291/ why it shouldn't be merged (and why your follow-up failing test is failing)?
[19:38:37] <aharoni>	 Hi
[19:38:41] <wikibugs>	 10Citoid, 10VisualEditor, 10Source-Metadata, 10Wikidata: [Story] Support for citoid in Wikidata - https://phabricator.wikimedia.org/T131661#2196519 (10DarTar)
[21:08:41] <wikibugs>	 10VisualEditor, 06Editing-Analysis: Generate Jan–Mar 2016 Editing metrics snapshot - https://phabricator.wikimedia.org/T127248#2196874 (10Neil_P._Quinn_WMF) p:05High>03Triage
[22:12:30] <wikibugs>	 10Cite, 06Editing-Department, 07Browser-Support-Google-Chrome, 07Design: The characters in [3] (inline footnote) linebreaks - https://phabricator.wikimedia.org/T132255#2197163 (10matmarex) Looks like a browser bug to me. I can't reproduce this using any other browser (checked Firefox, Opera 12, IE 11). The...
[22:15:54] <wikibugs>	 10Cite, 07Easy, 07Technical-Debt: <references/> list item must not wrap the text in <span> - https://phabricator.wikimedia.org/T49544#2197173 (10Danny_B) 05duplicate>03Open
[22:17:49] <wikibugs>	 10Cite, 07Easy, 13Patch-For-Review: Cite allows block content inside a span - https://phabricator.wikimedia.org/T128718#2197181 (10Danny_B)
[22:17:51] <wikibugs>	 10Cite, 07Easy, 07Technical-Debt: <references/> list item must not wrap the text in <span> - https://phabricator.wikimedia.org/T49544#511015 (10Danny_B)
[22:20:20] <wikibugs>	 10Cite, 07Easy, 13Patch-For-Review: Cite allows block content inside a span - https://phabricator.wikimedia.org/T128718#2197256 (10Danny_B) 05duplicate>03Resolved Duplicating in opposite way in favour of the older task which has longer history and other dupes attached.
[22:27:14] <wikibugs>	 10VisualEditor: VisualEditor saves section as page text (deleting all other sections) - https://phabricator.wikimedia.org/T132309#2194062 (10AlexMonk-WMF) Sounds to me like you log into an account set to either: * Remember last editor, which was VE * Always use VE And then load https://www.mediawiki.org/w/index....
[22:29:03] <wikibugs>	 10VisualEditor, 10Beta-Cluster-Infrastructure: [Regression] Search for pages/templates in VE is not working in Beta cluster - https://phabricator.wikimedia.org/T132408#2197341 (10Ryasmeen)
[22:31:25] <wikibugs>	 10VisualEditor, 10Beta-Cluster-Infrastructure: [Regression pre-wmf.21] Search for pages/templates in VE is not working in Beta cluster  - https://phabricator.wikimedia.org/T132408#2197356 (10Ryasmeen)
[22:35:42] <grrrit-wm>	 (03PS1) 10Alex Monk: Check wpSection before converting textbox contents for use in VE [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/282821 (https://phabricator.wikimedia.org/T132309) 
[22:37:29] <grrrit-wm>	 (03CR) 10Bartosz Dziewoński: "(When are we getting real section editing in VE? :( )" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/282821 (https://phabricator.wikimedia.org/T132309) (owner: 10Alex Monk)
[22:38:29] <wikibugs>	 10VisualEditor, 13Patch-For-Review: VisualEditor saves section as page text (deleting all other sections) - https://phabricator.wikimedia.org/T132309#2197365 (10Legoktm) Yes, my preference is set to "Always use VE".
[22:39:12] <wikibugs>	 10VisualEditor, 13Patch-For-Review: VisualEditor saves section as page text (deleting all other sections) - https://phabricator.wikimedia.org/T132309#2197367 (10Jdforrester-WMF) p:05Triage>03Unbreak! Ick.
[22:39:26] <grrrit-wm>	 (03CR) 10jenkins-bot: [V: 04-1] Check wpSection before converting textbox contents for use in VE [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/282821 (https://phabricator.wikimedia.org/T132309) (owner: 10Alex Monk)
[22:39:28] <grrrit-wm>	 (03CR) 10Jforrester: [C: 032] Check wpSection before converting textbox contents for use in VE [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/282821 (https://phabricator.wikimedia.org/T132309) (owner: 10Alex Monk)
[22:39:52] <wikibugs>	 10VisualEditor, 13Patch-For-Review: VisualEditor saves section as page text (deleting all other sections) - https://phabricator.wikimedia.org/T132309#2197369 (10AlexMonk-WMF) I'm not quite sure why the server sends you via the wikitext editor for this yet, but the automatic switch takes place without noticing...
[22:40:22] <grrrit-wm>	 (03PS1) 10Jforrester: Check wpSection before converting textbox contents for use in VE [extensions/VisualEditor] (wmf/1.27.0-wmf.20) - 10https://gerrit.wikimedia.org/r/282822 (https://phabricator.wikimedia.org/T132309) 
[22:40:30] <grrrit-wm>	 (03CR) 10Jforrester: [C: 032] Check wpSection before converting textbox contents for use in VE [extensions/VisualEditor] (wmf/1.27.0-wmf.20) - 10https://gerrit.wikimedia.org/r/282822 (https://phabricator.wikimedia.org/T132309) (owner: 10Jforrester)
[22:41:29] <James_F>	 Krenair: Presumably returnTo is set to action=edit?
[22:42:02] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki, 13Patch-For-Review: VisualEditor saves section as page text (deleting all other sections) - https://phabricator.wikimedia.org/T132309#2197375 (10Jdforrester-WMF)
[22:42:21] <Krenair>	 James_F, yes but we have server-side code to suppress the wikitext editor on action=edit depending on factors such as your preferences
[22:42:33] <James_F>	 Hmm. Yeah.
[22:42:40] <James_F>	 Unsuccessfully, it appears.
[22:42:46] <James_F>	 Oh, wait.
[22:42:50] <Krenair>	 https://phabricator.wikimedia.org/T130165 looks like it might be a dupe
[22:42:50] <James_F>	 You're now logged in.
[22:42:56] <James_F>	 But previously you were anon.
[22:43:04] <James_F>	 Do we read the preference or the cookie?
[22:43:22] <James_F>	 Or, in this case, do we read both?
[22:43:39] <Krenair>	 Cookie.
[22:43:57] <Krenair>	 Sometimes.
[22:44:01] <James_F>	 Probably.
[22:44:02] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki, 13Patch-For-Review: VisualEditor saves section as page text (deleting all other sections) - https://phabricator.wikimedia.org/T132309#2194062 (10Jdforrester-WMF)
[22:44:46] <Krenair>	 Cookie, unless you're logged in without the cookie being set, or you're logged in with an invalid cookie
[22:45:12] <Krenair>	 no, that's not right
[22:45:37] <Krenair>	 Cookie, unless you're logged in, or you have no cookie, or the cookie is invalid - then pref
[22:45:52] <Krenair>	 getLastEditor in DesktopArticleTarget.init
[22:46:24] <James_F>	 But server-side what do we do?
[22:46:55] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki, 13Patch-For-Review: VisualEditor saves section as page text (deleting all other sections) - https://phabricator.wikimedia.org/T132309#2197394 (10AlexMonk-WMF) a:03AlexMonk-WMF
[22:47:04] <grrrit-wm>	 (03CR) 10Alex Monk: "Bartosz: That's T117716" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/282821 (https://phabricator.wikimedia.org/T132309) (owner: 10Alex Monk)
[22:48:58] <wikibugs>	 10VisualEditor: Section edit lead to deleting the rest of the page - https://phabricator.wikimedia.org/T130165#2128022 (10AlexMonk-WMF) What was your editing tab preference? Between clicking the edit link and being presented with VE, did the page change to action=edit or veaction=edit?
[22:49:22] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki: Section edit lead to deleting the rest of the page - https://phabricator.wikimedia.org/T130165#2197400 (10AlexMonk-WMF)
[22:50:30] <grrrit-wm>	 (03Merged) 10jenkins-bot: Check wpSection before converting textbox contents for use in VE [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/282821 (https://phabricator.wikimedia.org/T132309) (owner: 10Alex Monk)
[22:51:28] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki, 10MediaWiki-Page-editing: Expose WikiPage.php's replaceSectionAtRev function through the API - https://phabricator.wikimedia.org/T117715#1782119 (10matmarex) Hmm… action=edit&section=N? :D
[22:51:43] <Krenair>	 James_F, something different
[22:56:25] <grrrit-wm>	 (03Merged) 10jenkins-bot: Check wpSection before converting textbox contents for use in VE [extensions/VisualEditor] (wmf/1.27.0-wmf.20) - 10https://gerrit.wikimedia.org/r/282822 (https://phabricator.wikimedia.org/T132309) (owner: 10Jforrester)
[22:57:29] <wikibugs>	 10Cite, 07Easy, 13Patch-For-Review: Cite allows block content inside a span - https://phabricator.wikimedia.org/T128718#2083733 (10Izno) >>! In T128718#2197256, @Danny_B wrote: > Duplicating in opposite way in favour of the older task which has longer history and other dupes attached.  Someone has a patch fo...
[22:59:53] <James_F>	 Krenair: Could you SWAT https://gerrit.wikimedia.org/r/#/c/282827/ ? I've a meeting…
[22:59:59] <Krenair>	 yes
[23:00:04] <James_F>	 Ta.
[23:02:45] <wikibugs>	 10Cite, 07Easy, 13Patch-For-Review: Cite allows block content inside a span - https://phabricator.wikimedia.org/T128718#2197443 (10Danny_B) Commit message can be changed to reflect the original task number...
[23:06:41] <grrrit-wm>	 (03PS1) 10Alex Monk: Try to make client and server code to get last editor work the same [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/282828 
[23:44:10] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki, 05WMF-deploy-2016-04-05_(1.27.0-wmf.20), 05WMF-deploy-2016-04-12_(1.27.0-wmf.21): VisualEditor saves section as page text (deleting all other sections) - https://phabricator.wikimedia.org/T132309#2197616 (10Jdforrester-WMF) 05Open>03Resolved
[23:54:06] <wikibugs>	 10VisualEditor, 10VisualEditor-MediaWiki, 05WMF-deploy-2016-04-12_(1.27.0-wmf.21): Treat RB 302s (used for File: pages from foreign file repos) as 404s - https://phabricator.wikimedia.org/T130757#2197659 (10Jdforrester-WMF)