[00:09:52] 10VisualEditor, 10MinervaNeue (Tracking), 10Readers-Web-Backlog (Tracking): [betalabs] Cannot switch to VE in MinervaNeue - https://phabricator.wikimedia.org/T186149 (10matmarex) >>! In T186149#3938329, @Esanders wrote: > The edit links in Minerva always load the MF editor for me, so seems like there are mor... [01:14:45] 10VisualEditor, 10VisualEditor-VisualDiffs, 10MW-1.35-notes (1.35.0-wmf.14; 2020-01-07), 10MobileFrontend (MobileFrontend Special Pages), and 3 others: Regression: issues with MobileDiff - https://phabricator.wikimedia.org/T242310 (10Jdlrobson) @nray @phuedx to test this you need to enable wikidiff2. Right... [09:07:15] 10Cite, 10WMDE-QWERTY-Team, 10Technical-Debt, 10User-thiemowmde: Rewrite LogicException using safer alternatives - https://phabricator.wikimedia.org/T243221 (10thiemowmde) This task is directly related to {T242434} and {T242437}. Also see {T240671}. [09:37:11] 10Cite, 10Book-Referencing, 10Patch-For-Review, 10Technical-Debt, and 2 others: Clean up implementation for "follow" cases - https://phabricator.wikimedia.org/T240858 (10JStrodt_WMDE) @thiemowmde Is there a screenshot of what the error message would look like? That would be handy for the announcements. Tha... [09:56:56] 10Cite, 10Book-Referencing, 10WMDE-Design, 10WMDE-QWERTY-Team, and 2 others: Full UX (and possibly PM) review of all new user-facing (error) messages - https://phabricator.wikimedia.org/T238188 (10thiemowmde) a:05Hanna_Petruschat_WMDE→03None [09:57:16] 10Cite, 10Book-Referencing, 10WMDE-Design, 10WMDE-QWERTY-Team, and 3 others: Full UX (and possibly PM) review of all new user-facing (error) messages - https://phabricator.wikimedia.org/T238188 (10thiemowmde) [11:31:51] 10Cite, 10Book-Referencing, 10WMDE-Design, 10WMDE-QWERTY-Team, and 3 others: Full UX (and possibly PM) review of all new user-facing (error) messages - https://phabricator.wikimedia.org/T238188 (10thiemowmde) [11:37:52] 10Cite, 10Book-Referencing, 10Patch-For-Review, 10Technical-Debt, and 2 others: Clean up implementation for "follow" cases - https://phabricator.wikimedia.org/T240858 (10thiemowmde) @JStrodt_WMDE: Here is a screenshot of the message that now appears in the text, at the position of the broken : {F31536... [11:54:59] 10Cite, 10Book-Referencing, 10WMDE-Design, 10WMDE-QWERTY-Team, and 3 others: Full UX (and possibly PM) review of all new user-facing (error) messages - https://phabricator.wikimedia.org/T238188 (10thiemowmde) [13:14:08] 10VisualEditor, 10VisualEditor-MediaWiki-References: Visual Editor should accept pre-formatted references via the Cite tool - https://phabricator.wikimedia.org/T243703 (10Aklapper) [13:16:22] 10Cite, 10Book-Referencing: Use category rather than page prop to mark Book Referencing pages - https://phabricator.wikimedia.org/T239989 (10thiemowmde) 05Open→03Declined Declined for now. Can be re-opened if user reports actually requesting this come in. [13:18:15] 10Cite, 10MW-1.35-notes (1.35.0-wmf.8; 2019-11-26): Review feature: Reference errors are split by user interface language - https://phabricator.wikimedia.org/T239988 (10thiemowmde) [13:20:13] 10Cite, 10Book-Referencing, 10Patch-For-Review, 10Technical-Debt, 10User-thiemowmde: Remove almost all uses of recursiveTagParse from Cite - https://phabricator.wikimedia.org/T239572 (10thiemowmde) [13:20:33] 10Cite, 10Book-Referencing, 10WMDE-QWERTY-Team, 10MW-1.35-notes (1.35.0-wmf.8; 2019-11-26), and 2 others: Remove (slow) Cite parser tests that aren't needed any more - https://phabricator.wikimedia.org/T238187 (10thiemowmde) [13:22:59] 10Cite, 10Book-Referencing, 10Epic: Don't break Cite VisualEditor integration - https://phabricator.wikimedia.org/T240186 (10thiemowmde) 05Open→03Resolved a:03thiemowmde We know #visualeditor doesn't break in any way, and does not cause any data loss. Making #visualeditor more compatible with #book-ref... [13:57:12] 10Cite, 10Wikisource, 10Patch-For-Review, 10Technical-Debt, 10good first task: list item must not wrap the text in - https://phabricator.wikimedia.org/T49544 (10Xover) >>! In T49544#5834946, @thiemowmde wrote: > I was very specifically asking for an example of bad rendering in a brow... [14:15:10] 10Cite, 10Wikisource, 10Patch-For-Review, 10Technical-Debt, 10good first task: list item must not wrap the text in - https://phabricator.wikimedia.org/T49544 (10ssastry) First off, the output of both Parsoid as well as Parser.php are both not HTML5 compliant. https://www.mediawiki.or... [14:32:57] 10Cite, 10Wikisource, 10Patch-For-Review, 10Technical-Debt: list item must not wrap the text in - https://phabricator.wikimedia.org/T49544 (10thiemowmde) [15:03:04] 10Cite, 10Book-Referencing, 10WMDE-Design, 10WMDE-QWERTY-Team, and 3 others: Nicer error handling for broken Cite references - https://phabricator.wikimedia.org/T238061 (10thiemowmde) From yesterdays story time about this ticket: * We had a look at all the different ways #Cite is reporting errors. There ar... [16:04:20] 10Cite, 10Book-Referencing, 10Patch-For-Review, 10Technical-Debt, and 2 others: Clean up implementation for "follow" cases - https://phabricator.wikimedia.org/T240858 (10Johan) This was published in https://meta.wikimedia.org/wiki/Tech/News/2020/03 by the way, if that wasn't obvious from the moving the tic... [16:04:26] 10VisualEditor, 10LibUp, 10OOjs, 10Utilities-UnicodeJS: Upgrade karma and related packages to latest version - https://phabricator.wikimedia.org/T243791 (10Jdforrester-WMF) We've been intentionally holding these back for consistency with MediaWiki's version. [16:05:14] 10VisualEditor, 10LibUp, 10OOjs, 10Utilities-UnicodeJS: Upgrade karma and related packages to latest version in libraries, once MediaWiki upgrades - https://phabricator.wikimedia.org/T243791 (10Jdforrester-WMF) [16:05:27] 10VisualEditor, 10LibUp, 10OOjs, 10Utilities-UnicodeJS: Upgrade karma and related packages to latest version in libraries, once MediaWiki upgrades - https://phabricator.wikimedia.org/T243791 (10Jdforrester-WMF) [17:13:14] 10VisualEditor: Disable displaytitle modification in Visual Editor - https://phabricator.wikimedia.org/T243850 (10Bdijkstra) [17:38:43] 10VisualEditor, 10LibUp, 10OOjs, 10Utilities-UnicodeJS: Upgrade karma and related packages to latest version in libraries, once MediaWiki upgrades - https://phabricator.wikimedia.org/T243791 (10Krinkle) Hm.. aside from VisualEditor I don't think we have any executable test code that runs both in and outsid... [18:00:39] 10VisualEditor, 10VisualEditor-VisualDiffs, 10MW-1.35-notes (1.35.0-wmf.14; 2020-01-07), 10MobileFrontend (MobileFrontend Special Pages), and 3 others: Regression: issues with MobileDiff - https://phabricator.wikimedia.org/T242310 (10phuedx) @Jdlrobson: Can this be tested on the beta cluster? [18:11:00] 10VisualEditor, 10VisualEditor-ContentEditable: Switching to a new Bullet point to Numbered list corrupts user input in VisualEditor - https://phabricator.wikimedia.org/T243852 (10Krinkle) [18:17:58] 10VisualEditor, 10VisualEditor-ContentEditable: Switching a new Bullet point to Numbered list corrupts user input in VisualEditor - https://phabricator.wikimedia.org/T243852 (10Krinkle) [18:39:58] 10Cite, 10Wikisource, 10Patch-For-Review, 10Technical-Debt: list item must not wrap the text in - https://phabricator.wikimedia.org/T49544 (10Aklapper) @Izno: Hi, please take a look at [our etiquette](https://www.mediawiki.org/wiki/Bug_management/Phabricator_etiquette) and keep convers... [18:45:34] 10Cite, 10Wikisource, 10Patch-For-Review, 10Technical-Debt: list item must not wrap the text in - https://phabricator.wikimedia.org/T49544 (10MJL) >>! In T49544#5834946, @thiemowmde wrote: > There is a number of tools, gadgets, user script, and whatnot that work with references. If any... [18:46:55] 10Cite, 10Book-Referencing, 10WMDE-Design, 10WMDE-QWERTY-Team, and 3 others: Nicer error handling for broken Cite references - https://phabricator.wikimedia.org/T238061 (10Izno) #2 would be great since most errors at the end of the day do end up there today. [19:09:21] (03CR) 10Krinkle: Implement ParsoidFetchTemplateData hook for Parsoid/PHP (031 comment) [extensions/TemplateData] - 10https://gerrit.wikimedia.org/r/552620 (https://phabricator.wikimedia.org/T238954) (owner: 10Subramanya Sastry) [19:11:01] 10VisualEditor, 10VisualEditor-ContentEditable: Switching a new Bullet point to Numbered list corrupts user input in VisualEditor - https://phabricator.wikimedia.org/T243852 (10matmarex) [19:11:03] 10VisualEditor: Typed characters get misplaced after changing a bullet item to a numbered item or vice versa - https://phabricator.wikimedia.org/T179751 (10matmarex) [19:11:06] 10TemplateData, 10VisualEditor, 10MediaWiki-Parser, 10Parsoid, 10Technical-Debt: Evaluate how to structure internal calls to TemplateData in PHP - https://phabricator.wikimedia.org/T243854 (10Krinkle) [19:11:08] 10VisualEditor, 10VisualEditor-ContentEditable: Switching a new Bullet point to Numbered list corrupts user input in VisualEditor - https://phabricator.wikimedia.org/T243852 (10matmarex) I could swear I also filed a bug about this a couple years ago, but I can't find it for the life of me. [19:14:39] (03PS1) 10Krinkle: Minor clean up in new ParsoidFetchTemplateData hook [extensions/TemplateData] - 10https://gerrit.wikimedia.org/r/568011 [19:15:26] 10TemplateData, 10VisualEditor, 10MediaWiki-Parser, 10Parsoid, 10Technical-Debt: Evaluate how to structure internal calls to TemplateData in PHP - https://phabricator.wikimedia.org/T243854 (10Krinkle) [19:19:25] 10Cite, 10Wikisource, 10Patch-For-Review, 10Technical-Debt: list item must not wrap the text in - https://phabricator.wikimedia.org/T49544 (10Xover) >>! In T49544#5836366, @ssastry wrote: > Let us please not make changes like changing to
without thinking through the implic... [19:29:54] 10Cite, 10MediaWiki-extension-requests: Allow paragraphs in references (Cite.php) - https://phabricator.wikimedia.org/T243857 (10Jiou7) [19:58:52] (03CR) 10Bartosz Dziewoński: [C: 04-1] "Per my last comment" [extensions/WikiEditor] - 10https://gerrit.wikimedia.org/r/497073 (owner: 10Fomafix) [20:02:10] 10VisualEditor, 10VisualEditor-Tables: Vertical merging of table rows in Visual Editor may cause unwanted loss of a whole row - https://phabricator.wikimedia.org/T243753 (10matmarex) 05Open→03Invalid [20:07:16] 10VisualEditor, 10VisualEditor-ContentEditable: Clicking on a slug while the selection is inside a table causes an exception - https://phabricator.wikimedia.org/T243860 (10dchan) [20:22:44] 10VisualEditor, 10VisualEditor-ContentEditable: Clicking on a slug while the selection is inside a table causes an exception - https://phabricator.wikimedia.org/T243860 (10dchan) [20:23:44] 10VisualEditor, 10VisualEditor-ContentEditable: Clicking on a slug while the selection is inside a table causes an exception - https://phabricator.wikimedia.org/T243860 (10dchan) The table selection update is incorrect, because we're calling tx.translateRange with excludeInsertion=false, but there should alway... [20:28:41] (03PS1) 10Divec: Always use excludeInsertion when translating table selections [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/568042 (https://phabricator.wikimedia.org/T243860) [21:12:05] (03CR) 10Bartosz Dziewoński: [C: 03+2] Always use excludeInsertion when translating table selections [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/568042 (https://phabricator.wikimedia.org/T243860) (owner: 10Divec) [21:12:37] 10VisualEditor-ContentEditable, 10Patch-For-Review, 10VisualEditor (Current work): Clicking on a slug while the selection is inside a table causes an exception - https://phabricator.wikimedia.org/T243860 (10matmarex) [21:14:29] (03Merged) 10jenkins-bot: Always use excludeInsertion when translating table selections [VisualEditor/VisualEditor] - 10https://gerrit.wikimedia.org/r/568042 (https://phabricator.wikimedia.org/T243860) (owner: 10Divec) [21:55:25] 10Cite: Allow paragraphs in references (Cite.php) - https://phabricator.wikimedia.org/T243857 (10Aklapper) Hi @Jiou7, thanks for taking the time to report this and welcome to Wikimedia Phabricator! This request is not about Requests for new MediaWiki extensions, hence removing #mediawiki-extension-requests. [22:31:54] 10VisualEditor, 10VisualEditor-VisualDiffs, 10MW-1.35-notes (1.35.0-wmf.14; 2020-01-07), 10MobileFrontend (MobileFrontend Special Pages), and 2 others: Regression: issues with MobileDiff - https://phabricator.wikimedia.org/T242310 (10Jdlrobson) a:05Jdlrobson→03Edtadros [22:42:30] (03CR) 10C. Scott Ananian: [C: 03+2] Minor clean up in new ParsoidFetchTemplateData hook [extensions/TemplateData] - 10https://gerrit.wikimedia.org/r/568011 (owner: 10Krinkle) [23:06:04] (03Merged) 10jenkins-bot: Minor clean up in new ParsoidFetchTemplateData hook [extensions/TemplateData] - 10https://gerrit.wikimedia.org/r/568011 (owner: 10Krinkle) [23:08:12] 10TemplateData, 10VisualEditor, 10VisualEditor-MediaWiki: VisualEditor ignores autogenerated parameter info for templates without TemplateData - https://phabricator.wikimedia.org/T243868 (10TK-999) [23:10:32] 10TemplateData, 10VisualEditor, 10VisualEditor-MediaWiki: VisualEditor ignores autogenerated parameter info for templates without TemplateData - https://phabricator.wikimedia.org/T243868 (10TK-999) I think this is an un-intended side-effect of rEVED8fd621a. We could change the check on line 337 to only ignor...