Subject	Repo	Branch	Lines +/-
Update VE core submodule to master (495cd7d65)	mediawiki/extensions/VisualEditor	master	+1 -1
SurfaceFragment: Avoid double-annotating in insertDocument, as well	VisualEditor/VisualEditor	master	+44 -26
Update VE core submodule to master (7b51cb6ab)	mediawiki/extensions/VisualEditor	master	+1 -1
SurfaceFragment: avoid double-annotating in insertContent	VisualEditor/VisualEditor	master	+55 -2

Customize query in gerrit

Related Objects

Mentioned In: T154493: Clicking the "remove" button in the link context menu for a newly-created link is a no-op
Mentioned Here: rGVED495cd7d65439: Re-apply "Remove $returnFocusTo hack now null is supported upstream"
rGVED7b51cb6aba3c: DiffElement: Resolve URLs in links and target at new window

Event Timeline

Jdforrester-WMF created this task.Aug 17 2016, 9:10 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 17 2016, 9:10 PM

Test case:

Foo
Bar

Copy an 'o' to the end of 'Bar' and you get:

Baro

Esanders assigned this task to dchan.Aug 17 2016, 9:18 PM

I think this goes back a long way (I see this same behaviour in revisions from 2014).

Jdforrester-WMF moved this task from To Triage to TR0: Interrupt on the VisualEditor board.Aug 18 2016, 5:49 PM

Jdforrester-WMF renamed this task from Paste into (end of?) an annotation serialises very poorly to Paste identical annotated content into (end of?) an annotation serialises very poorly because it gets double-annotated.Aug 23 2016, 6:13 PM

Jdforrester-WMF removed dchan as the assignee of this task.

Jdforrester-WMF lowered the priority of this task from High to Medium.

Jdforrester-WMF changed the point value for this task from 1 to 8.

Jdforrester-WMF added a subscriber: dchan.

Change 344417 had a related patch set uploaded (by DLynch):
[VisualEditor/VisualEditor@master] SurfaceFragment: avoid double-annotating in insertContent

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

gerritbot added a project: Patch-For-Review.Mar 23 2017, 4:41 PM

That patch fixes Ed's test case. It makes the assumption that when moving content around like this, all that matters is the comparable annotation objects, and discards the ones on the pasted content which are comparable-to the surrounding annotations that are going to get applied.

I chose the order of application that'll get us Baro, but it'd be easy to adjust it so we get Baro instead. It's just a matter of which annotation set gets priority.

If the approach seems good, I can add it for external pastes as well.

Change 344417 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] SurfaceFragment: avoid double-annotating in insertContent

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

Change 344692 had a related patch set uploaded (by Jforrester):
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (7b51cb6ab)

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

(Now waiting for the external paste fix, but fixed for internal pastes.)

Jdforrester-WMF assigned this task to DLynch.Mar 24 2017, 9:34 PM

Jdforrester-WMF mentioned this in T154493: Clicking the "remove" button in the link context menu for a newly-created link is a no-op.Mar 24 2017, 10:18 PM

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

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

Jdforrester-WMF removed a project: Patch-For-Review.Mar 24 2017, 11:58 PM

Change 344813 had a related patch set uploaded (by DLynch):
[VisualEditor/VisualEditor@master] SurfaceFragment: Avoid double-annotating in insertDocument, as well

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

gerritbot added a project: Patch-For-Review.Mar 26 2017, 4:26 AM

Change 344813 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] SurfaceFragment: Avoid double-annotating in insertDocument, as well

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

Change 344803 had a related patch set uploaded (by Jforrester):
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (495cd7d65)

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

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

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

Jdforrester-WMF closed this task as Resolved.Mar 28 2017, 3:50 PM

Jdforrester-WMF edited projects, added: MW-1.29-release (WMF-deploy-2017-03-28_(1.29.0-wmf.18)); removed: Patch-For-Review.

Restricted Application added a project: User-Ryasmeen. · View Herald TranscriptMar 28 2017, 3:50 PM

Paste identical annotated content into (end of?) an annotation serialises very poorly because it gets double-annotatedClosed, ResolvedPublic8 Estimated Story PointsActions

Description

Details

Related Objects

Event Timeline

Paste identical annotated content into (end of?) an annotation serialises very poorly because it gets double-annotated
Closed, ResolvedPublic8 Estimated Story Points
Actions