X-Git-Url: https://git.immae.eu/?p=perso%2FImmae%2FProjets%2Fpackagist%2Fludivine-ckeditor-component.git;a=blobdiff_plain;f=sources%2Fcore%2Fdom%2Felement.js;fp=sources%2Fcore%2Fdom%2Felement.js;h=31451f9b123064b6919038f3b5988d12b7d56c48;hp=e02ff179f8eaa27b96e7c715b75c7db3bc60baa7;hb=1794320dcfdfcd19572fb1676294f9853a6bbc20;hpb=7183f6a6a21ad9124e70c997e0168459f377a9f2 diff --git a/sources/core/dom/element.js b/sources/core/dom/element.js index e02ff17..31451f9 100644 --- a/sources/core/dom/element.js +++ b/sources/core/dom/element.js @@ -308,7 +308,7 @@ CKEDITOR.dom.element.clearMarkers = function( database, element, removeFromDatab */ appendText: function( text ) { // On IE8 it is impossible to append node to script tag, so we use its text. - // On the contrary, on Safari the text property is unpredictable in links. (#13232) + // On the contrary, on Safari the text property is unpredictable in links. (http://dev.ckeditor.com/ticket/13232) if ( this.$.text != null && CKEDITOR.env.ie && CKEDITOR.env.version < 9 ) this.$.text += text; else @@ -377,7 +377,7 @@ CKEDITOR.dom.element.clearMarkers = function( database, element, removeFromDatab // In case of Internet Explorer, we must check if there is no background-color // added to the element. In such case, we have to overwrite it to prevent "switching it off" - // by a browser (#14667). + // by a browser (http://dev.ckeditor.com/ticket/14667). if ( CKEDITOR.env.ie && !CKEDITOR.env.edge ) { tmpElement = new CKEDITOR.dom.element( 'div' ); @@ -452,7 +452,7 @@ CKEDITOR.dom.element.clearMarkers = function( database, element, removeFromDatab */ getHtml: function() { var retval = this.$.innerHTML; - // Strip tags in IE. (#3341). + // Strip tags in IE. (http://dev.ckeditor.com/ticket/3341). return CKEDITOR.env.ie ? retval.replace( /<\?[^>]*>/g, '' ) : retval; }, @@ -467,7 +467,7 @@ CKEDITOR.dom.element.clearMarkers = function( database, element, removeFromDatab getOuterHtml: function() { if ( this.$.outerHTML ) { // IE includes the tag in the outerHTML of - // namespaced element. So, we must strip it here. (#3341) + // namespaced element. So, we must strip it here. (http://dev.ckeditor.com/ticket/3341) return this.$.outerHTML.replace( /<\?[^>]*>/, '' ); } @@ -618,7 +618,7 @@ CKEDITOR.dom.element.clearMarkers = function( database, element, removeFromDatab return this.$[ name ]; case 'style': - // IE does not return inline styles via getAttribute(). See #2947. + // IE does not return inline styles via getAttribute(). See http://dev.ckeditor.com/ticket/2947. return this.$.style.cssText; case 'contenteditable': @@ -679,7 +679,7 @@ CKEDITOR.dom.element.clearMarkers = function( database, element, removeFromDatab function( propertyName ) { var style = this.getWindow().$.getComputedStyle( this.$, null ); - // Firefox may return null if we call the above on a hidden iframe. (#9117) + // Firefox may return null if we call the above on a hidden iframe. (http://dev.ckeditor.com/ticket/9117) return style ? style.getPropertyValue( propertyName ) : ''; } : function( propertyName ) { return this.$.currentStyle[ CKEDITOR.tools.cssStyleToDomStyle( propertyName ) ]; @@ -972,7 +972,7 @@ CKEDITOR.dom.element.clearMarkers = function( database, element, removeFromDatab elementWindow, elementWindowFrame; // Webkit and Opera report non-zero offsetHeight despite that - // element is inside an invisible iframe. (#4542) + // element is inside an invisible iframe. (http://dev.ckeditor.com/ticket/4542) if ( isVisible && CKEDITOR.env.webkit ) { elementWindow = this.getWindow(); @@ -1033,7 +1033,7 @@ CKEDITOR.dom.element.clearMarkers = function( database, element, removeFromDatab // attribute, which will be marked as "specified", even if the // outerHTML of the element is not displaying the class attribute. // Note : I was not able to reproduce it outside the editor, - // but I've faced it while working on the TC of #1391. + // but I've faced it while working on the TC of http://dev.ckeditor.com/ticket/1391. if ( this.getAttribute( 'class' ) ) { return true; } @@ -1057,7 +1057,7 @@ CKEDITOR.dom.element.clearMarkers = function( database, element, removeFromDatab var attrs = this.$.attributes, attrsNum = attrs.length; - // The _moz_dirty attribute might get into the element after pasting (#5455) + // The _moz_dirty attribute might get into the element after pasting (http://dev.ckeditor.com/ticket/5455) var execludeAttrs = { 'data-cke-expando': 1, _moz_dirty: 1 }; return attrsNum > 0 && ( attrsNum > 2 || !execludeAttrs[ attrs[ 0 ].nodeName ] || ( attrsNum == 2 && !execludeAttrs[ attrs[ 1 ].nodeName ] ) ); @@ -1164,7 +1164,7 @@ CKEDITOR.dom.element.clearMarkers = function( database, element, removeFromDatab 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) + // queuing them to be moved later. (http://dev.ckeditor.com/ticket/5567) var pendingNodes = []; while ( sibling.data( 'cke-bookmark' ) || sibling.isEmptyInlineRemoveable() ) { @@ -1194,7 +1194,7 @@ CKEDITOR.dom.element.clearMarkers = function( database, element, removeFromDatab } return function( inlineOnly ) { - // Merge empty links and anchors also. (#5567) + // Merge empty links and anchors also. (http://dev.ckeditor.com/ticket/5567) if ( !( inlineOnly === false || CKEDITOR.dtd.$removeEmpty[ this.getName() ] || this.is( 'a' ) ) ) { return; } @@ -1253,7 +1253,7 @@ CKEDITOR.dom.element.clearMarkers = function( database, element, removeFromDatab }; } else if ( CKEDITOR.env.ie8Compat && CKEDITOR.env.secure ) { return function( name, value ) { - // IE8 throws error when setting src attribute to non-ssl value. (#7847) + // IE8 throws error when setting src attribute to non-ssl value. (http://dev.ckeditor.com/ticket/7847) if ( name == 'src' && value.match( /^http:\/\// ) ) { try { standard.apply( this, arguments ); @@ -1501,7 +1501,7 @@ CKEDITOR.dom.element.clearMarkers = function( database, element, removeFromDatab clientLeft = $docElem.clientLeft || body.$.clientLeft || 0, needAdjustScrollAndBorders = true; - // #3804: getBoundingClientRect() works differently on IE and non-IE + // http://dev.ckeditor.com/ticket/3804: getBoundingClientRect() works differently on IE and non-IE // browsers, regarding scroll positions. // // On IE, the top position of the element is always 0, no matter @@ -1516,12 +1516,12 @@ CKEDITOR.dom.element.clearMarkers = function( database, element, removeFromDatab needAdjustScrollAndBorders = ( quirks && inBody ) || ( !quirks && inDocElem ); } - // #12747. + // http://dev.ckeditor.com/ticket/12747. if ( needAdjustScrollAndBorders ) { var scrollRelativeLeft, scrollRelativeTop; - // See #12758 to know more about document.(documentElement|body).scroll(Left|Top) in Webkit. + // See http://dev.ckeditor.com/ticket/12758 to know more about document.(documentElement|body).scroll(Left|Top) in Webkit. if ( CKEDITOR.env.webkit || ( CKEDITOR.env.ie && CKEDITOR.env.version >= 12 ) ) { scrollRelativeLeft = body.$.scrollLeft || $docElem.scrollLeft; scrollRelativeTop = body.$.scrollTop || $docElem.scrollTop; @@ -1603,7 +1603,7 @@ CKEDITOR.dom.element.clearMarkers = function( database, element, removeFromDatab parent.$.clientHeight && parent.$.clientHeight < parent.$.scrollHeight; // Skip body element, which will report wrong clientHeight when containing - // floated content. (#9523) + // floated content. (http://dev.ckeditor.com/ticket/9523) if ( overflowed && !parent.is( 'body' ) ) this.scrollIntoParent( parent, alignToTop, 1 ); @@ -1677,7 +1677,7 @@ CKEDITOR.dom.element.clearMarkers = function( database, element, removeFromDatab } // [WebKit] Reset stored scrollTop value to not break scrollIntoView() method flow. - // Scrolling breaks when range.select() is used right after element.scrollIntoView(). (#14659) + // Scrolling breaks when range.select() is used right after element.scrollIntoView(). (http://dev.ckeditor.com/ticket/14659) if ( CKEDITOR.env.webkit ) { var editor = this.getEditor( false ); @@ -1857,7 +1857,7 @@ CKEDITOR.dom.element.clearMarkers = function( database, element, removeFromDatab this.getParent( true ) && this.$.parentNode.replaceChild( newNode.$, this.$ ); newNode.$[ 'data-cke-expando' ] = this.$[ 'data-cke-expando' ]; this.$ = newNode.$; - // Bust getName's cache. (#8663) + // Bust getName's cache. (http://dev.ckeditor.com/ticket/8663) delete this.getName; },