mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/VisualEditor
synced 2024-11-15 18:39:52 +00:00
4041d3f342
We would dirty-diff "</span>\n<!-- comment -->\n<span>" to "</span>\n\n<!-- comment --><span>", i.e. the second newline made a bunny-hop to the left over the comment. The actual bug turned out to involve a double bunny-hop, with "</span> <!-- comment -->\n<span>" turning into "</span>\n <!--comment --><span>", i.e. the newline bunny-hops both the comment and the space. This happened because outputWrappedMetaItems() didn't take wrappedWhitespace into account when restoring meta items and associated whitespace. I hacked a check for wrappedWhitespace into it, but we should really just rewrite this pile of hacks into a unified system for queuing and processing both whitespace and metadata. Change-Id: I4375f4c07983ffec6877d0371aeaa9bf6e65fd6e |
||
---|---|---|
.. | ||
lineardata | ||
nodes | ||
ve.dm.AnnotationSet.test.js | ||
ve.dm.BranchNode.test.js | ||
ve.dm.Converter.test.js | ||
ve.dm.Document.test.js | ||
ve.dm.DocumentSynchronizer.test.js | ||
ve.dm.example.js | ||
ve.dm.IndexValueStore.test.js | ||
ve.dm.InternalList.test.js | ||
ve.dm.LeafNode.test.js | ||
ve.dm.LinearData.test.js | ||
ve.dm.MetaList.test.js | ||
ve.dm.Model.test.js | ||
ve.dm.ModelRegistry.test.js | ||
ve.dm.Node.test.js | ||
ve.dm.NodeFactory.test.js | ||
ve.dm.Surface.test.js | ||
ve.dm.SurfaceFragment.test.js | ||
ve.dm.Transaction.test.js | ||
ve.dm.TransactionProcessor.test.js |