Ticket #3618: 3618_2.patch

_source/plugins/list/plugin.js

@@ -476 +476 @@
                         for ( i = 0 ; i < listsCreated.length ; i++ )
                         {
                                 listNode = listsCreated[i];
-                                var stopFlag = false,
-                                        currentNode = listNode;
-
-                                while ( !stopFlag )
-                                {
-                                        currentNode = currentNode.getNext();
-                                        if ( currentNode && currentNode.type == CKEDITOR.NODE_TEXT && emptyTextRegex.test( currentNode.getText() ) )
-                                                continue;
-                                        stopFlag = true;
-                                }
-
-                                if ( currentNode && currentNode.getName() == this.type )
-                                {
-                                        currentNode.remove();
-                                        currentNode.moveChildren( listNode );
-                                }
+                                var mergeSibling;
+                                ( mergeSibling = CKEDITOR.tools.bind( function( rtl ){
+
+                                        var stopFlag = false,
+                                                currentNode = listNode;
+                                        while ( !stopFlag )
+                                        {
+                                                currentNode = currentNode[ rtl ? 'getPrevious' : 'getNext' ].call( currentNode );
+                                                if ( currentNode && currentNode.type == CKEDITOR.NODE_TEXT &&
+                                                     emptyTextRegex.test( currentNode.getText() ) )
+                                                        continue;
+                                                stopFlag = true;
+                                        }
+                                        if ( currentNode && currentNode.getName &&
+                                             currentNode.getName() == this.type )
+                                        {
+                                                currentNode.remove();
+                                                currentNode.moveChildren( listNode, rtl ? true : false );
+                                        }
+                                }, this ) )();
 
-                                stopFlag = false;
-                                currentNode = listNode;
-                                while ( !stopFlag )
-                                {
-                                        currentNode = currentNode.getNext();
-                                        if ( currentNode && currentNode.type == CKEDITOR.NODE_TEXT && emptyTextRegex.test( currentNode.getText() ) )
-                                                continue;
-                                        stopFlag = true;
-                                }
-                                if ( currentNode && currentNode.getName() == this.type  )
-                                {
-                                        currentNode.remove();
-                                        currentNode.moveChildren( listNode, true );
-                                }
-                        }
+                                mergeSibling( true );
+                        }
 
                         // Clean up, restore selection and update toolbar button states.
                         CKEDITOR.dom.element.clearAllMarkers( database );