Page MenuHomePhabricator

dchan (David Chan)
User

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Saturday

  • Clear sailing ahead.

User Details

User Since
Oct 23 2014, 10:14 AM (234 w, 8 h)
Availability
Available
IRC Nick
divec
LDAP User
Unknown
MediaWiki User
DChan (WMF) [ Global Accounts ]

Recent Activity

Thu, Apr 11

dchan added a comment to T219812: Editor wants to quickly move back to whole page editing.

My only immediate thought to add is a reactive corrector may need to make the same small correction across a few places. E.g. updating a statistic mentioned in an infobox and in free text. For consistency, you wouldn't want to commit that change in two stages.

Thu, Apr 11, 6:31 AM · VisualEditor, Editing Design

Thu, Apr 4

dchan added a comment to T217223: Backspacing multiple VE paragraphs in Gboard causes model corruption.

I am reasonably satisfied with Change 498528 now. I cannot find a case where this breaks input, having tested on multiple types of IME, on Chromium, Firefox and Safari, and on Android, iOS, Windows, Linux and MacOS. Of course, there are almost limitless combinations, so it's always possible there is some breakage I've not managed to find.

Thu, Apr 4, 11:04 AM · Patch-For-Review, VisualEditor (Current work), VisualEditor-ContentLanguage, VisualEditor-ContentEditable

Mar 18 2019

dchan added a parent task for T202719: Backspace across paragraphs is broken on mobile VE (Android Chrome): T218545: Cope with IMEs that make uncancellable alterations to DOM structural nodes.
Mar 18 2019, 9:19 AM · Browser-Support-Android-Google-Chrome, VisualEditor (Current work), Editing Q1 2018-19 mobile improvements, VisualEditor-MediaWiki-Mobile, VisualEditor-ContentEditable
dchan added a subtask for T218545: Cope with IMEs that make uncancellable alterations to DOM structural nodes: T202719: Backspace across paragraphs is broken on mobile VE (Android Chrome).
Mar 18 2019, 9:19 AM · VisualEditor-MediaWiki-Mobile, VisualEditor-ContentLanguage, VisualEditor-ContentEditable, VisualEditor
dchan created T218545: Cope with IMEs that make uncancellable alterations to DOM structural nodes.
Mar 18 2019, 9:18 AM · VisualEditor-MediaWiki-Mobile, VisualEditor-ContentLanguage, VisualEditor-ContentEditable, VisualEditor

Mar 1 2019

dchan added a comment to T215908: Section Editing: Pre-Deployment QA .

To see the real speed benefits of section editing, we may need to test long articles on entry-level devices. Here are recordings of a stark difference, editing United States on the (entry-level) Xiaomi Redmi 6A.

Mar 1 2019, 5:54 PM · User-Ryasmeen, VisualEditor-MediaWiki-Mobile, Section Editing, VisualEditor
dchan added a comment to T217223: Backspacing multiple VE paragraphs in Gboard causes model corruption.

Ok, with Gboard English, this is what happens when you select "de</h1><h1>fg" and press backspace:

  1. The selection collapses to the end
  2. A keydown event with an unknown keycode is fired
  3. The content is deleted (unifying the paragraphs)
Mar 1 2019, 5:12 AM · Patch-For-Review, VisualEditor (Current work), VisualEditor-ContentLanguage, VisualEditor-ContentEditable

Feb 27 2019

dchan created T217223: Backspacing multiple VE paragraphs in Gboard causes model corruption.
Feb 27 2019, 4:20 AM · Patch-For-Review, VisualEditor (Current work), VisualEditor-ContentLanguage, VisualEditor-ContentEditable

Feb 12 2019

dchan added a comment to T209987: [Engineering EPIC]: Isolate Section Editing.

See also T215717 (which is a related optimization task, but not a blocking subtask)

Feb 12 2019, 12:56 AM · VisualEditor-MediaWiki-Mobile, Section Editing, VisualEditor

Feb 10 2019

dchan created T215717: When section editing, do not build whole CE tree.
Feb 10 2019, 9:18 AM · VisualEditor-MediaWiki-Mobile, Section Editing, VisualEditor (Current work), VisualEditor-ContentEditable

Feb 9 2019

dchan added a comment to T209874: CX2: Unable to type diacritics in the translation.

Thanks, those "steps to reproduce" are impressively precise. I will investigate.

Feb 9 2019, 2:34 PM · VisualEditor, ContentTranslation

Feb 8 2019

