Page MenuHomePhabricator

Document corruption when inserting linebreak while annotating
Closed, ResolvedPublic

Description

  1. Open "Empty" in standalone demo, or an empty page in MediaWiki VisualEditor
  2. Press Ctrl+B, type "foo<enter>bar"
  3. Observe that the "b" never appears, and by the time you type "r" there is an exception thrown

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Setting to high as it puts the view out of sync with the model.

Ah, i see ... with TreeModifier, pressing Enter at the end of a paragraph leaves that paragraph unchanged (whereas before TreeModifier, it was getting rebuilt entirely). Then it moves the focus into the new paragraph below. That means we're escaping the paragraph without cleaning up the unicorns, which the current code assumes to be impossible.

That's not a very safe assumption in any case: some IME probably has a different way of violating it.

Change 592398 had a related patch set uploaded (by Divec; owner: Divec):
[VisualEditor/VisualEditor@master] Cleanup unicorns even in another content branch node

https://gerrit.wikimedia.org/r/592398

Change 592398 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] Cleanup unicorns even in another content branch node

https://gerrit.wikimedia.org/r/592398

Change 592698 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (e571a9e73)

https://gerrit.wikimedia.org/r/592698

Change 592698 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (e571a9e73)

https://gerrit.wikimedia.org/r/592698

Esanders edited projects, added Skipped QA; removed Editing QA.
Esanders updated the task description. (Show Details)