mirror of
https://gerrit.wikimedia.org/r/mediawiki/extensions/VisualEditor
synced 2024-11-29 00:30:44 +00:00
7702ec10dc
We were populating empty content nodes with zero-length text nodes to make round-trip tests in the test suite work (otherwise blanking a paragraph leaves behind a zero-length text node whereas creating an empty paragraph does not), but the empty nodes are causing problems in CE apparently. * Do not create empty text nodes when constructing a node tree * Be more careful with text-only replacements: ** Don't resize a text node to zero, remove it instead ** There may not be a text node to resize at all, build it in that case ** Switch nodeRange to nodeOuterRange, this was probably broken before Tests: * Change test case for zero-length text node to assert that there is *no* zero-length text node :) * Remove a test case concerning an empty text node from the ve.ce.TextNode suite Change-Id: Ie677457f2f0a7823a517ba3077b844ef52a20fcc |
||
|---|---|---|
| .. | ||
| ce | ||
| dm | ||
| image.png | ||
| index.html | ||
| ve.BranchNode.test.js | ||
| ve.Document.test.js | ||
| ve.example.js | ||
| ve.Factory.test.js | ||
| ve.LeafNode.test.js | ||
| ve.Node.test.js | ||
| ve.qunit.js | ||
| ve.Range.test.js | ||
| ve.test.js | ||