dchan added a comment to T215567: Split TreeModifier into a tree diff generator and tree diff applier.

Oh, good point – yes, that is correct.

Feb 8 2019, 10:29 PM · VisualEditor-DataModel, VisualEditor

Feb 7 2019

dchan created T215567: Split TreeModifier into a tree diff generator and tree diff applier.
Feb 7 2019, 11:09 PM · VisualEditor-DataModel, VisualEditor

Jan 22 2019

dchan added a comment to T211236: After changing a link target, the link is selected such that typing will delete the link.

Let's go forward with your patch, because it's simple, neat and works fine for links, which in practice are by far the most significant case.

Jan 22 2019, 3:41 AM · Verified, VisualEditor-EditingTools, MW-1.33-notes (1.33.0-wmf.14; 2019-01-22), VisualEditor (Current work)

Jan 17 2019

dchan added a comment to T211236: After changing a link target, the link is selected such that typing will delete the link.

@DLynch : Do you mean when setting up a view selection or a model selection?

Jan 17 2019, 1:52 PM · Verified, VisualEditor-EditingTools, MW-1.33-notes (1.33.0-wmf.14; 2019-01-22), VisualEditor (Current work)

Jan 14 2019

dchan added a comment to T211236: After changing a link target, the link is selected such that typing will delete the link.

I think the current approach is about right for links, which have "nails" anchoring the annotation boundaries – I'll review the details of the patch later on.

Jan 14 2019, 11:17 PM · Verified, VisualEditor-EditingTools, MW-1.33-notes (1.33.0-wmf.14; 2019-01-22), VisualEditor (Current work)

Jan 11 2019

dchan added a comment to T209984: Isolate Section Editing: Determine New Workflows .

@Neil_P._Quinn_WMF : With our implementation of section editing (T76541), we're fetching, loading and editing against the full article's data model in any case, so in technical terms a "continue editing with full article" button would be feasible.

Jan 11 2019, 6:58 PM · User-Ryasmeen, VisualEditor

Dec 15 2018

dchan added subtasks for T149667: Amazing Article Annotations: T20231: provide a way to specify what text/statement is supported by a <ref> block., T89578: Allow readers to mark a sentence/selection as needing a citation.
Dec 15 2018, 3:35 AM · Parsing-Team, Cite, VisualEditor, ContentTranslation, MediaWiki-extensions-Translate, Wikispeech, Wikimedia-Developer-Summit (2017)
dchan added a parent task for T20231: provide a way to specify what text/statement is supported by a <ref> block.: T149667: Amazing Article Annotations.
Dec 15 2018, 3:35 AM · Cite
dchan added a parent task for T89578: Allow readers to mark a sentence/selection as needing a citation: T149667: Amazing Article Annotations.
Dec 15 2018, 3:35 AM · Cite

Nov 26 2018

dchan added a comment to T209874: CX2: Unable to type diacritics in the translation.

Hello, thanks for the detailed explanation.

Nov 26 2018, 8:58 PM · VisualEditor, ContentTranslation

Nov 20 2018

dchan added a comment to T209533: Research faking linear selections on mobile.

