/*\r
-Copyright (c) 2003-2010, CKSource - Frederico Knabben. All rights reserved.\r
+Copyright (c) 2003-2013, CKSource - Frederico Knabben. All rights reserved.\r
For licensing, see LICENSE.html or http://ckeditor.com/license\r
*/\r
\r
else\r
element.hide();\r
}\r
+\r
+ dialog.layout();\r
};\r
\r
// Loads the parameters in a selected link to the link dialog fields.\r
\r
var parseLink = function( editor, element )\r
{\r
- var href = ( element && ( element.getAttribute( '_cke_saved_href' ) || element.getAttribute( 'href' ) ) ) || '',\r
+ var href = ( element && ( element.data( 'cke-saved-href' ) || element.getAttribute( 'href' ) ) ) || '',\r
javascriptMatch,\r
emailMatch,\r
anchorMatch,\r
// IE BUG: target attribute is an empty string instead of null in IE if it's not set.\r
if ( !target )\r
{\r
- var onclick = element.getAttribute( '_cke_pa_onclick' ) || element.getAttribute( 'onclick' ),\r
+ var onclick = element.data( 'cke-pa-onclick' ) || element.getAttribute( 'onclick' ),\r
onclickMatch = onclick && onclick.match( popupRegex );\r
if ( onclickMatch )\r
{\r
var featureMatch;\r
while ( ( featureMatch = popupFeaturesRegex.exec( onclickMatch[2] ) ) )\r
{\r
- if ( featureMatch[2] == 'yes' || featureMatch[2] == '1' )\r
+ // Some values should remain numbers (#7300)\r
+ if ( ( featureMatch[2] == 'yes' || featureMatch[2] == '1' ) && !( featureMatch[1] in { height:1, width:1, top:1, left:1 } ) )\r
retval.target[ featureMatch[1] ] = true;\r
else if ( isFinite( featureMatch[2] ) )\r
retval.target[ featureMatch[1] ] = featureMatch[2];\r
advAttr( 'advId', 'id' );\r
advAttr( 'advLangDir', 'dir' );\r
advAttr( 'advAccessKey', 'accessKey' );\r
- advAttr( 'advName', 'name' );\r
+\r
+ retval.adv.advName =\r
+ element.data( 'cke-saved-name' )\r
+ || element.getAttribute( 'name' )\r
+ || '';\r
advAttr( 'advLangCode', 'lang' );\r
advAttr( 'advTabIndex', 'tabindex' );\r
advAttr( 'advTitle', 'title' );\r
advAttr( 'advContentType', 'type' );\r
- advAttr( 'advCSSClasses', 'class' );\r
+ CKEDITOR.plugins.link.synAnchorSelector ?\r
+ retval.adv.advCSSClasses = getLinkClass( element )\r
+ : advAttr( 'advCSSClasses', 'class' );\r
advAttr( 'advCharset', 'charset' );\r
advAttr( 'advStyles', 'style' );\r
+ advAttr( 'advRel', 'rel' );\r
}\r
\r
// Find out whether we have any anchors in the editor.\r
- // Get all IMG elements in CK document.\r
- var elements = editor.document.getElementsByTag( 'img' ),\r
- realAnchors = new CKEDITOR.dom.nodeList( editor.document.$.anchors ),\r
- anchors = retval.anchors = [];\r
+ var anchors = retval.anchors = [],\r
+ i, count, item;\r
\r
- for ( var i = 0; i < elements.count() ; i++ )\r
+ // For some browsers we set contenteditable="false" on anchors, making document.anchors not to include them, so we must traverse the links manually (#7893).\r
+ if ( CKEDITOR.plugins.link.emptyAnchorFix )\r
{\r
- var item = elements.getItem( i );\r
- if ( item.getAttribute( '_cke_realelement' ) && item.getAttribute( '_cke_real_element_type' ) == 'anchor' )\r
+ var links = editor.document.getElementsByTag( 'a' );\r
+ for ( i = 0, count = links.count(); i < count; i++ )\r
{\r
- anchors.push( editor.restoreRealElement( item ) );\r
+ item = links.getItem( i );\r
+ if ( item.data( 'cke-saved-name' ) || item.hasAttribute( 'name' ) )\r
+ anchors.push( { name : item.data( 'cke-saved-name' ) || item.getAttribute( 'name' ), id : item.getAttribute( 'id' ) } );\r
+ }\r
+ }\r
+ else\r
+ {\r
+ var anchorList = new CKEDITOR.dom.nodeList( editor.document.$.anchors );\r
+ for ( i = 0, count = anchorList.count(); i < count; i++ )\r
+ {\r
+ item = anchorList.getItem( i );\r
+ anchors[ i ] = { name : item.getAttribute( 'name' ), id : item.getAttribute( 'id' ) };\r
}\r
}\r
\r
- for ( i = 0 ; i < realAnchors.count() ; i++ )\r
- anchors.push( realAnchors.getItem( i ) );\r
-\r
- for ( i = 0 ; i < anchors.length ; i++ )\r
+ if ( CKEDITOR.plugins.link.fakeAnchor )\r
{\r
- item = anchors[ i ];\r
- anchors[ i ] = { name : item.getAttribute( 'name' ), id : item.getAttribute( 'id' ) };\r
+ var imgs = editor.document.getElementsByTag( 'img' );\r
+ for ( i = 0, count = imgs.count(); i < count; i++ )\r
+ {\r
+ if ( ( item = CKEDITOR.plugins.link.tryRestoreFakeAnchor( editor, imgs.getItem( i ) ) ) )\r
+ anchors.push( { name : item.getAttribute( 'name' ), id : item.getAttribute( 'id' ) } );\r
+ }\r
}\r
\r
// Record down the selected element in the dialog.\r
this._.selectedElement = element;\r
-\r
return retval;\r
};\r
\r
return 'String.fromCharCode(' + encodedChars.join( ',' ) + ')';\r
}\r
\r
+ function getLinkClass( ele )\r
+ {\r
+ var className = ele.getAttribute( 'class' );\r
+ return className ? className.replace( /\s*(?:cke_anchor_empty|cke_anchor)(?:\s*$)?/g, '' ) : '';\r
+ }\r
+\r
+ var commonLang = editor.lang.common,\r
+ linkLang = editor.lang.link;\r
+\r
return {\r
- title : editor.lang.link.title,\r
+ title : linkLang.title,\r
minWidth : 350,\r
minHeight : 230,\r
contents : [\r
{\r
id : 'info',\r
- label : editor.lang.link.info,\r
- title : editor.lang.link.info,\r
+ label : linkLang.info,\r
+ title : linkLang.info,\r
elements :\r
[\r
{\r
id : 'linkType',\r
type : 'select',\r
- label : editor.lang.link.type,\r
+ label : linkLang.type,\r
'default' : 'url',\r
items :\r
[\r
- [ editor.lang.link.toUrl, 'url' ],\r
- [ editor.lang.link.toAnchor, 'anchor' ],\r
- [ editor.lang.link.toEmail, 'email' ]\r
+ [ linkLang.toUrl, 'url' ],\r
+ [ linkLang.toAnchor, 'anchor' ],\r
+ [ linkLang.toEmail, 'email' ]\r
],\r
onChange : linkTypeChanged,\r
setup : function( data )\r
{\r
id : 'protocol',\r
type : 'select',\r
- label : editor.lang.common.protocol,\r
+ label : commonLang.protocol,\r
'default' : 'http://',\r
items :\r
[\r
[ 'https://\u200E', 'https://' ],\r
[ 'ftp://\u200E', 'ftp://' ],\r
[ 'news://\u200E', 'news://' ],\r
- [ editor.lang.link.other , '' ]\r
+ [ linkLang.other , '' ]\r
],\r
setup : function( data )\r
{\r
{\r
type : 'text',\r
id : 'url',\r
- label : editor.lang.common.url,\r
+ label : commonLang.url,\r
required: true,\r
onLoad : function ()\r
{\r
this.allowOnChange = false;\r
var protocolCmb = this.getDialog().getContentElement( 'info', 'protocol' ),\r
url = this.getValue(),\r
- urlOnChangeProtocol = /^(http|https|ftp|news):\/\/(?=.)/gi,\r
- urlOnChangeTestOther = /^((javascript:)|[#\/\.\?])/gi;\r
+ urlOnChangeProtocol = /^(http|https|ftp|news):\/\/(?=.)/i,\r
+ urlOnChangeTestOther = /^((javascript:)|[#\/\.\?])/i;\r
\r
var protocol = urlOnChangeProtocol.exec( url );\r
if ( protocol )\r
dialog.getValueOf( 'info', 'linkType' ) != 'url' )\r
return true;\r
\r
+ if ( (/javascript\:/).test( this.getValue() ) ) {\r
+ alert( commonLang.invalidValue );\r
+ return false;\r
+ }\r
+\r
if ( this.getDialog().fakeObj ) // Edit Anchor.\r
return true;\r
\r
- var func = CKEDITOR.dialog.validate.notEmpty( editor.lang.link.noUrl );\r
+ var func = CKEDITOR.dialog.validate.notEmpty( linkLang.noUrl );\r
return func.apply( this );\r
},\r
setup : function( data )\r
id : 'browse',\r
hidden : 'true',\r
filebrowser : 'info:url',\r
- label : editor.lang.common.browseServer\r
+ label : commonLang.browseServer\r
}\r
]\r
},\r
{\r
type : 'fieldset',\r
id : 'selectAnchorText',\r
- label : editor.lang.link.selectAnchor,\r
+ label : linkLang.selectAnchor,\r
setup : function( data )\r
{\r
if ( data.anchors.length > 0 )\r
type : 'select',\r
id : 'anchorName',\r
'default' : '',\r
- label : editor.lang.link.anchorName,\r
+ label : linkLang.anchorName,\r
style : 'width: 100%;',\r
items :\r
[\r
type : 'select',\r
id : 'anchorId',\r
'default' : '',\r
- label : editor.lang.link.anchorId,\r
+ label : linkLang.anchorId,\r
style : 'width: 100%;',\r
items :\r
[\r
type : 'html',\r
id : 'noAnchors',\r
style : 'text-align: center;',\r
- html : '<div role="label" tabIndex="-1">' + CKEDITOR.tools.htmlEncode( editor.lang.link.noAnchors ) + '</div>',\r
+ html : '<div role="note" tabIndex="-1">' + CKEDITOR.tools.htmlEncode( linkLang.noAnchors ) + '</div>',\r
// Focus the first element defined in above html.\r
focus : true,\r
setup : function( data )\r
{\r
type : 'text',\r
id : 'emailAddress',\r
- label : editor.lang.link.emailAddress,\r
+ label : linkLang.emailAddress,\r
required : true,\r
validate : function()\r
{\r
dialog.getValueOf( 'info', 'linkType' ) != 'email' )\r
return true;\r
\r
- var func = CKEDITOR.dialog.validate.notEmpty( editor.lang.link.noEmail );\r
+ var func = CKEDITOR.dialog.validate.notEmpty( linkLang.noEmail );\r
return func.apply( this );\r
},\r
setup : function( data )\r
{\r
type : 'text',\r
id : 'emailSubject',\r
- label : editor.lang.link.emailSubject,\r
+ label : linkLang.emailSubject,\r
setup : function( data )\r
{\r
if ( data.email )\r
{\r
type : 'textarea',\r
id : 'emailBody',\r
- label : editor.lang.link.emailBody,\r
+ label : linkLang.emailBody,\r
rows : 3,\r
'default' : '',\r
setup : function( data )\r
},\r
{\r
id : 'target',\r
- label : editor.lang.link.target,\r
- title : editor.lang.link.target,\r
+ label : linkLang.target,\r
+ title : linkLang.target,\r
elements :\r
[\r
{\r
{\r
type : 'select',\r
id : 'linkTargetType',\r
- label : editor.lang.common.target,\r
+ label : commonLang.target,\r
'default' : 'notSet',\r
style : 'width : 100%;',\r
'items' :\r
[\r
- [ editor.lang.common.notSet, 'notSet' ],\r
- [ editor.lang.link.targetFrame, 'frame' ],\r
- [ editor.lang.link.targetPopup, 'popup' ],\r
- [ editor.lang.common.targetNew, '_blank' ],\r
- [ editor.lang.common.targetTop, '_top' ],\r
- [ editor.lang.common.targetSelf, '_self' ],\r
- [ editor.lang.common.targetParent, '_parent' ]\r
+ [ commonLang.notSet, 'notSet' ],\r
+ [ linkLang.targetFrame, 'frame' ],\r
+ [ linkLang.targetPopup, 'popup' ],\r
+ [ commonLang.targetNew, '_blank' ],\r
+ [ commonLang.targetTop, '_top' ],\r
+ [ commonLang.targetSelf, '_self' ],\r
+ [ commonLang.targetParent, '_parent' ]\r
],\r
onChange : targetChanged,\r
setup : function( data )\r
{\r
if ( data.target )\r
- this.setValue( data.target.type );\r
+ this.setValue( data.target.type || 'notSet' );\r
+ targetChanged.call( this );\r
},\r
commit : function( data )\r
{\r
{\r
type : 'text',\r
id : 'linkTargetName',\r
- label : editor.lang.link.targetFrameName,\r
+ label : linkLang.targetFrameName,\r
'default' : '',\r
setup : function( data )\r
{\r
},\r
{\r
type : 'vbox',\r
- width : 260,\r
+ width : '100%',\r
align : 'center',\r
padding : 2,\r
id : 'popupFeatures',\r
[\r
{\r
type : 'fieldset',\r
- label : editor.lang.link.popupFeatures,\r
+ label : linkLang.popupFeatures,\r
children :\r
[\r
{\r
{\r
type : 'checkbox',\r
id : 'resizable',\r
- label : editor.lang.link.popupResizable,\r
+ label : linkLang.popupResizable,\r
setup : setupPopupParams,\r
commit : commitPopupParams\r
},\r
{\r
type : 'checkbox',\r
id : 'status',\r
- label : editor.lang.link.popupStatusBar,\r
+ label : linkLang.popupStatusBar,\r
setup : setupPopupParams,\r
commit : commitPopupParams\r
\r
{\r
type : 'checkbox',\r
id : 'location',\r
- label : editor.lang.link.popupLocationBar,\r
+ label : linkLang.popupLocationBar,\r
setup : setupPopupParams,\r
commit : commitPopupParams\r
\r
{\r
type : 'checkbox',\r
id : 'toolbar',\r
- label : editor.lang.link.popupToolbar,\r
+ label : linkLang.popupToolbar,\r
setup : setupPopupParams,\r
commit : commitPopupParams\r
\r
{\r
type : 'checkbox',\r
id : 'menubar',\r
- label : editor.lang.link.popupMenuBar,\r
+ label : linkLang.popupMenuBar,\r
setup : setupPopupParams,\r
commit : commitPopupParams\r
\r
{\r
type : 'checkbox',\r
id : 'fullscreen',\r
- label : editor.lang.link.popupFullScreen,\r
+ label : linkLang.popupFullScreen,\r
setup : setupPopupParams,\r
commit : commitPopupParams\r
\r
{\r
type : 'checkbox',\r
id : 'scrollbars',\r
- label : editor.lang.link.popupScrollBars,\r
+ label : linkLang.popupScrollBars,\r
setup : setupPopupParams,\r
commit : commitPopupParams\r
\r
{\r
type : 'checkbox',\r
id : 'dependent',\r
- label : editor.lang.link.popupDependent,\r
+ label : linkLang.popupDependent,\r
setup : setupPopupParams,\r
commit : commitPopupParams\r
\r
[\r
{\r
type : 'text',\r
- widths : [ '30%', '70%' ],\r
+ widths : [ '50%', '50%' ],\r
labelLayout : 'horizontal',\r
- label : editor.lang.link.popupWidth,\r
+ label : commonLang.width,\r
id : 'width',\r
setup : setupPopupParams,\r
commit : commitPopupParams\r
{\r
type : 'text',\r
labelLayout : 'horizontal',\r
- widths : [ '55%', '45%' ],\r
- label : editor.lang.link.popupLeft,\r
+ widths : [ '50%', '50%' ],\r
+ label : linkLang.popupLeft,\r
id : 'left',\r
setup : setupPopupParams,\r
commit : commitPopupParams\r
{\r
type : 'text',\r
labelLayout : 'horizontal',\r
- widths : [ '30%', '70%' ],\r
- label : editor.lang.link.popupHeight,\r
+ widths : [ '50%', '50%' ],\r
+ label : commonLang.height,\r
id : 'height',\r
setup : setupPopupParams,\r
commit : commitPopupParams\r
{\r
type : 'text',\r
labelLayout : 'horizontal',\r
- label : editor.lang.link.popupTop,\r
- widths : [ '55%', '45%' ],\r
+ label : linkLang.popupTop,\r
+ widths : [ '50%', '50%' ],\r
id : 'top',\r
setup : setupPopupParams,\r
commit : commitPopupParams\r
},\r
{\r
id : 'upload',\r
- label : editor.lang.link.upload,\r
- title : editor.lang.link.upload,\r
+ label : linkLang.upload,\r
+ title : linkLang.upload,\r
hidden : true,\r
filebrowser : 'uploadButton',\r
elements :\r
{\r
type : 'file',\r
id : 'upload',\r
- label : editor.lang.common.upload,\r
+ label : commonLang.upload,\r
style: 'height:40px',\r
size : 29\r
},\r
{\r
type : 'fileButton',\r
id : 'uploadButton',\r
- label : editor.lang.common.uploadSubmit,\r
+ label : commonLang.uploadSubmit,\r
filebrowser : 'info:url',\r
'for' : [ 'upload', 'upload' ]\r
}\r
},\r
{\r
id : 'advanced',\r
- label : editor.lang.link.advanced,\r
- title : editor.lang.link.advanced,\r
+ label : linkLang.advanced,\r
+ title : linkLang.advanced,\r
elements :\r
[\r
{\r
{\r
type : 'text',\r
id : 'advId',\r
- label : editor.lang.link.id,\r
+ label : linkLang.id,\r
setup : setupAdvParams,\r
commit : commitAdvParams\r
},\r
{\r
type : 'select',\r
id : 'advLangDir',\r
- label : editor.lang.link.langDir,\r
+ label : linkLang.langDir,\r
'default' : '',\r
style : 'width:110px',\r
items :\r
[\r
- [ editor.lang.common.notSet, '' ],\r
- [ editor.lang.link.langDirLTR, 'ltr' ],\r
- [ editor.lang.link.langDirRTL, 'rtl' ]\r
+ [ commonLang.notSet, '' ],\r
+ [ linkLang.langDirLTR, 'ltr' ],\r
+ [ linkLang.langDirRTL, 'rtl' ]\r
],\r
setup : setupAdvParams,\r
commit : commitAdvParams\r
type : 'text',\r
id : 'advAccessKey',\r
width : '80px',\r
- label : editor.lang.link.acccessKey,\r
+ label : linkLang.acccessKey,\r
maxLength : 1,\r
setup : setupAdvParams,\r
commit : commitAdvParams\r
[\r
{\r
type : 'text',\r
- label : editor.lang.link.name,\r
+ label : linkLang.name,\r
id : 'advName',\r
setup : setupAdvParams,\r
commit : commitAdvParams\r
},\r
{\r
type : 'text',\r
- label : editor.lang.link.langCode,\r
+ label : linkLang.langCode,\r
id : 'advLangCode',\r
width : '110px',\r
'default' : '',\r
},\r
{\r
type : 'text',\r
- label : editor.lang.link.tabIndex,\r
+ label : linkLang.tabIndex,\r
id : 'advTabIndex',\r
width : '80px',\r
maxLength : 5,\r
[\r
{\r
type : 'text',\r
- label : editor.lang.link.advisoryTitle,\r
+ label : linkLang.advisoryTitle,\r
'default' : '',\r
id : 'advTitle',\r
setup : setupAdvParams,\r
},\r
{\r
type : 'text',\r
- label : editor.lang.link.advisoryContentType,\r
+ label : linkLang.advisoryContentType,\r
'default' : '',\r
id : 'advContentType',\r
setup : setupAdvParams,\r
[\r
{\r
type : 'text',\r
- label : editor.lang.link.cssClasses,\r
+ label : linkLang.cssClasses,\r
'default' : '',\r
id : 'advCSSClasses',\r
setup : setupAdvParams,\r
},\r
{\r
type : 'text',\r
- label : editor.lang.link.charset,\r
+ label : linkLang.charset,\r
'default' : '',\r
id : 'advCharset',\r
setup : setupAdvParams,\r
},\r
{\r
type : 'hbox',\r
+ widths : [ '45%', '55%' ],\r
children :\r
[\r
{\r
type : 'text',\r
- label : editor.lang.link.styles,\r
+ label : linkLang.rel,\r
+ 'default' : '',\r
+ id : 'advRel',\r
+ setup : setupAdvParams,\r
+ commit : commitAdvParams\r
+ },\r
+ {\r
+ type : 'text',\r
+ label : linkLang.styles,\r
'default' : '',\r
id : 'advStyles',\r
+ validate : CKEDITOR.dialog.validate.inlineStyle( editor.lang.common.invalidInlineStyle ),\r
setup : setupAdvParams,\r
commit : commitAdvParams\r
-\r
}\r
]\r
}\r
],\r
onShow : function()\r
{\r
- this.fakeObj = false;\r
-\r
var editor = this.getParentEditor(),\r
selection = editor.getSelection(),\r
element = null;\r
// Fill in all the relevant fields if there's already one link selected.\r
if ( ( element = plugin.getSelectedLink( editor ) ) && element.hasAttribute( 'href' ) )\r
selection.selectElement( element );\r
- else if ( ( element = selection.getSelectedElement() ) && element.is( 'img' )\r
- && element.getAttribute( '_cke_real_element_type' )\r
- && element.getAttribute( '_cke_real_element_type' ) == 'anchor' )\r
- {\r
- this.fakeObj = element;\r
- element = editor.restoreRealElement( this.fakeObj );\r
- selection.selectElement( this.fakeObj );\r
- }\r
else\r
element = null;\r
\r
},\r
onOk : function()\r
{\r
- var attributes = { href : 'javascript:void(0)/*' + CKEDITOR.tools.getNextNumber() + '*/' },\r
+ var attributes = {},\r
removeAttributes = [],\r
- data = { href : attributes.href },\r
+ data = {},\r
me = this,\r
editor = this.getParentEditor();\r
\r
{\r
case 'url':\r
var protocol = ( data.url && data.url.protocol != undefined ) ? data.url.protocol : 'http://',\r
- url = ( data.url && data.url.url ) || '';\r
- attributes._cke_saved_href = ( url.indexOf( '/' ) === 0 ) ? url : protocol + url;\r
+ url = ( data.url && CKEDITOR.tools.trim( data.url.url ) ) || '';\r
+ attributes[ 'data-cke-saved-href' ] = ( url.indexOf( '/' ) === 0 ) ? url : protocol + url;\r
break;\r
case 'anchor':\r
var name = ( data.anchor && data.anchor.name ),\r
id = ( data.anchor && data.anchor.id );\r
- attributes._cke_saved_href = '#' + ( name || id || '' );\r
+ attributes[ 'data-cke-saved-href' ] = '#' + ( name || id || '' );\r
break;\r
case 'email':\r
\r
}\r
}\r
\r
- attributes._cke_saved_href = linkHref.join( '' );\r
+ attributes[ 'data-cke-saved-href' ] = linkHref.join( '' );\r
break;\r
}\r
\r
addFeature( 'top' );\r
\r
onclickList.push( featureList.join( ',' ), '\'); return false;' );\r
- attributes[ '_cke_pa_onclick' ] = onclickList.join( '' );\r
+ attributes[ 'data-cke-pa-onclick' ] = onclickList.join( '' );\r
+\r
+ // Add the "target" attribute. (#5074)\r
+ removeAttributes.push( 'target' );\r
}\r
else\r
{\r
else\r
removeAttributes.push( 'target' );\r
\r
- removeAttributes.push( '_cke_pa_onclick', 'onclick' );\r
+ removeAttributes.push( 'data-cke-pa-onclick', 'onclick' );\r
}\r
}\r
\r
removeAttributes.push( attrName );\r
};\r
\r
- if ( this._.selectedElement )\r
- advAttr( 'advId', 'id' );\r
+ advAttr( 'advId', 'id' );\r
advAttr( 'advLangDir', 'dir' );\r
advAttr( 'advAccessKey', 'accessKey' );\r
- advAttr( 'advName', 'name' );\r
+\r
+ if ( data.adv[ 'advName' ] )\r
+ attributes[ 'name' ] = attributes[ 'data-cke-saved-name' ] = data.adv[ 'advName' ];\r
+ else\r
+ removeAttributes = removeAttributes.concat( [ 'data-cke-saved-name', 'name' ] );\r
+\r
advAttr( 'advLangCode', 'lang' );\r
advAttr( 'advTabIndex', 'tabindex' );\r
advAttr( 'advTitle', 'title' );\r
advAttr( 'advCSSClasses', 'class' );\r
advAttr( 'advCharset', 'charset' );\r
advAttr( 'advStyles', 'style' );\r
+ advAttr( 'advRel', 'rel' );\r
}\r
\r
+\r
+ var selection = editor.getSelection();\r
+\r
+ // Browser need the "href" fro copy/paste link to work. (#6641)\r
+ attributes.href = attributes[ 'data-cke-saved-href' ];\r
+\r
if ( !this._.selectedElement )\r
{\r
// Create element if current selection is collapsed.\r
- var selection = editor.getSelection(),\r
- ranges = selection.getRanges();\r
+ var ranges = selection.getRanges( true );\r
if ( ranges.length == 1 && ranges[0].collapsed )\r
{\r
// Short mailto link text view (#5736).\r
var text = new CKEDITOR.dom.text( data.type == 'email' ?\r
- data.email.address : attributes._cke_saved_href, editor.document );\r
+ data.email.address : attributes[ 'data-cke-saved-href' ], editor.document );\r
ranges[0].insertNode( text );\r
ranges[0].selectNodeContents( text );\r
selection.selectRanges( ranges );\r
var style = new CKEDITOR.style( { element : 'a', attributes : attributes } );\r
style.type = CKEDITOR.STYLE_INLINE; // need to override... dunno why.\r
style.apply( editor.document );\r
-\r
- // Id. Apply only to the first link.\r
- if ( data.adv && data.adv.advId )\r
- {\r
- var links = this.getParentEditor().document.$.getElementsByTagName( 'a' );\r
- for ( i = 0 ; i < links.length ; i++ )\r
- {\r
- if ( links[i].href == attributes.href )\r
- {\r
- links[i].id = data.adv.advId;\r
- break;\r
- }\r
- }\r
- }\r
}\r
else\r
{\r
// We're only editing an existing link, so just overwrite the attributes.\r
var element = this._.selectedElement,\r
- href = element.getAttribute( '_cke_saved_href' ),\r
+ href = element.data( 'cke-saved-href' ),\r
textView = element.getHtml();\r
\r
- // IE BUG: Setting the name attribute to an existing link doesn't work.\r
- // Must re-create the link from weired syntax to workaround.\r
- if ( CKEDITOR.env.ie && attributes.name != element.getAttribute( 'name' ) )\r
- {\r
- var newElement = new CKEDITOR.dom.element( '<a name="' + CKEDITOR.tools.htmlEncode( attributes.name ) + '">',\r
- editor.document );\r
-\r
- selection = editor.getSelection();\r
-\r
- element.moveChildren( newElement );\r
- element.copyAttributes( newElement, { name : 1 } );\r
- newElement.replace( element );\r
- element = newElement;\r
-\r
- selection.selectElement( element );\r
- }\r
-\r
element.setAttributes( attributes );\r
element.removeAttributes( removeAttributes );\r
+\r
+ if ( data.adv && data.adv.advName && CKEDITOR.plugins.link.synAnchorSelector )\r
+ element.addClass( element.getChildCount() ? 'cke_anchor' : 'cke_anchor_empty' );\r
+\r
// Update text view when user changes protocol (#4612).\r
if ( href == textView || data.type == 'email' && textView.indexOf( '@' ) != -1 )\r
{\r
// Short mailto link text view (#5736).\r
element.setHtml( data.type == 'email' ?\r
- data.email.address : attributes._cke_saved_href );\r
+ data.email.address : attributes[ 'data-cke-saved-href' ] );\r
}\r
- // Make the element display as an anchor if a name has been set.\r
- if ( element.getAttribute( 'name' ) )\r
- element.addClass( 'cke_anchor' );\r
- else\r
- element.removeClass( 'cke_anchor' );\r
-\r
- if ( this.fakeObj )\r
- editor.createFakeElement( element, 'cke_anchor', 'anchor' ).replace( this.fakeObj );\r
\r
+ selection.selectElement( element );\r
delete this._.selectedElement;\r
}\r
},\r
{\r
var linkType = this.getContentElement( 'info', 'linkType' ),\r
urlField;\r
- if ( linkType && linkType.getValue( ) == 'url' )\r
+ if ( linkType && linkType.getValue() == 'url' )\r
{\r
urlField = this.getContentElement( 'info', 'url' );\r
urlField.select();\r