mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/VisualEditor
synced 2024-11-25 14:56:20 +00:00
c6cb537f1a
This was broken in three different ways: * On the way in, we were applying whitespace to an array of elements rather than the actual element, so the whitespace wasn't stored. * Whitespace processing on the way out was skipped for aliens because they had their own code path. Refactored this so alien openings and regular openings share much more code, including whitespace output. * Somewhat unrelatedly, innerPost output was broken for paragraphs containing inline elements, because the inline elements' processing polluted lastOuterPost. Discovered this because my test with inline aliens also happened to be the first test of whitespace preservation in paragraphs with inline content elements. Fixed by explicitly skipping content nodes when outputting whitespace. Fixed these issues and added a test case. Change-Id: I8edb61a008e60ace886b1a841b3417682ec39c32 |
||
|---|---|---|
| .. | ||
| annotations | ||
| nodes | ||
| ve.dm.Annotation.js | ||
| ve.dm.AnnotationFactory.js | ||
| ve.dm.BranchNode.js | ||
| ve.dm.Converter.js | ||
| ve.dm.Document.js | ||
| ve.dm.DocumentSynchronizer.js | ||
| ve.dm.js | ||
| ve.dm.LeafNode.js | ||
| ve.dm.Node.js | ||
| ve.dm.NodeFactory.js | ||
| ve.dm.Surface.js | ||
| ve.dm.SurfaceFragment.js | ||
| ve.dm.Transaction.js | ||
| ve.dm.TransactionProcessor.js | ||