X-Git-Url: https://jasonwoof.com/gitweb/?a=blobdiff_plain;f=_source%2Fcore%2Fdom%2Felement.js;h=77031f07f3ebeac03bdf5d4df72f00025ca01d89;hb=refs%2Ftags%2Fv3.4.1;hp=534d6e1ac6a6d834cff7b4ce3ebd4f516a9bc553;hpb=7cd80714081a8ffdf4a1a8d2c72f120ed5ef3d6d;p=ckeditor.git
diff --git a/_source/core/dom/element.js b/_source/core/dom/element.js
index 534d6e1..77031f0 100644
--- a/_source/core/dom/element.js
+++ b/_source/core/dom/element.js
@@ -1,5 +1,5 @@
/*
-Copyright (c) 2003-2009, CKSource - Frederico Knabben. All rights reserved.
+Copyright (c) 2003-2010, CKSource - Frederico Knabben. All rights reserved.
For licensing, see LICENSE.html or http://ckeditor.com/license
*/
@@ -243,10 +243,13 @@ CKEDITOR.tools.extend( CKEDITOR.dom.element.prototype,
lastChild = lastChild.getPrevious();
if ( !lastChild || !lastChild.is || !lastChild.is( 'br' ) )
{
- this.append(
- CKEDITOR.env.opera ?
+ var bogus = CKEDITOR.env.opera ?
this.getDocument().createText('') :
- this.getDocument().createElement( 'br' ) );
+ this.getDocument().createElement( 'br' );
+
+ CKEDITOR.env.gecko && bogus.setAttribute( 'type', '_moz' );
+
+ this.append( bogus );
}
},
@@ -256,15 +259,15 @@ CKEDITOR.tools.extend( CKEDITOR.dom.element.prototype,
* @param {CKEDITOR.dom.element} parent The anscestor element to get broken.
* @example
* // Before breaking:
- * // This is some sample test text
- * // If "element" is and "parent" is :
- * // This is some sample test text
+ * // <b>This <i>is some<span /> sample</i> test text</b>
+ * // If "element" is <span /> and "parent" is <i>:
+ * // <b>This <i>is some</i><span /><i> sample</i> test text</b>
* element.breakParent( parent );
* @example
* // Before breaking:
- * // This is some sample test text
- * // If "element" is and "parent" is :
- * // This is some sample test text
+ * // <b>This <i>is some<span /> sample</i> test text</b>
+ * // If "element" is <span /> and "parent" is <b>:
+ * // <b>This <i>is some</i></b><span /><b><i> sample</i> test text</b>
* element.breakParent( parent );
*/
breakParent : function( parent )
@@ -328,7 +331,9 @@ CKEDITOR.tools.extend( CKEDITOR.dom.element.prototype,
*/
getHtml : function()
{
- return this.$.innerHTML;
+ var retval = this.$.innerHTML;
+ // Strip tags in IE. (#3341).
+ return CKEDITOR.env.ie ? retval.replace( /<\?[^>]*>/g, '' ) : retval;
},
getOuterHtml : function()
@@ -616,7 +621,6 @@ CKEDITOR.tools.extend( CKEDITOR.dom.element.prototype,
}
return (
- /** @ignore */
this.getName = function()
{
return nodeName;
@@ -773,6 +777,31 @@ CKEDITOR.tools.extend( CKEDITOR.dom.element.prototype,
},
/**
+ * Whether it's an empty inline elements which has no visual impact when removed.
+ */
+ isEmptyInlineRemoveable : function()
+ {
+ if ( !CKEDITOR.dtd.$removeEmpty[ this.getName() ] )
+ return false;
+
+ var children = this.getChildren();
+ for ( var i = 0, count = children.count(); i < count; i++ )
+ {
+ var child = children.getItem( i );
+
+ if ( child.type == CKEDITOR.NODE_ELEMENT && child.getAttribute( '_cke_bookmark' ) )
+ continue;
+
+ if ( child.type == CKEDITOR.NODE_ELEMENT && !child.isEmptyInlineRemoveable()
+ || child.type == CKEDITOR.NODE_TEXT && CKEDITOR.tools.trim( child.getText() ) )
+ {
+ return false;
+ }
+ }
+ return true;
+ },
+
+ /**
* Indicates that the element has defined attributes.
* @returns {Boolean} True if the element has attributes.
* @example
@@ -821,8 +850,16 @@ CKEDITOR.tools.extend( CKEDITOR.dom.element.prototype,
:
function()
{
- var attributes = this.$.attributes;
- return ( attributes.length > 1 || ( attributes.length == 1 && attributes[0].nodeName != '_cke_expando' ) );
+ var attrs = this.$.attributes,
+ attrsNum = attrs.length;
+
+ // The _moz_dirty attribute might get into the element after pasting (#5455)
+ var execludeAttrs = { _cke_expando : 1, _moz_dirty : 1 };
+
+ return attrsNum > 0 &&
+ ( attrsNum > 2 ||
+ !execludeAttrs[ attrs[0].nodeName ] ||
+ ( attrsNum == 2 && !execludeAttrs[ attrs[1].nodeName ] ) );
},
/**
@@ -870,6 +907,56 @@ CKEDITOR.tools.extend( CKEDITOR.dom.element.prototype,
}
},
+ mergeSiblings : ( function()
+ {
+ function mergeElements( element, sibling, isNext )
+ {
+ if ( sibling && sibling.type == CKEDITOR.NODE_ELEMENT )
+ {
+ // Jumping over bookmark nodes and empty inline elements, e.g. ,
+ // queuing them to be moved later. (#5567)
+ var pendingNodes = [];
+
+ while ( sibling.getAttribute( '_cke_bookmark' )
+ || sibling.isEmptyInlineRemoveable() )
+ {
+ pendingNodes.push( sibling );
+ sibling = isNext ? sibling.getNext() : sibling.getPrevious();
+ if ( !sibling || sibling.type != CKEDITOR.NODE_ELEMENT )
+ return;
+ }
+
+ if ( element.isIdentical( sibling ) )
+ {
+ // Save the last child to be checked too, to merge things like
+ // =>
+ var innerSibling = isNext ? element.getLast() : element.getFirst();
+
+ // Move pending nodes first into the target element.
+ while( pendingNodes.length )
+ pendingNodes.shift().move( element, !isNext );
+
+ sibling.moveChildren( element, !isNext );
+ sibling.remove();
+
+ // Now check the last inner child (see two comments above).
+ if ( innerSibling && innerSibling.type == CKEDITOR.NODE_ELEMENT )
+ innerSibling.mergeSiblings();
+ }
+ }
+ }
+
+ return function()
+ {
+ // Merge empty links and anchors also. (#5567)
+ if ( !( CKEDITOR.dtd.$removeEmpty[ this.getName() ] || this.is( 'a' ) ) )
+ return;
+
+ mergeElements( this, this.getNext(), true );
+ mergeElements( this, this.getPrevious() );
+ };
+ } )(),
+
/**
* Shows this element (display it).
* @example
@@ -987,8 +1074,16 @@ CKEDITOR.tools.extend( CKEDITOR.dom.element.prototype,
removeAttributes : function ( attributes )
{
- for ( var i = 0 ; i < attributes.length ; i++ )
- this.removeAttribute( attributes[ i ] );
+ if ( CKEDITOR.tools.isArray( attributes ) )
+ {
+ for ( var i = 0 ; i < attributes.length ; i++ )
+ this.removeAttribute( attributes[ i ] );
+ }
+ else
+ {
+ for ( var attr in attributes )
+ attributes.hasOwnProperty( attr ) && this.removeAttribute( attr );
+ }
},
/**
@@ -1075,11 +1170,13 @@ CKEDITOR.tools.extend( CKEDITOR.dom.element.prototype,
function()
{
this.$.style.MozUserSelect = 'none';
+ this.on( 'dragstart', function (evt) { evt.data.preventDefault(); } );
}
: CKEDITOR.env.webkit ?
function()
{
this.$.style.KhtmlUserSelect = 'none';
+ this.on( 'dragstart', function (evt) { evt.data.preventDefault(); } );
}
:
function()
@@ -1344,7 +1441,7 @@ CKEDITOR.tools.extend( CKEDITOR.dom.element.prototype,
if ( attrName in skipAttributes )
continue;
- if( attrName == 'checked' && ( attrValue = this.getAttribute( attrName ) ) )
+ if ( attrName == 'checked' && ( attrValue = this.getAttribute( attrName ) ) )
dest.setAttribute( attrName, attrValue );
// IE BUG: value attribute is never specified even if it exists.
else if ( attribute.specified ||
@@ -1385,7 +1482,7 @@ CKEDITOR.tools.extend( CKEDITOR.dom.element.prototype,
this.moveChildren( newNode );
// Replace the node.
- this.$.parentNode.replaceChild( newNode.$, this.$ );
+ this.getParent() && this.$.parentNode.replaceChild( newNode.$, this.$ );
newNode.$._cke_expando = this.$._cke_expando;
this.$ = newNode.$;
},
@@ -1425,5 +1522,35 @@ CKEDITOR.tools.extend( CKEDITOR.dom.element.prototype,
if ( !event.data.getTarget().hasClass( 'cke_enable_context_menu' ) )
event.data.preventDefault();
} );
- }
+ },
+
+ /**
+ * Update the element's size with box model awareness.
+ * @name CKEDITOR.dom.element.setSize
+ * @param {String} type [width|height]
+ * @param {Number} size The length unit in px.
+ * @param isBorderBox Apply the {@param width} and {@param height} based on border box model.
+ */
+ setSize : ( function()
+ {
+ var sides = {
+ width : [ "border-left-width", "border-right-width","padding-left", "padding-right" ],
+ height : [ "border-top-width", "border-bottom-width", "padding-top", "padding-bottom" ]
+ };
+
+ return function( type, size, isBorderBox )
+ {
+ if ( typeof size == 'number' )
+ {
+ if ( isBorderBox && !( CKEDITOR.env.ie && CKEDITOR.env.quirks ) )
+ {
+ var adjustment = 0;
+ for ( var i = 0, len = sides [ type ].length; i < len; i++ )
+ adjustment += parseInt( this.getComputedStyle( sides [ type ][ i ] ) || 0, 10 );
+ size -= adjustment;
+ }
+ this.setStyle( type, size + 'px' );
+ }
+ };
+ })()
});