Ticket #3167: 3167.patch

_source/plugins/enterkey/plugin.js

@@ -211 +211 @@
 
                 var isEndOfBlock = range.checkEndOfBlock();
 
-                var elementPath = new CKEDITOR.dom.elementPath( range.getBoundaryNodes().startNode );
+                var startNode = range.getBoundaryNodes().startNode;
+                
+                /*
+                 * It is possible (and correct) for getBoundaryNodes() to return a startNode
+                 * that's outside of a collapsed selection's parent element in IE. e.g.
+                 *
+                 * <li>Some text here^</li></ul>extra   <-- extra becomes the startNode
+                 *
+                 * Such a startNode makes sense for the style system, but it doesn't make
+                 * sense when we want to split that <li>.
+                 *
+                 * So for IE, we'll need to a do an extra check on the native range's parent
+                 * element. (#3167)
+                 */
+                if ( CKEDITOR.env.ie )
+                {
+                        var $range = editor.getSelection().getNative().createRange();
 
+                        if ( $range.parentElement )
+                        {
+                                var parentElement = new CKEDITOR.dom.element( $range.parentElement() );
+                                if ( !parentElement.contains( startNode ) )
+                                        startNode = parentElement;
+                        }
+                }
+
+                var elementPath = new CKEDITOR.dom.elementPath( startNode );
+
                 var startBlock = elementPath.block,
                         startBlockTag = startBlock && elementPath.block.getName();