Still thinking about programmatic (a.k.a. "fake") selections, but stepping back from the idea that they should be as similar as possible to real ones, some sort of mechanism to select large amounts of content would be useful for large copy/paste tasks (see https://www.mediawiki.org/wiki/Topic:Ul1orwmwk2q32l1v for one such request)

Nov 20 2018, 6:01 PM · VisualEditor-MediaWiki-Mobile, VisualEditor-ContentEditable, VisualEditor

Nov 16 2018

dchan added a comment to T209648: Changing text in a TargetWidget does not enable dialog's "Apply" button (in media, gallery, reference dialog) until second keypress.

Oooh, thanks. Do we think the behaviour happens because of setTimeout subtlety? (That would instinctively be my first guess but I haven't checked).

Nov 16 2018, 2:25 AM · User-Ryasmeen, Verified, MW-1.33-notes (1.33.0-wmf.6; 2018-11-27), VisualEditor (Current work), Patch-For-Review, VisualEditor-EditingTools

Nov 14 2018

dchan created T209533: Research faking linear selections on mobile.
Nov 14 2018, 8:12 PM · VisualEditor-MediaWiki-Mobile, VisualEditor-ContentEditable, VisualEditor
dchan added a parent task for T202723: Mobile native selection context menu can obscure the VE toolbar: T209505: EPIC: Fix selection issues on mobile.
Nov 14 2018, 4:59 PM · MW-1.33-notes (1.33.0-wmf.22; 2019-03-19), Editing QA, VisualEditor (Current work), Editing Q1 2018-19 mobile improvements, VisualEditor-MediaWiki-Mobile, VisualEditor-ContentEditable
dchan added subtasks for T209505: EPIC: Fix selection issues on mobile: T136840: Selection dragging opens context menu on mobile, causing surface to move and breaking drag in Android Chrome, T204387: Can't drag a collapsed selection across a reference on mobile, T202723: Mobile native selection context menu can obscure the VE toolbar, T203555: On Android Chrome, Thai text selections next to a link can collapse unexpectedly, T204718: Mobile: Selection handles obscured after scrollCursorIntoView .
Nov 14 2018, 4:59 PM · VisualEditor-MediaWiki-Mobile, VisualEditor-ContentEditable, VisualEditor
dchan added a parent task for T136840: Selection dragging opens context menu on mobile, causing surface to move and breaking drag in Android Chrome: T209505: EPIC: Fix selection issues on mobile.
Nov 14 2018, 4:59 PM · Browser-Support-Android-Google-Chrome, VisualEditor-ContentEditable, VisualEditor-MediaWiki-Mobile, VisualEditor
dchan created T209505: EPIC: Fix selection issues on mobile.
Nov 14 2018, 4:51 PM · VisualEditor-MediaWiki-Mobile, VisualEditor-ContentEditable, VisualEditor

Oct 31 2018

dchan updated the task description for T208428: Decide mobile target hardware specification and decide performance acceptance criteria.
Oct 31 2018, 4:08 PM · Internet-Archive, VisualEditor-MediaWiki-Mobile, VisualEditor
dchan added a comment to T206228: Analyse load time performance of FA-length pages on mobile.

See also T208428 .

Oct 31 2018, 4:05 PM · Performance-Team (Radar), Performance-Team-notice, VisualEditor (Current work), VisualEditor-MediaWiki-Mobile, VisualEditor-MediaWiki, VisualEditor-Performance
dchan updated the task description for T208428: Decide mobile target hardware specification and decide performance acceptance criteria.
Oct 31 2018, 4:00 PM · Internet-Archive, VisualEditor-MediaWiki-Mobile, VisualEditor
dchan updated the task description for T208428: Decide mobile target hardware specification and decide performance acceptance criteria.
Oct 31 2018, 2:57 PM · Internet-Archive, VisualEditor-MediaWiki-Mobile, VisualEditor
dchan created T208428: Decide mobile target hardware specification and decide performance acceptance criteria.
Oct 31 2018, 2:56 PM · Internet-Archive, VisualEditor-MediaWiki-Mobile, VisualEditor

Oct 30 2018

dchan moved T201877: EPIC: What are the current best UI practices for mobile editing? from In progress to Code review on the VisualEditor (Current work) board.
Oct 30 2018, 5:04 PM · VisualEditor, Wikipedia-iOS-App-Backlog, Wikipedia-Android-App-Backlog, Design-Research, Design

Oct 29 2018

dchan created T208228: DM Selections should not cache document model data.
Oct 29 2018, 2:54 PM · Patch-For-Review, MW-1.33-notes (1.33.0-wmf.4; 2018-11-13), VisualEditor (Current work), VisualEditor-DataModel

Oct 24 2018

dchan created T207874: Find a good way round IME limitations, especially on mobile.
Oct 24 2018, 6:11 PM · VisualEditor-MediaWiki-Mobile, VisualEditor-ContentEditable, VisualEditor

Oct 10 2018

dchan added a comment to T204387: Can't drag a collapsed selection across a reference on mobile.

I filed an upstream bug about the dragging across ce=false: https://bugs.chromium.org/p/chromium/issues/detail?id=894003

Oct 10 2018, 10:53 AM · VisualEditor (Current work), VisualEditor-MediaWiki-Mobile

Oct 9 2018

dchan added a comment to T204387: Can't drag a collapsed selection across a reference on mobile.

Maybe we should experiment to see whether we can get anywhere using newer features now fairly reliably available on Chrome mobile, e.g. shadow DOM.

Oct 9 2018, 1:07 PM · VisualEditor (Current work), VisualEditor-MediaWiki-Mobile

Oct 4 2018

dchan added a comment to T202138: Snapshot: what proportion of edits are made using the visual and wikitext editors?.

Hi @Neil_P._Quinn_WMF , the configuration variables shown here should help:

Oct 4 2018, 12:07 AM · VisualEditor (Current work), Editing Q1 2018-19 mobile improvements, Product-Analytics

Sep 26 2018

dchan updated the task description for T205535: Remove dedicated annotation operation types.
Sep 26 2018, 1:29 PM · MW-1.33-notes (1.33.0-wmf.4; 2018-11-13), VisualEditor (Current work), Patch-For-Review, Technical-Debt, VisualEditor-DataModel
marcella awarded T205535: Remove dedicated annotation operation types a Mountain of Wealth token.
Sep 26 2018, 1:07 PM · MW-1.33-notes (1.33.0-wmf.4; 2018-11-13), VisualEditor (Current work), Patch-For-Review, Technical-Debt, VisualEditor-DataModel
dchan updated the task description for T205535: Remove dedicated annotation operation types.
Sep 26 2018, 12:54 PM · MW-1.33-notes (1.33.0-wmf.4; 2018-11-13), VisualEditor (Current work), Patch-For-Review, Technical-Debt, VisualEditor-DataModel
dchan created T205535: Remove dedicated annotation operation types.
Sep 26 2018, 11:51 AM · MW-1.33-notes (1.33.0-wmf.4; 2018-11-13), VisualEditor (Current work), Patch-For-Review, Technical-Debt, VisualEditor-DataModel

Sep 19 2018

dchan added a comment to T203580: Create technical debt heat map for VE codebase.

Here's a version where the text is more or less readable, but JPEG compressed enough that Phabricator accepts it:

Sep 19 2018, 4:20 PM · VisualEditor (Current work), Technical-Debt

Sep 14 2018

dchan added a comment to T204387: Can't drag a collapsed selection across a reference on mobile.

I guess this may be the same issue as in T203555.

Sep 14 2018, 11:07 PM · VisualEditor (Current work), VisualEditor-MediaWiki-Mobile

Sep 13 2018

marcella awarded T203580: Create technical debt heat map for VE codebase a Doubloon token.
Sep 13 2018, 7:05 PM · VisualEditor (Current work), Technical-Debt
dchan updated the task description for T204192: Research text input support for Indian scripts.
Sep 13 2018, 7:13 AM · Language-Team, VisualEditor
dchan updated the task description for T204192: Research text input support for Indian scripts.
Sep 13 2018, 7:08 AM · Language-Team, VisualEditor
dchan updated the task description for T204192: Research text input support for Indian scripts.
Sep 13 2018, 6:57 AM · Language-Team, VisualEditor

Sep 10 2018

dchan claimed T203555: On Android Chrome, Thai text selections next to a link can collapse unexpectedly.
Sep 10 2018, 4:24 PM · VisualEditor (Current work), VisualEditor-ContentLanguage, VisualEditor-ContentEditable
dchan moved T203555: On Android Chrome, Thai text selections next to a link can collapse unexpectedly from Ready for Pick Up to In progress on the VisualEditor (Current work) board.
Sep 10 2018, 4:24 PM · VisualEditor (Current work), VisualEditor-ContentLanguage, VisualEditor-ContentEditable
dchan moved T203555: On Android Chrome, Thai text selections next to a link can collapse unexpectedly from Up next to Current work on the VisualEditor board.
Sep 10 2018, 4:23 PM · VisualEditor (Current work), VisualEditor-ContentLanguage, VisualEditor-ContentEditable
dchan edited projects for T203555: On Android Chrome, Thai text selections next to a link can collapse unexpectedly, added: VisualEditor; removed VisualEditor (Current work).

With input debugging enabled, it's easier to see that if the selection pin lands between the pre-close nail and the post-close nail then the selection collapses. In fact you can see this just editing <h1><a href='x'>foo</a> bar</h1>.

Sep 10 2018, 4:20 PM · VisualEditor (Current work), VisualEditor-ContentLanguage, VisualEditor-ContentEditable

Sep 5 2018

dchan moved T202719: Backspace across paragraphs is broken on mobile VE (Android Chrome) from To Triage to Needs Discussion/Analysis on the VisualEditor board.
Sep 5 2018, 4:41 PM · Browser-Support-Android-Google-Chrome, VisualEditor (Current work), Editing Q1 2018-19 mobile improvements, VisualEditor-MediaWiki-Mobile, VisualEditor-ContentEditable
dchan moved T203555: On Android Chrome, Thai text selections next to a link can collapse unexpectedly from To Triage to Needs Discussion/Analysis on the VisualEditor board.
Sep 5 2018, 4:38 PM · VisualEditor (Current work), VisualEditor-ContentLanguage, VisualEditor-ContentEditable
dchan added a comment to T202723: Mobile native selection context menu can obscure the VE toolbar.

Given the context disappearance is an iOS-specific issue, would it be worth doing the 60px scroll on non-iOS devices only?

Sep 5 2018, 2:09 PM · MW-1.33-notes (1.33.0-wmf.22; 2019-03-19), Editing QA, VisualEditor (Current work), Editing Q1 2018-19 mobile improvements, VisualEditor-MediaWiki-Mobile, VisualEditor-ContentEditable
dchan closed T64028: VisualEditor: Cursor skips by one character for Thai characters (expected skipping by one block) as Resolved.

This is working now; I guess it has been since 2d8c3e799601d3eb70127b32e262dbe1d64a7e4e .

Sep 5 2018, 1:39 PM · User-Ryasmeen, I18n, VisualEditor, VisualEditor-ContentLanguage
dchan closed T64028: VisualEditor: Cursor skips by one character for Thai characters (expected skipping by one block), a subtask of T35077: VisualEditor multilingual input / i18n issues (tracking), as Resolved.
Sep 5 2018, 1:39 PM · VisualEditor, Tracking-Neverending, VisualEditor-ContentLanguage
dchan added a parent task for T203555: On Android Chrome, Thai text selections next to a link can collapse unexpectedly: T202647: Assess VE mobile text input situation for each target wiki.
Sep 5 2018, 12:23 PM · VisualEditor (Current work), VisualEditor-ContentLanguage, VisualEditor-ContentEditable
dchan added a subtask for T202647: Assess VE mobile text input situation for each target wiki: T203555: On Android Chrome, Thai text selections next to a link can collapse unexpectedly.
Sep 5 2018, 12:23 PM · VisualEditor-MediaWiki-Mobile, Editing Q1 2018-19 mobile improvements, VisualEditor (Current work), VisualEditor-ContentEditable, VisualEditor-ContentLanguage
dchan created T203555: On Android Chrome, Thai text selections next to a link can collapse unexpectedly.
Sep 5 2018, 12:20 PM · VisualEditor (Current work), VisualEditor-ContentLanguage, VisualEditor-ContentEditable
dchan added a comment to T50429: [Epic] Support editing parts of a page in VisualEditor-MediaWiki.

Thanks @kaldari! As I understand it the fundamental differences between T76541 and T146072 are:

Sep 5 2018, 2:29 AM · VisualEditor, VisualEditor-Performance, VisualEditor-MediaWiki

Sep 4 2018

dchan updated the task description for T202647: Assess VE mobile text input situation for each target wiki.
Sep 4 2018, 5:24 PM · VisualEditor-MediaWiki-Mobile, Editing Q1 2018-19 mobile improvements, VisualEditor (Current work), VisualEditor-ContentEditable, VisualEditor-ContentLanguage

Sep 3 2018

dchan updated the task description for T202647: Assess VE mobile text input situation for each target wiki.
Sep 3 2018, 6:02 PM · VisualEditor-MediaWiki-Mobile, Editing Q1 2018-19 mobile improvements, VisualEditor (Current work), VisualEditor-ContentEditable, VisualEditor-ContentLanguage
dchan renamed T202647: Assess VE mobile text input situation for each target wiki from Assess the current VE-mobile situation for target wikis to Assess VE mobile text input situation for each target wiki.
Sep 3 2018, 6:00 PM · VisualEditor-MediaWiki-Mobile, Editing Q1 2018-19 mobile improvements, VisualEditor (Current work), VisualEditor-ContentEditable, VisualEditor-ContentLanguage

Aug 29 2018

dchan added a comment to T188839: When editing source, sometimes characters get put in strange places if you have too many characters on a line (due to space collapsing).

@matmarex Ohhh that's a really good point. And unfortunately you're right to be suspicious; some IMEs can add / remove whitespace without committing all candidate text so we can't safely rerender.

Aug 29 2018, 3:43 PM · VisualEditor, MediaWiki-extensions-CodeMirror, Patch-For-Review, VisualEditor-ContentEditable, VisualEditor-MediaWiki-2017WikitextEditor

Aug 28 2018

dchan added a comment to T188839: When editing source, sometimes characters get put in strange places if you have too many characters on a line (due to space collapsing).

@matmarex , do you want to go forward with the solution you suggested? (I.e. use &nbsp; for every other space, on NWE only)

Aug 28 2018, 8:37 AM · VisualEditor, MediaWiki-extensions-CodeMirror, Patch-For-Review, VisualEditor-ContentEditable, VisualEditor-MediaWiki-2017WikitextEditor

Aug 26 2018

dchan added a comment to T202719: Backspace across paragraphs is broken on mobile VE (Android Chrome).

Hmm, looking at the Filibuster trace, it looks very much like the native backspace action leaves the DOM unchanged. Will investigate further.

Aug 26 2018, 12:09 PM · Browser-Support-Android-Google-Chrome, VisualEditor (Current work), Editing Q1 2018-19 mobile improvements, VisualEditor-MediaWiki-Mobile, VisualEditor-ContentEditable

Aug 24 2018

dchan updated subscribers of T202723: Mobile native selection context menu can obscure the VE toolbar.

@Esanders , should we work round this by having a minimum gap above the CE surface?

Aug 24 2018, 12:29 PM · MW-1.33-notes (1.33.0-wmf.22; 2019-03-19), Editing QA, VisualEditor (Current work), Editing Q1 2018-19 mobile improvements, VisualEditor-MediaWiki-Mobile, VisualEditor-ContentEditable
dchan updated the task description for T202723: Mobile native selection context menu can obscure the VE toolbar.
Aug 24 2018, 12:26 PM · MW-1.33-notes (1.33.0-wmf.22; 2019-03-19), Editing QA, VisualEditor (Current work), Editing Q1 2018-19 mobile improvements, VisualEditor-MediaWiki-Mobile, VisualEditor-ContentEditable
dchan created T202723: Mobile native selection context menu can obscure the VE toolbar.
Aug 24 2018, 12:24 PM · MW-1.33-notes (1.33.0-wmf.22; 2019-03-19), Editing QA, VisualEditor (Current work), Editing Q1 2018-19 mobile improvements, VisualEditor-MediaWiki-Mobile, VisualEditor-ContentEditable
dchan created T202719: Backspace across paragraphs is broken on mobile VE (Android Chrome).
Aug 24 2018, 11:12 AM · Browser-Support-Android-Google-Chrome, VisualEditor (Current work), Editing Q1 2018-19 mobile improvements, VisualEditor-MediaWiki-Mobile, VisualEditor-ContentEditable

Aug 23 2018

dchan added a comment to T202148: Instrument editing pipeline to be able to figure out which common editing features are used.

I don't know much about this, but will/can we get the data broken down by wiki (and therefore effectively by language)? That could be pretty useful for script / IME support purposes

Aug 23 2018, 5:19 PM · MW-1.33-notes (1.33.0-wmf.19; 2019-02-26), MW-1.32-notes (WMF-deploy-2018-10-16 (1.32.0-wmf.26)), Patch-For-Review, Editing Q1 2018-19 mobile improvements, VisualEditor (Current work), Product-Analytics
dchan created T202647: Assess VE mobile text input situation for each target wiki.
Aug 23 2018, 4:55 PM · VisualEditor-MediaWiki-Mobile, Editing Q1 2018-19 mobile improvements, VisualEditor (Current work), VisualEditor-ContentEditable, VisualEditor-ContentLanguage

Aug 20 2018

dchan updated the task description for T116275: DM/DOM sync can break if you use an IME.
Aug 20 2018, 2:05 PM · MW-1.27-release (WMF-deploy-2015-12-08_(1.27.0-wmf.8)), Epic, VisualEditor-ContentEditable, VisualEditor
dchan reopened T116275: DM/DOM sync can break if you use an IME as "Open".

It's still possible to trigger this bug, though Change 247800 made things much better. Will update the description.

Aug 20 2018, 1:45 PM · MW-1.27-release (WMF-deploy-2015-12-08_(1.27.0-wmf.8)), Epic, VisualEditor-ContentEditable, VisualEditor
dchan moved T201870: Cursor into and out of table captions correctly with up/down arrow from Ready for Pick Up to Code review on the VisualEditor (Current work) board.
Aug 20 2018, 1:13 PM · Verified, MW-1.32-notes (WMF-deploy-2018-08-28 (1.32.0-wmf.19)), VisualEditor (Current work), Accessibility, VisualEditor-Tables

Aug 19 2018

dchan claimed T201870: Cursor into and out of table captions correctly with up/down arrow.

Ok, I think we should worry about Firefox in a separate task, because the issues are different (and I guess more intractable).

Aug 19 2018, 6:31 AM · Verified, MW-1.32-notes (WMF-deploy-2018-08-28 (1.32.0-wmf.19)), VisualEditor (Current work), Accessibility, VisualEditor-Tables
dchan added a comment to T201870: Cursor into and out of table captions correctly with up/down arrow.

Ok, I added some code for cursoring downwards out of a caption. Note though that none of this code fixes things on Firefox.

Aug 19 2018, 6:22 AM · Verified, MW-1.32-notes (WMF-deploy-2018-08-28 (1.32.0-wmf.19)), VisualEditor (Current work), Accessibility, VisualEditor-Tables

Aug 13 2018

dchan added a comment to T188839: When editing source, sometimes characters get put in strange places if you have too many characters on a line (due to space collapsing).

I agree with @matmarex 's proposal that we shouldn't revert to the &nbsp; behaviour for visual editing (because we want nbsp to represent actual non-breaking spaces). It's a different issue in wikitext because non-breaking spaces are inferred heuristically.

Aug 13 2018, 11:43 AM · VisualEditor, MediaWiki-extensions-CodeMirror, Patch-For-Review, VisualEditor-ContentEditable, VisualEditor-MediaWiki-2017WikitextEditor

Aug 9 2018

dchan moved T201599: It is possible to get the cursor into a block slug from Ready for Pick Up to In progress on the VisualEditor (Current work) board.
Aug 9 2018, 10:20 AM · MW-1.32-notes (WMF-deploy-2018-08-21 (1.32.0-wmf.18)), VisualEditor (Current work), VisualEditor-ContentEditable
dchan edited projects for T201599: It is possible to get the cursor into a block slug, added: VisualEditor (Current work); removed VisualEditor.
Aug 9 2018, 10:19 AM · MW-1.32-notes (WMF-deploy-2018-08-21 (1.32.0-wmf.18)), VisualEditor (Current work), VisualEditor-ContentEditable
dchan added a comment to T200819: Clicking to the left of a line moves the cursor few lines above in some cases .

Ok, I created T201599 for the separate slug cursoring issue, which is really a corruption problem as @Deskana suggests.

Aug 9 2018, 10:05 AM · VisualEditor (Current work)
dchan created T201599: It is possible to get the cursor into a block slug.
Aug 9 2018, 10:02 AM · MW-1.32-notes (WMF-deploy-2018-08-21 (1.32.0-wmf.18)), VisualEditor (Current work), VisualEditor-ContentEditable

Aug 8 2018

dchan added a comment to T200819: Clicking to the left of a line moves the cursor few lines above in some cases .

And the problem happens because Chromium natively puts the cursor into the slug even though the click is outside the slug, but not in Firefox because its native behaviour is different (it puts the cursor directly into the surrounding div).

Aug 8 2018, 8:45 PM · VisualEditor (Current work)
dchan added a comment to T200819: Clicking to the left of a line moves the cursor few lines above in some cases .

Clicking twice to the left of the node puts the cursor into the slug text and lets you edit it

Aug 8 2018, 8:37 PM · VisualEditor (Current work)
dchan claimed T200819: Clicking to the left of a line moves the cursor few lines above in some cases .

This is reproducible with any focusable block node, e.g. on this minimal document:

<hr>x
Aug 8 2018, 8:34 PM · VisualEditor (Current work)

Jul 27 2018

dchan added a comment to T199298: Visual diff fails with "jQuery.Deferred exception: listContents is undefined".

I feel the following code isn't that sensible now that meta items live in the main linear data:

Jul 27 2018, 6:20 PM · Verified-in-Phase2, MW-1.32-notes (WMF-deploy-2018-07-31 (1.32.0-wmf.15)), VisualEditor (Current work), VisualEditor-VisualDiffs
dchan added a comment to T199011: CX2: Paragraph becomes unusable when switching to 'Don't use machine translation' or 'Use source text' .

Thanks! I'm not seeing the error, with parameters: page=Foo&from=en&to=es&targettitle=Bar&version=2&revision=<XXX> . It seems to work fine for me, on both my local installation and on wikipedia.org . Can you confirm a particular page+paragraph that fails for you on wikipedia.org?

Jul 27 2018, 2:38 PM · User-Ryasmeen, MW-1.32-notes (WMF-deploy-2018-07-24 (1.32.0-wmf.14)), Language-2018-July-September, ContentTranslation
dchan added a comment to T199011: CX2: Paragraph becomes unusable when switching to 'Don't use machine translation' or 'Use source text' .

Hi @Petar.petkovic, can you tell me steps to reproduce the continuing issues (after Change 446198) you mentioned above? The points you make in T199754 look good, but I'd like to have a failing test case to verify against.

Jul 27 2018, 2:01 PM · User-Ryasmeen, MW-1.32-notes (WMF-deploy-2018-07-24 (1.32.0-wmf.14)), Language-2018-July-September, ContentTranslation
dchan added a comment to T196521: Uncaught TypeError: Cannot read property 'getOffset' of null appears when typing a character in Find field while RegEx option is enabled.

I'm also failing to reproduce this — I tried on 4c51ee9ec57678f29b320868161d570dba7c4ccb and on aca98533eab67e3154c4b243cbcadd155b69398e .

Jul 27 2018, 1:00 PM · MW-1.32-notes (WMF-deploy-2018-08-07 (1.32.0-wmf.16)), User-Ryasmeen, Verified, VisualEditor (Current work)

Jul 12 2018

dchan added a comment to T199464: PolyGerrit unified diff view is too narrow without enabling "Fit to screen" setting.

Thanks, that solves it for me - is there any way that can be the default (at least for Unified)?

Jul 12 2018, 5:18 PM · Gerrit
dchan created T199464: PolyGerrit unified diff view is too narrow without enabling "Fit to screen" setting.
Jul 12 2018, 4:50 PM · Gerrit

Jul 10 2018

dchan added a comment to T190941: Our HTTPS private keys seem to be obtainable under non-US surveillance laws, creating a worldwide MITM attack vector.

Sure, thanks

Jul 10 2018, 6:31 AM · Security

Jun 25 2018

dchan added a comment to T193587: CX2: Rich text editing in dialogs is not possible since cards are shown in a deactivated tools column.

Let me comment on the specific issue of modality. There are definitely parts of the VE code that assume dialogs operating on a portion of the document must be modal. One example is that there's a transaction undo stack that mirrors the dialog opening stack. Another is that there are places where stored offsets could be invalidated if the main document remains editable while the dialog is visible.

Jun 25 2018, 11:19 AM · User-Ryasmeen, MW-1.32-notes (WMF-deploy-2018-07-10 (1.32.0-wmf.12)), Language-2018-Apr-June, ContentTranslation

Jun 14 2018

dchan added a comment to T196921: Three slugs in every header/footer on Wikisource VisualEditor, two of them don't work (cause an exception).

@Esanders : yes, that seems to solve the issue, both in my local testing and when I monkey patch the live pl.wikisource.org page above.

Jun 14 2018, 7:29 PM · Verified, MW-1.32-notes (WMF-deploy-2018-06-26 (1.32.0-wmf.10)), VisualEditor (Current work), ProofreadPage, VisualEditor-MediaWiki

May 20 2018

dchan added a comment to T76541: Client-side approach to true section editing: Load/display document-level data (e.g. internal lists, meta lists) for the whole document but only the content for a part of the document.

As well as the approach in https://gerrit.wikimedia.org/r/433753 , we should also consider the following alternative:

  • A class provides the LinearData interface on a live slice of the linear data (performing offset translation and bounds checking)
  • ve.dm.Document builds on top of that class

That approach would more robustly prevent access outside the slice. But one drawback would be apparent dangling references in the slice.

May 20 2018, 7:47 AM · User-Ryasmeen, MW-1.33-notes (1.33.0-wmf.19; 2019-02-26), VisualEditor (Current work), Epic, VisualEditor-MediaWiki, VisualEditor-Performance

May 18 2018

dchan added a comment to T49344: Internal nodes should eventually be in a separate document ("sub-documents").

Here is a proposal for refinement:

May 18 2018, 2:10 PM · Epic, Patch-For-Review, VisualEditor-Performance, VisualEditor, Technical-Debt

May 2 2018

dchan added a comment to T193062: Undo/redo on citations doesn't work correctly.

With change 430413, "change citation, undo, redo" seems to work fine in all the cases I've tried.

May 2 2018, 4:53 PM · User-Ryasmeen, Verified, MW-1.32-notes (WMF-deploy-2018-05-08 (1.32.0-wmf.3)), VisualEditor (Current work), VisualEditor-EditingTools
dchan added a comment to T193062: Undo/redo on citations doesn't work correctly.

Hmm, thanks, it seems I overlooked that there are a few different ways reference-instead-of-copy can happen here - investigating further.

May 2 2018, 9:10 AM · User-Ryasmeen, Verified, MW-1.32-notes (WMF-deploy-2018-05-08 (1.32.0-wmf.3)), VisualEditor (Current work), VisualEditor-EditingTools