Changeset 237

Timestamp:

2007-04-07 15:16:52 (21 months ago)

Author:

alfonsoml

Message:

Land in trunk the fix for #318, multiple linked images were merged in a single link.

Location:

FCKeditor/trunk

Files:

• editor/_source/internals/fck_ie.js (modified) (1 diff)
• _whatsnew.html (modified) (1 diff)

FCKeditor/trunk/editor/_source/internals/fck_ie.js

@@ -357 +357 @@
         if ( url.length > 0 )
         {
+                // If there are several images, and you try to link each one, all the images get inside the link:
+                // <img><img> -> <a><img></a><img> -> <a><img><img></a> due to the call to 'CreateLink' (bug in IE)
+                if (FCKSelection.GetType() == 'Control')
+                {
+                        // Create a link
+                        var oLink = this.EditorDocument.createElement( 'A' ) ;
+                        oLink.href = url ;
+
+                        // Get the selected object
+                        var oControl = FCKSelection.GetSelectedElement() ;
+                        // Put the link just before the object
+                        oControl.parentNode.insertBefore(oLink, oControl) ;
+                        // Move the object inside the link
+                        oControl.parentNode.removeChild( oControl ) ;
+                        oLink.appendChild( oControl ) ;
+                        
+                        return oLink ;
+                }
+
                 // Generate a temporary name for the link.
                 var sTempUrl = 'javascript:void(0);/*' + ( new Date().getTime() ) + '*/' ;

FCKeditor/trunk/_whatsnew.html

@@ -47 +47 @@
                         keep the Type attribute properly (it was converted to lowercase when the properties dialog was 
                         opened again).</li>
+                <li>[<a target="_blank" href="http://dev.fckeditor.net/ticket/318">#318</a>] Multiple linked images 
+                        are merged in a single link in IE.</li>
         </ul>
         <h3>