Changeset 4521

Timestamp:

11/23/09 08:08:11 (3 years ago)

Author:

garry.yao

Message:

Fixing #4041 : open context menu on multiple cells to remove them result in only one removed.

Location:

CKEditor/trunk

Files:

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

CKEditor/trunk/CHANGES.html

@@ -68 +68 @@
                 <li><a href="http://dev.fckeditor.net/ticket/3912">#3912</a> : Fixed UIColor not working in IE when page has more than 30+ editors.</li>
                 <li><a href="http://dev.fckeditor.net/ticket/4031">#4031</a> : Fixed mouse cursor on toolbar combo has more than 3 shapes.</li>
+                <li><a href="http://dev.fckeditor.net/ticket/4041">#4041</a> : Fixed open context menu on multiple cells to remove them result in only one removed.</li>
         <h3>
                 CKEditor 3.0.1</h3>

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

@@ -153 +153 @@
                                                 doc.on( 'mousedown', disableSave );
                                                 doc.on( 'mouseup',
-                                                        function()
+                                                        function( evt )
                                                         {
+                                                                // IE context-menu event in table cells collapse
+                                                                // whatever selection is, avoiding saving this
+                                                                // 'wrong' snapshot.(#3001)
+                                                                var evt = evt.data;
+                                                                if ( evt.$.button == 2
+                                                                                && evt.getTarget().hasAscendant( 'table' ) )
+                                                                        return;
+
                                                                 saveEnabled = true;
                                                                 setTimeout( function()