Changeset 4516

Timestamp:

11/23/09 07:49:09 (4 years ago)

Author:

garry.yao

Message:

Fixing #3950: table column insertion incorrect when selecting empty cell area.

Location:

CKEditor/trunk

Files:

• CHANGES.html (modified) (1 diff)
• _source/plugins/selection/plugin.js (modified) (2 diffs)

CKEditor/trunk/CHANGES.html

@@ -65 +65 @@
                 <li><a href="http://dev.fckeditor.net/ticket/4651">#4651</a> : Fixed 'Deleted' and 'Inserted' text style is not rendering in wysiwyg mode and is wrong is source mode.</li>
                 <li><a href="http://dev.fckeditor.net/ticket/4654">#4654</a> : Fixed 'CKEDITOR.config.font_defaultLabel(fontSize_defaultLabel)' is not working.</li>
-        </ul>
+                <li><a href="http://dev.fckeditor.net/ticket/3950">#3950</a> : Fixed table column insertion incorrect when selecting empty cell area.</li>
         <h3>
                 CKEditor 3.0.1</h3>

CKEditor/trunk/_source/plugins/selection/plugin.js

@@ -1 @@
-/*
+/*
 Copyright (c) 2003-2009, CKSource - Frederico Knabben. All rights reserved.
 For licensing, see LICENSE.html or http://ckeditor.com/license
@@ -641 +641 @@
                                                                 var startContainer = range.startContainer,
                                                                         startOffset = range.startOffset;
+                                                                // Limit the fix only to non-block elements.(#3950)
                                                                 if ( startOffset == ( startContainer.getChildCount ?
-                                                                        startContainer.getChildCount() : startContainer.getLength() ) )
+                                                                         startContainer.getChildCount() : startContainer.getLength() )
+                                                                         && !startContainer.isBlockBoundary() )
                                                                         range.setStartAfter( startContainer );
                                                                 else break;