Pasting a table into the middle of text causes all the corruptions with TreeModifier
Closed, ResolvedPublic8 Estimated Story Points
Actions

Assigned To

Authored By

	Esanders
	May 22 2017, 9:57 AM

Description

Copy some rows of a table, e.g. from here and paste into the middle of a sentence in VE.

Pro-tip: Don't do this is the middle of a live demo.

Details

	Subject	Repo	Branch	Lines +/-
	Update VE core submodule to master (834fd702f)	mediawiki/extensions/VisualEditor	master	+8 -1
	TransactionProcessor: modify DM tree branches instead of rebuilding them	VisualEditor/VisualEditor	master	+1 K -192

Customize query in gerrit

Related Objects
Search...

Status	Assigned	Task
Resolved	dchan	T70779 VisualEditor: Metas at the end of a link annotation's anchor cause it to be split; should instead change with it
Open	None	T185520 Get rid of the legacy metaList accessor function in surface/document
Resolved	dchan	T56299 (Re-) unify data and metadata
Resolved	dchan	T162761 Introduce DM events for "a node of this type was inserted/deleted/modified"?
Resolved	dchan	T162762 ve.dm.TransactionProcessor: Restructure DM tree branches using existing nodes, instead of rebuilding them afresh
Resolved	dchan	T166041 Pasting a table into the middle of text causes all the corruptions with TreeModifier

Event Timeline

Esanders created this task.May 22 2017, 9:57 AM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMay 22 2017, 9:57 AM

Pro-tip: Don't do this is the middle of a live demo.

For my live demo, I first reverted the TreeModifier commit. Did you do that too?

Yup - leaving this here as a test case for David.

Jdforrester-WMF added a parent task: T162762: ve.dm.TransactionProcessor: Restructure DM tree branches using existing nodes, instead of rebuilding them afresh.May 23 2017, 8:44 AM

Jdforrester-WMF triaged this task as High priority.May 30 2017, 7:23 PM

Jdforrester-WMF moved this task from To Triage to TR3: Language support on the VisualEditor board.

Jdforrester-WMF set the point value for this task to 8.

Change 357171 had a related patch set uploaded (by Jforrester; owner: Divec):
[VisualEditor/VisualEditor@master] TransactionProcessor: modify DM tree branches instead of rebuilding them

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

gerritbot added a project: Patch-For-Review.Jun 20 2017, 5:55 PM

Change 357171 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] TransactionProcessor: modify DM tree branches instead of rebuilding them

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

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

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

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

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

• Deskana closed this task as Resolved.Sep 7 2017, 11:02 AM

Restricted Application added a project: User-Ryasmeen. · View Herald TranscriptSep 7 2017, 11:02 AM

Jdforrester-WMF edited projects, added MW-1.30-release-notes (WMF-deploy-2017-09-12_(1.30.0-wmf.18)); removed Patch-For-Review.Sep 13 2017, 3:37 PM

Pasting a table into the middle of text causes all the corruptions with TreeModifierClosed, ResolvedPublic8 Estimated Story PointsActions

Description

Details

Related ObjectsSearch...

Event Timeline

Pasting a table into the middle of text causes all the corruptions with TreeModifier
Closed, ResolvedPublic8 Estimated Story Points
Actions

Related Objects
Search...