Ticket #4129: 4129.patch

_source/plugins/list/plugin.js

@@ -402 +402 @@
                                                 ranges[ 0 ].selectNodeContents( paragraph );
                                         selection.selectRanges( ranges );
                                 }
-                        }
+                                // Maybe a single range there enclosing the whole list,
+                                // turn on the list state manually(#4129).
+                                else
+                                {
+                                        var range = ranges.length == 1 && ranges[ 0 ],
+                                                enclosedNode = range && range.getEnclosedNode();
+                                        if ( enclosedNode && enclosedNode.is
+                                                && this.type == enclosedNode.getName() )
+                                        {
+                                                setState.call( this, editor, CKEDITOR.TRISTATE_ON );
+                                        }
+                                }
+                        }
 
                         var bookmarks = selection.createBookmarks( true );
 

_source/core/dom/range.js

@@ -1610 +1610 @@
                                 return false;
                 },
 
+                /**
+                 * Get the single node enclosed within the range if there's one.
+                 */
+                getEnclosedNode : function()
+                {
+                        var walkerRange = this.clone(),
+                                walker = new CKEDITOR.dom.walker( walkerRange ),
+                                isNotBookmarks = CKEDITOR.dom.walker.bookmark( true ),
+                                isNotWhitespaces = CKEDITOR.dom.walker.whitespaces( true ),
+                                evaluator = function( node )
+                                {
+                                        return isNotWhitespaces( node ) && isNotBookmarks( node );
+                                };
+                        walkerRange.evaluator = evaluator;
+                        var node = walker.next();
+                        walker.reset();
+                        var previous = walker.previous();
+                        return previous.equals( node ) ? node : null;
+                },
+
                 getTouchedStartNode : function()
                 {
                         var container = this.startContainer ;

_source/core/dom/walker.js

@@ -304 +304 @@
                         lastBackward : function()
                         {
                                 return iterateToLast.call( this, true );
+                        },
+
+                        reset : function()
+                        {
+                                delete this.current;
+                                this._ = {};
                         }
 
                 }

CHANGES.html

@@ -219 +219 @@
                 <li><a href="http://dev.fckeditor.net/ticket/4145">#4145</a> : Fixed the on demand 
                         (&quot;basic&quot;) loading model of the editor.</li>
                 <li><a href="http://dev.fckeditor.net/ticket/4139">#4139</a> : Fixed list plugin regression of [3903].</li>
+                <li><a href="http://dev.fckeditor.net/ticket/4129">#4129</a> : Fixed unable to remove single list wihtin document with Ctrl-A.</li>
         </ul>
         <h3>
                 CKEditor 3.0 RC</h3>