Catrope has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/405549 )
Change subject: SurfaceSynchronizer: Translate selections when unapplying
......................................................................
SurfaceSynchronizer: Translate selections when unapplying
Bug: T185395
Change-Id: If5df0b5755e56f99ca981bcee88485a4b71ddd40
---
M src/dm/ve.dm.SurfaceSynchronizer.js
1 file changed, 2 insertions(+), 6 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/VisualEditor/VisualEditor
refs/changes/49/405549/1
diff --git a/src/dm/ve.dm.SurfaceSynchronizer.js
b/src/dm/ve.dm.SurfaceSynchronizer.js
index ddd679e..8e01c18 100644
--- a/src/dm/ve.dm.SurfaceSynchronizer.js
+++ b/src/dm/ve.dm.SurfaceSynchronizer.js
@@ -147,13 +147,9 @@
* @inheritdoc
*/
ve.dm.SurfaceSynchronizer.prototype.unapplyChange = function ( change ) {
- var authorId, nullSelections = {};
- // Author selections are potentially invalid now, null them all out
- for ( authorId in this.authorSelections ) {
- nullSelections[ authorId ] = new ve.dm.NullSelection( this.doc
);
- }
- this.applyNewSelections( nullSelections );
change.unapplyTo( this.surface );
+ // Translate all selections for what we just unapplied
+ this.applyNewSelections( this.authorSelections, change.reversed() );
};
/**
--
To view, visit https://gerrit.wikimedia.org/r/405549
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: If5df0b5755e56f99ca981bcee88485a4b71ddd40
Gerrit-PatchSet: 1
Gerrit-Project: VisualEditor/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Catrope <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
