Last modified: 2013-10-25 21:09:52 UTC
dm.Surface.undo() and .redo() just call documentModel.rollback() / .commit() directly, without going through all the other stuff in .change() . Presumably this other stuff is useful, so we should figure out if bypassing it causes bugs.
In theory Surface.change() should also have a corresponding Surface.unchange() method (or a 'reversed' parameter).
Change 88728 had a related patch set uploaded by Catrope: Implement ve.dm.Surface.prototype.undo() and redo() in terms of change() https://gerrit.wikimedia.org/r/88728
Change 88728 merged by jenkins-bot: Implement ve.dm.Surface.prototype.undo() and redo() in terms of change() https://gerrit.wikimedia.org/r/88728