Ticket #3481: 3481.patch

editor/_source/internals/fck.js

@@ -957 +957 @@
         FCKTools.AddEventListener( FCK.EditorDocument, 'mousemove', _FCK_MouseEventsListener ) ;
         FCKTools.AddEventListener( FCK.EditorDocument, 'mousedown', _FCK_MouseEventsListener ) ;
         FCKTools.AddEventListener( FCK.EditorDocument, 'mouseup', _FCK_MouseEventsListener ) ;
+        if ( FCKBrowserInfo.IsSafari )
+        {
+                // #3481: WebKit has a bug with paste where the paste contents may leak
+                // outside table cells. So add padding nodes before and after the paste.
+                FCKTools.AddEventListener( FCK.EditorDocument, 'paste', function( evt )
+                {
+                        var range = new FCKDomRange( FCK.EditorWindow );
+                        var nodeBefore = FCK.EditorDocument.createTextNode( '\ufeff' );
+                        var nodeAfter = FCK.EditorDocument.createTextNode( '\ufeff' );
+                        range.MoveToSelection();
+                        range.DeleteContents();
 
+                        // Insert padding nodes.
+                        range.InsertNode( nodeBefore );
+                        range.Collapse();
+                        range.InsertNode( nodeAfter );
+
+                        // Move the selection to between the padding nodes.
+                        range.MoveToPosition( nodeAfter, 3 );
+                        range.Select();
+
+                        // Remove the padding nodes after the paste is done.
+                        setTimeout( function()
+                                {
+                                        nodeBefore.parentNode.removeChild( nodeBefore );
+                                        nodeAfter.parentNode.removeChild( nodeAfter );
+                                }, 0 );
+                } );
+        }
+
         // Most of the CTRL key combos do not work under Safari for onkeydown and onkeypress (See #1119)
         // But we can use the keyup event to override some of these...
         if ( FCKBrowserInfo.IsSafari )

_whatsnew.html

@@ -62 +62 @@
                         had no effect if ProcessHTMLEntities is false.</li>
                 <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/3880">#3880</a>] Fixed some minor
                         logical and typing mistakes in fckdomrange_ie.js.</li>
+                <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/3481">#3481</a>] Fixed an issue in
+                        WebKit where paste actions inside table cells may leak outside of the table cell.</li>
         </ul>
         <p>
                 <a href="_whatsnew_history.html">See previous versions history</a></p>