Ticket #3375: 3375.patch

_source/plugins/link/dialogs/link.js

@@ -1131 +1131 @@
                                 // Create element if current selection is collapsed.
                                 var selection = editor.getSelection(),
                                         ranges = selection.getRanges();
+                                
+                                // Create an padding node for inline position.
+                                var paddingInine = editor.document.createElement( 'span' );
+                                paddingInine.setHtml( '' );     // Zero Width No-Break Space (U+FEFF).
+                                paddingInine = paddingInine.getFirst();         
+
                                 if ( ranges.length == 1 && ranges[0].collapsed )
                                 {
-                                        var text = new CKEDITOR.dom.text( attributes._cke_saved_href, editor.document );
-                                        ranges[0].insertNode( text );
-                                        ranges[0].selectNodeContents( text );
+                                        var textNode = new CKEDITOR.dom.text( 
+                                                attributes._cke_saved_href, editor.document );
+                                        // Avoid text unappendable at the end of link.
+                                        ranges[0].insertNode( textNode );
+                                        ranges[0].selectNodeContents( textNode );
+                                        ranges[0].collapse();
+                                        ranges[0].insertNode( paddingInine.clone( true ) );
+                                        ranges[0].selectNodeContents( textNode );
                                         selection.selectRanges( ranges );
                                 }
 
@@ -1143 +1154 @@
                                 var style = new CKEDITOR.style( { element : 'a', attributes : attributes } );
                                 style.type = CKEDITOR.STYLE_INLINE;             // need to override... dunno why.
                                 style.apply( editor.document );
+                                
+                                // Avoid unable to exist link problem with Firefox.
+                                if( CKEDITOR.env.gecko && ranges[0].checkEndOfBlock() )
+                                {
+                                        var link = ranges[0].getCommonAncestor().getAscendant( 'a' );
+                                        paddingInine.clone( true ).insertAfter( link );
+                                }
 
                                 // Id. Apply only to the first link.
                                 if ( data.adv && data.adv.advId )