Change Details

Example HTML: ``` <h1><a href='x'><i>A<span rel='ve:Alien'>X</span></i>B</a>x</h1> ``` Turns into: ``` [ '<heading>', 'A', '<alienInline>', '</alienInline>', 'B', 'x', '</heading>' ] ``` Offset 3 is the `</alienInline>`. Requesting offset 3 will point at the "X" inside the alien, which is questionable. If a valid cursor position is desired, it should probably return outside of it. If offset 4 is requested, the return is: ``` {node: <i class="ve-ce-textStyleAnnotation ve-ce-italicAnnotation">, offset: 0} ``` This is definitely wrong, since the offset jumps back to the start of the containing element.

**Steps to reproduce:** 1. Edit the following HTML: ``` <h1><a href='x'><i>A<span rel='ve:Alien'>X</span></i>B</a>x</h1> ``` 2. Press Right Arrow three times. The 'X' should be selected. 3. Press Right Arrow once more. **Observed behaviour:** The cursor jumps back to before the 'A' (inside the link). **Expected behaviour:** The cursor should be just after the 'X'. **Cause:** The getNodeAndOffset function returns incorrect values for the offset after the close of a non-focusable node. In this example, the linear model is: ``` [ '<heading>', 'A', '<alienInline>', '</alienInline>', 'B', 'x', '</heading>' ] ``` For offset 4 (immediately after the alienInline), the position returned is: ``` {node: <i class="ve-ce-textStyleAnnotation ve-ce-italicAnnotation">, offset: 0} ``` This is definitely wrong, since it is back at the start of the containing element (hence the cursor jump).

Example**Steps to reproduce:** 1. Edit the following HTML: ``` <h1><a href='x'><i>A<span rel='ve:Alien'>X</span></i>B</a>x</h1> ``` Turns into2. Press Right Arrow three times. The 'X' should be selected. 3. Press Right Arrow once more. **Observed behaviour:** The cursor jumps back to before the 'A' (inside the link). **Expected behaviour:** The cursor should be just after the 'X'. **Cause:** The getNodeAndOffset function returns incorrect values for the offset after the close of a non-focusable node. In this example, the linear model is: ``` [ '<heading>', 'A', '<alienInline>', '</alienInline>', 'B', 'x', '</heading>' ] ``` Offset 3 is the `</alienInline>`. Requesting offset 3 will point at the "X" inside the alien, which is questionable. If a valid cursor position is desired, it should probably return outside of it. IfFor offset 4 is requested(immediately after the alienInline), the position returned is: ``` {node: <i class="ve-ce-textStyleAnnotation ve-ce-italicAnnotation">, offset: 0} ``` This is definitely wrong, since the offset jumpit is back toat the start of the containing element (hence the cursor jump).