[00:17:04] 10VisualEditor, 10VisualEditor-MediaWiki-Mobile: [Regression wmf.23] Viewport sometimes jumps to the top abruptly on mobile VE - https://phabricator.wikimedia.org/T219200 (10ppelberg) ###Meta @matmarex: what does "Normal" priority mean to you? Asked differently: when you receive an email that indicates a task'... [00:26:50] (03PS2) 10Aaron Schulz: Clean up serialization cache code a bit and add more metrics [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499570 [12:29:40] (03CR) 10Esanders: [C: 03+2] ve.init.mw.MobileArticleTarget: Improve iOS Safari by moving 'padding-top' for toolbar [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499590 (https://phabricator.wikimedia.org/T219066) (owner: 10Bartosz Dziewoński) [12:43:38] (03Merged) 10jenkins-bot: ve.init.mw.MobileArticleTarget: Improve iOS Safari by moving 'padding-top' for toolbar [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499590 (https://phabricator.wikimedia.org/T219066) (owner: 10Bartosz Dziewoński) [12:45:28] (03CR) 10jenkins-bot: ve.init.mw.MobileArticleTarget: Improve iOS Safari by moving 'padding-top' for toolbar [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499590 (https://phabricator.wikimedia.org/T219066) (owner: 10Bartosz Dziewoński) [13:43:45] 10VisualEditor-MediaWiki-Mobile, 10Editing QA, 10MW-1.33-notes (1.33.0-wmf.24; 2019-04-02), 10Patch-For-Review, 10VisualEditor (Current work): Cursor pushed off screen when focussing first line in iOS - https://phabricator.wikimedia.org/T219066 (10Esanders) [14:07:55] 10VisualEditor, 10OTRS: Overlap between save changes and log out icons on visual editor in OTRSwiki - https://phabricator.wikimedia.org/T219506 (10alanajjar) [14:26:11] 10VisualEditor: Feature: Recovery of edits after a crash - https://phabricator.wikimedia.org/T214543 (10ppelberg) Losing in-process edits (or any kind of work for that matter!) is frustrating and something we're actively thinking about [1] – thank you for raising this, @jamespharvey20. As to the particular issu... [14:27:43] 10VisualEditor-MediaWiki, 10Patch-For-Review, 10VisualEditor (Current work): Add a useful call-to-action in the context for citation needed templates - https://phabricator.wikimedia.org/T211243 (10Esanders) [14:35:01] 10VisualEditor, 10VisualEditor-MediaWiki-Mobile: Table contexts misplaced on first focus - https://phabricator.wikimedia.org/T219508 (10Esanders) [14:35:48] 10VisualEditor-MediaWiki-Mobile, 10Editing QA, 10MW-1.33-notes (1.33.0-wmf.24; 2019-04-02), 10Patch-For-Review, 10VisualEditor (Current work): Cursor pushed off screen when focussing first line in iOS - https://phabricator.wikimedia.org/T219066 (10Esanders) Fixes the described issue, but caused T219508. [14:36:23] 10VisualEditor, 10VisualEditor-MediaWiki-Mobile: Table contexts misplaced on first focus - https://phabricator.wikimedia.org/T219508 (10Esanders) We probably just need to emit a `position` event when the padding is changed? [14:38:54] 10VisualEditor, 10VisualEditor-MediaWiki-Mobile: [Pre-deployment bug] Long pressing on a text to select it while the keyboard is open makes the browser unresponsive - https://phabricator.wikimedia.org/T219034 (10JTannerWMF) Okay thanks @Ryasmeen, should we move this to the freezer then @Ryasmeen @ppelberg @mat... [14:39:57] 10VisualEditor, 10ContentTranslation: Small visual glitch when keyboard shortcut for link inspector is pressed in VE's read-only mode - https://phabricator.wikimedia.org/T219246 (10JTannerWMF) = [14:41:36] (03PS1) 10Esanders: Emit 'position' event on surface view after changing doc padding [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499783 (https://phabricator.wikimedia.org/T219508) [14:41:45] 10VisualEditor, 10VisualEditor-MediaWiki-Mobile, 10Patch-For-Review: Table contexts misplaced on first focus - https://phabricator.wikimedia.org/T219508 (10Esanders) a:03Esanders [14:42:14] 10VisualEditor-MediaWiki-Mobile, 10Patch-For-Review, 10VisualEditor (Current work): Table contexts misplaced on first focus - https://phabricator.wikimedia.org/T219508 (10Esanders) [14:53:32] 10VisualEditor: Mobile VisualEditor should always show a close button - https://phabricator.wikimedia.org/T218787 (10ppelberg) //...may trap the user until they either figure out how that tapping outside the editor shows a close button or close the browser tab. It's not even that clear that the article is in "ed... [15:03:17] 10Citoid, 10VisualEditor: Citoid should only usurp " 10Citoid: Rename 'citefromid' to Citoid - https://phabricator.wikimedia.org/T219512 (10Esanders) [15:51:17] (03PS1) 10Esanders: Expose citoid map and conditionally register citefromid command [extensions/Citoid] - 10https://gerrit.wikimedia.org/r/499799 (https://phabricator.wikimedia.org/T219510) [15:58:48] (03PS2) 10Esanders: Expose citoid map and conditionally register citefromid command [extensions/Citoid] - 10https://gerrit.wikimedia.org/r/499799 (https://phabricator.wikimedia.org/T219510) [16:18:22] 10VisualEditor-DataModel, 10VisualEditor (Current work): Loss of 'alt' texts when manipulating image presentation form - https://phabricator.wikimedia.org/T216285 (10DLynch) a:03DLynch [16:24:10] (03PS1) 10DLynch: MWInlineImageNode: don't discard alt text when serializing [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499805 (https://phabricator.wikimedia.org/T216285) [16:45:39] 10VisualEditor, 10MinervaNeue, 10Editing Design , 10Readers-Web-Backlog (Design): Differences between read mode and edit mode on mobile: inconsistent underlines under top-level heading - https://phabricator.wikimedia.org/T218436 (10iamjessklein) +1 to removing excess headings for at least the time being. T... [16:48:15] (03CR) 10Krinkle: [C: 03+2] Clean up serialization cache code a bit and add more metrics [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499570 (owner: 10Aaron Schulz) [17:49:53] (03Merged) 10jenkins-bot: Clean up serialization cache code a bit and add more metrics [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499570 (owner: 10Aaron Schulz) [17:51:56] (03CR) 10jenkins-bot: Clean up serialization cache code a bit and add more metrics [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499570 (owner: 10Aaron Schulz) [18:06:17] 10VisualEditor, 10MinervaNeue, 10Editing Design , 10Readers-Web-Backlog (Design): Differences between read mode and edit mode on mobile: inconsistent underlines under top-level heading - https://phabricator.wikimedia.org/T218436 (10alexhollender) [18:07:06] 10VisualEditor, 10MinervaNeue, 10Readers-Web-Backlog, 10Editing Design : Differences between read mode and edit mode on mobile: inconsistent underlines under top-level heading - https://phabricator.wikimedia.org/T218436 (10alexhollender) Thanks for your input @iamjessklein. I've added our decision to the d... [18:16:43] (03PS1) 10Aaron Schulz: Avoid keeping excess keys from storeInSerializationCache() in cache [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499846 (https://phabricator.wikimedia.org/T203786) [18:22:11] (03CR) 10jerkins-bot: [V: 04-1] Avoid keeping excess keys from storeInSerializationCache() in cache [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499846 (https://phabricator.wikimedia.org/T203786) (owner: 10Aaron Schulz) [18:35:16] 10VisualEditor-MediaWiki, 10MW-1.33-notes (1.33.0-wmf.20; 2019-03-05), 10Verified, 10VisualEditor (Current work): Allow selection of the page title in Visual Editor - https://phabricator.wikimedia.org/T214790 (10Esanders) Looks like those gadget links get erased by VE anyway, as we manually set the firstHe... [18:40:20] (03PS1) 10Esanders: Fix cursor on #firstHeading [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499850 (https://phabricator.wikimedia.org/T214790) [18:56:24] (03CR) 10Esanders: [C: 04-1] "Can we add a test? (ve.dm.mwExample?)" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499805 (https://phabricator.wikimedia.org/T216285) (owner: 10DLynch) [19:16:47] (03CR) 10Bartosz Dziewoński: [C: 03+2] Emit 'position' event on surface view after changing doc padding [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499783 (https://phabricator.wikimedia.org/T219508) (owner: 10Esanders) [19:18:02] (03PS1) 10Bartosz Dziewoński: ve.init.mw.MobileArticleTarget: Set 'padding-top' on the right node when section-editing [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499870 (https://phabricator.wikimedia.org/T219066) [19:18:13] (03PS2) 10Aaron Schulz: Avoid keeping excess keys from storeInSerializationCache() in cache [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499846 (https://phabricator.wikimedia.org/T203786) [19:40:30] (03Merged) 10jenkins-bot: Emit 'position' event on surface view after changing doc padding [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499783 (https://phabricator.wikimedia.org/T219508) (owner: 10Esanders) [19:47:32] (03CR) 10jenkins-bot: Emit 'position' event on surface view after changing doc padding [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499783 (https://phabricator.wikimedia.org/T219508) (owner: 10Esanders) [19:51:30] 10Cite, 10Citoid, 10VisualEditor, 10Technical-Debt: Migrate Wikimedia wikis to MediaWiki:cite-tool-definition.json rather than MediaWiki:visualeditor-cite-tool-definition.json and drop b/c - https://phabricator.wikimedia.org/T219551 (10Jdforrester-WMF) p:05Triage→03Lowest [19:55:47] 10Cite, 10Citoid, 10VisualEditor, 10Technical-Debt: Migrate Wikimedia wikis to MediaWiki:cite-tool-definition.json rather than MediaWiki:visualeditor-cite-tool-definition.json and drop b/c - https://phabricator.wikimedia.org/T219551 (10Jdforrester-WMF) [19:57:01] (03PS1) 10VolkerE: Fix link dialog target icons size and position [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/499881 [19:57:26] (03CR) 10Jforrester: [C: 03+1] Expose citoid map and conditionally register citefromid command (032 comments) [extensions/Citoid] - 10https://gerrit.wikimedia.org/r/499799 (https://phabricator.wikimedia.org/T219510) (owner: 10Esanders) [20:01:03] (03CR) 10Bartosz Dziewoński: [C: 04-1] "Looks okay" (031 comment) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499850 (https://phabricator.wikimedia.org/T214790) (owner: 10Esanders) [20:02:07] (03CR) 10VolkerE: "Before/After" [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/499881 (owner: 10VolkerE) [20:02:15] (03PS2) 10Bartosz Dziewoński: Fix cursor on #firstHeading [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499850 (https://phabricator.wikimedia.org/T214790) (owner: 10Esanders) [20:02:35] (03CR) 10Bartosz Dziewoński: [C: 03+2] "Tweaked code comments" [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499850 (https://phabricator.wikimedia.org/T214790) (owner: 10Esanders) [20:14:59] 10VisualEditor-MediaWiki-Mobile, 10Section Editing, 10Patch-For-Review, 10VisualEditor (Current work): Deploy mobile section editing to the next set of wikis (listed in the description) - https://phabricator.wikimedia.org/T218939 (10ppelberg) @JTannerWMF: for a patch and config change such as T218939, do... [20:18:37] (03Merged) 10jenkins-bot: Fix cursor on #firstHeading [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499850 (https://phabricator.wikimedia.org/T214790) (owner: 10Esanders) [20:25:17] 10VisualEditor-MediaWiki, 10MW-1.33-notes (1.33.0-wmf.20; 2019-03-05), 10Verified, 10VisualEditor (Current work): Allow selection of the page title in Visual Editor - https://phabricator.wikimedia.org/T214790 (10matmarex) (The main patch already went through QA) [20:28:32] (03PS1) 10Esanders: Fix appearance of collapsed deactivated selections [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/499888 [20:29:22] (03PS1) 10Esanders: ve.dm.Node: Document attributeChange event [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/499889 [20:35:45] 10VisualEditor-MediaWiki-Mobile, 10Section Editing, 10Patch-For-Review, 10VisualEditor (Current work): Deploy mobile section editing to the next set of wikis (listed in the description) - https://phabricator.wikimedia.org/T218939 (10Esanders) @ppelberg we would typical do a sanity check as part of the depl... [20:42:37] (03PS2) 10Esanders: ve.init.mw.MobileArticleTarget: Set 'padding-top' on the correct node when section-editing [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499870 (https://phabricator.wikimedia.org/T219066) (owner: 10Bartosz Dziewoński) [20:42:41] (03CR) 10Esanders: [C: 03+2] ve.init.mw.MobileArticleTarget: Set 'padding-top' on the correct node when section-editing [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499870 (https://phabricator.wikimedia.org/T219066) (owner: 10Bartosz Dziewoński) [20:50:48] (03CR) 10Bartosz Dziewoński: [C: 03+2] ve.dm.Node: Document attributeChange event [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/499889 (owner: 10Esanders) [20:51:08] 10VisualEditor-MediaWiki-Mobile, 10Verified, 10Verified-in-Phase0, 10Verified-in-Phase2, 10VisualEditor (Current work): Selection lags behind scroll on iPhone - https://phabricator.wikimedia.org/T217798 (10Ryasmeen) [20:52:29] (03Merged) 10jenkins-bot: ve.dm.Node: Document attributeChange event [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/499889 (owner: 10Esanders) [20:54:34] 10VisualEditor, 10VisualEditor-MediaWiki-Mobile: [Pre-deployment bug] Long pressing on a text to select it while the keyboard is open makes the browser unresponsive - https://phabricator.wikimedia.org/T219034 (10Ryasmeen) Yeah I am okay with that @JTannerWMF, specially since it's not blocking any workflow. [20:54:46] (03CR) 10Bartosz Dziewoński: [C: 03+2] Fix appearance of collapsed deactivated selections [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/499888 (owner: 10Esanders) [20:55:01] (03Merged) 10jenkins-bot: ve.init.mw.MobileArticleTarget: Set 'padding-top' on the correct node when section-editing [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499870 (https://phabricator.wikimedia.org/T219066) (owner: 10Bartosz Dziewoński) [20:55:24] 10VisualEditor, 10VisualEditor-MediaWiki-Mobile: [Pre-deployment bug] Long pressing on a text to select it while the keyboard is open makes the browser unresponsive for a while - https://phabricator.wikimedia.org/T219034 (10Ryasmeen) [20:55:52] (03CR) 10jenkins-bot: ve.dm.Node: Document attributeChange event [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/499889 (owner: 10Esanders) [20:56:25] (03Merged) 10jenkins-bot: Fix appearance of collapsed deactivated selections [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/499888 (owner: 10Esanders) [20:57:46] (03CR) 10jenkins-bot: ve.init.mw.MobileArticleTarget: Set 'padding-top' on the correct node when section-editing [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499870 (https://phabricator.wikimedia.org/T219066) (owner: 10Bartosz Dziewoński) [20:57:54] (03CR) 10jenkins-bot: Fix appearance of collapsed deactivated selections [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/499888 (owner: 10Esanders) [21:02:36] 10VisualEditor-MediaWiki, 10VisualEditor (Current work): VE doesn't let you enable it in namespaces whose labels happen to be ints - https://phabricator.wikimedia.org/T219562 (10Jdforrester-WMF) p:05Triage→03Normal [21:07:42] 10VisualEditor-MediaWiki-Mobile, 10Section Editing, 10Patch-For-Review, 10VisualEditor (Current work): Deploy mobile section editing to the next set of wikis (listed in the description) - https://phabricator.wikimedia.org/T218939 (10ppelberg) >>! In T218939#5067587, @Esanders wrote: > @ppelberg we would ty... [21:16:47] 10VisualEditor: [EPIC: Focus] Isolate Section Editing - https://phabricator.wikimedia.org/T209955 (10ppelberg) [21:16:50] (03CR) 10Bartosz Dziewoński: [C: 03+2] Expose citoid map and conditionally register citefromid command [extensions/Citoid] - 10https://gerrit.wikimedia.org/r/499799 (https://phabricator.wikimedia.org/T219510) (owner: 10Esanders) [21:18:09] 10VisualEditor-MediaWiki, 10VisualEditor (Current work): VE doesn't let you enable it in namespaces whose labels happen to be ints - https://phabricator.wikimedia.org/T219562 (10Jdforrester-WMF) a:05matmarex→03None [21:19:21] 10VisualEditor-MediaWiki, 10VisualEditor (Current work): VE doesn't let you enable it in namespaces whose labels happen to be ints - https://phabricator.wikimedia.org/T219562 (10Jdforrester-WMF) Patch works around this bug, it doesn't address it. [21:19:32] 10Citoid, 10MW-1.33-notes (1.33.0-wmf.24; 2019-04-02), 10Patch-For-Review, 10VisualEditor (Current work): Citoid should only usurp " 10VisualEditor-MediaWiki, 10VisualEditor (Current work): VE doesn't let you enable it in namespaces whose labels happen to be ints - https://phabricator.wikimedia.org/T219562 (10matmarex) a:03matmarex [21:28:30] 10VisualEditor-MediaWiki, 10VisualEditor (Current work): VE doesn't let you enable it in namespaces whose labels happen to be ints - https://phabricator.wikimedia.org/T219562 (10Esanders) This is because of a limitation of PHP, from https://wiki.php.net/rfc/convert_numeric_keys_in_object_array_casts : > The fi... [21:29:46] 10VisualEditor-MediaWiki, 10VisualEditor (Current work): VE doesn't let you enable it in namespaces whose labels happen to be ints - https://phabricator.wikimedia.org/T219562 (10matmarex) We mostly refer to namespaces by ID in config options, $wgVisualEditorAvailableNamespaces is just weird. We can easily make... [21:30:22] 10VisualEditor-MediaWiki, 10MobileFrontend, 10Editing QA, 10MW-1.33-notes (1.33.0-wmf.23; 2019-03-26), and 2 others: Editing toolbar on mobile VE is not sticky (can scroll out of view) on iOS - https://phabricator.wikimedia.org/T218414 (10Ryasmeen) @JTannerWMF: Now it can go to Product Owner Review :) It j... [21:30:47] 10VisualEditor-MediaWiki, 10MobileFrontend, 10MW-1.33-notes (1.33.0-wmf.23; 2019-03-26), 10Verified, and 3 others: Editing toolbar on mobile VE is not sticky (can scroll out of view) on iOS - https://phabricator.wikimedia.org/T218414 (10Ryasmeen) [21:57:47] 10VisualEditor-MediaWiki-Mobile, 10Section Editing, 10VisualEditor (Current work): Deploy mobile section editing to the next set of wikis (listed in the description) - https://phabricator.wikimedia.org/T218939 (10Jdforrester-WMF) [22:20:35] (03PS1) 10Jforrester: Update VE core submodule to master (132022a44) [extensions/VisualEditor] - 10https://gerrit.wikimedia.org/r/499934 [22:23:54] 10VisualEditor-MediaWiki, 10MobileFrontend, 10MW-1.33-notes (1.33.0-wmf.23; 2019-03-26), 10User-Ryasmeen, and 4 others: Editing toolbar on mobile VE is not sticky (can scroll out of view) on iOS - https://phabricator.wikimedia.org/T218414 (10ppelberg) 05Open→03Resolved [22:27:05] (03CR) 10Bartosz Dziewoński: [C: 03+2] Expose citoid map and conditionally register citefromid command [extensions/Citoid] - 10https://gerrit.wikimedia.org/r/499799 (https://phabricator.wikimedia.org/T219510) (owner: 10Esanders) [22:46:05] (03Merged) 10jenkins-bot: Expose citoid map and conditionally register citefromid command [extensions/Citoid] - 10https://gerrit.wikimedia.org/r/499799 (https://phabricator.wikimedia.org/T219510) (owner: 10Esanders) [22:53:37] 10VisualEditor-MediaWiki, 10MW-1.33-notes (1.33.0-wmf.14; 2019-01-22), 10Verified, 10VisualEditor (Current work): Education dots rendering issues - https://phabricator.wikimedia.org/T214203 (10ppelberg) Hmm. In following the steps below, I //seem// to still be encountering, "...circles [that] have very sli... [22:54:49] 10VisualEditor, 10Operations, 10Traffic, 10Wikimedia-Apache-configuration, 10User-Ryasmeen: Visual Editor gets stuck opening article (net::ERR_SPDY_PROTOCOL_ERROR 200/Loading failed for the