Ticket #4075: 4075.patch

File 4075.patch, 2.7 KB (added by Garry Yao, 11 years ago)
  • _source/plugins/styles/plugin.js

     
    443443                                {
    444444                                        if ( parent.getName() == elementName )
    445445                                        {
    446                                                 for ( var attName in def.attribs )
     446                                                for ( var attName in def.attributes )
    447447                                                {
    448448                                                        if ( styleNode.getAttribute( attName ) == parent.getAttribute( attName ) )
    449449                                                                styleNode.removeAttribute( attName );
  • _source/core/dom/element.js

     
    774774                                                                // outerHTML of the element is not displaying the class attribute.
    775775                                                                // Note : I was not able to reproduce it outside the editor,
    776776                                                                // but I've faced it while working on the TC of #1391.
    777                                                                 if ( this.getAttribute( 'class' ) > 0 )
     777                                                                if ( this.getAttribute( 'class' ) )
    778778                                                                        return true;
    779779
    780780                                                        // Attributes to be ignored.
  • _source/tests/core/dom/element.html

     
    507507                        assert.isTrue( element.hasAttributes() );
    508508                },
    509509
     510                /**
     511                 * Test detecting of 'class' attribute in IE6/7.
     512                 */
     513                test_hasAttributes_4075 : function()
     514                {
     515                        if ( !CKEDITOR.env.ie && ( CKEDITOR.env.ie7Compat || CKEDITOR.env.ie6Compat ) )
     516                                return;
     517                        var element = new CKEDITOR.dom.element( 'span' );
     518                        element.setAttribute( 'class', 'value' );
     519                        assert.isTrue( element.hasAttributes(), 'Failed to detect "class" attribute existence.' );
     520                },
     521
    510522                test_getDocumentPosition : function()
    511523                {
    512524                        // Assign the page location of the element.
  • CHANGES.html

     
    166166                <li><a href="http://dev.fckeditor.net/ticket/3994">#3994</a> : Insert horizontal line at end of document cause error.</li>
    167167                <li><a href="http://dev.fckeditor.net/ticket/4074">#4074</a> : Indent error with 'indentClasses' config specified.</li>
    168168                <li><a href="http://dev.fckeditor.net/ticket/4057">#4057</a> : Fixed anchor is lost after switch between editing modes.</li>
     169                <li><a href="http://dev.fckeditor.net/ticket/4075">#4075</a> : [IE6/7]Fixed apply custom inline style with "class" attribute failed.</li>
    169170        </ul>
    170171        <h3>
    171172                CKEditor 3.0 RC</h3>
© 2003 – 2020 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy