Now that we have use cases for serialising transactions (auto-save, CollabPad), we should consider a more compact format, David's initial proposal is
[ { type: 'retain', length: 5 }, { type: 'replace', remove: [ 'a' ], insert: [ 'b' ] }, { type: 'retain', length: 3 } ]
->
[ 5, [ [ 'a' ], [ 'b' ] ], 3 ]
There are also attribute and annotate operations to consider. They could be left as is as they are much rarer.
Change 414026 had a related patch set uploaded (by Divec; owner: Divec):
[VisualEditor/VisualEditor@master] WIP: Minify the serialized form of Transactions
Change 414104 had a related patch set uploaded (by Esanders; owner: Esanders):
[VisualEditor/VisualEditor@master] Transactions: Only set insertedDataOffset/Length when meaningful
Change 414104 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] Transactions: Only set insertedDataOffset/Length when meaningful
Change 413904 had a related patch set uploaded (by Esanders; owner: Esanders):
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (d946b72)
Change 414127 had a related patch set uploaded (by Divec; owner: Divec):
[VisualEditor/VisualEditor@master] WIP: Minify the serialized form of Changes
Change 413904 had a related patch set uploaded (by Esanders; owner: Esanders):
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (08520dc)
For a typical authoring session, document history as a Change will contain a large proportion of transactions that are consecutive text insertions in the same place, with the same annotation as the insertion context. There may be a transaction for every keystroke (but somewhat fewer for many IMEs).
Therefore it makes sense to optimize the Change serialization to just use the inserted text string for such transactions.
Change 413904 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (e6b9330)
After the latest patches a serialized change looks like this:
{"start":1,"transactions":[{"operations":[646,["s","h"],982],"authorId":null}],"stores":[null,null,null,null,null],"selections":{}}
We could drop authorId when it's null, selections when they are empty, and stores when it is all null.
Change 414026 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] Minify the serialized form of Transactions
Change 414127 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] Minify the serialized form of Changes
Change 414601 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (35cf6d9ff)
Change 414601 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (35cf6d9ff)
Change 414653 had a related patch set uploaded (by Esanders; owner: Esanders):
[VisualEditor/VisualEditor@master] Further minification of tranasctions
Change 414653 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] Further minification of tranasctions
Change 414664 had a related patch set uploaded (by Esanders; owner: Esanders):
[VisualEditor/VisualEditor@master] ve.dm.Change: More minification
Change 414664 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] ve.dm.Change: More minification
Change 414679 had a related patch set uploaded (by Esanders; owner: Esanders):
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (a8da324)
Change 414679 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (a8da324)