Changeset 7249


Ignore:
Timestamp:
09/01/11 19:44:46 (3 years ago)
Author:
garry.yao
Message:

#8321: Regression of [7242].

Location:
CKEditor/trunk/_source/core/dom
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/trunk/_source/core/dom/element.js

    r7242 r7249  
    720720                }, 
    721721 
    722                 isEditable : function() 
    723                 { 
    724                         if ( this.isReadOnly() || !this.isVisible() ) 
     722                /** 
     723                 * Decide whether one element is able to receive cursor. 
     724                 * @param {Boolean} [textCursor=true] Only consider element that could receive text child. 
     725                 */ 
     726                isEditable : function( textCursor ) 
     727                { 
     728                        var name = this.getName(); 
     729 
     730                        if ( this.isReadOnly() 
     731                                        || this.getComputedStyle( 'display' ) == 'none' 
     732                                        || this.getComputedStyle( 'visibility' ) == 'hidden' 
     733                                        || CKEDITOR.dtd.$nonEditable[ name ] ) 
     734                        { 
    725735                                return false; 
    726  
    727                         // Get the element name. 
    728                         var name = this.getName(); 
    729  
    730                         // Get the element DTD (defaults to span for unknown elements). 
    731                         var dtd = !CKEDITOR.dtd.$nonEditable[ name ] 
    732                                                 && ( CKEDITOR.dtd[ name ] || CKEDITOR.dtd.span ); 
    733  
    734                         // In the DTD # == text node. 
    735                         return ( dtd && dtd['#'] ); 
     736                        } 
     737 
     738                        if ( textCursor !== false ) 
     739                        { 
     740                                // Get the element DTD (defaults to span for unknown elements). 
     741                                var dtd = CKEDITOR.dtd[ name ] || CKEDITOR.dtd.span; 
     742                                // In the DTD # == text node. 
     743                                return ( dtd && dtd[ '#'] ); 
     744                        } 
     745 
     746                        return true; 
    736747                }, 
    737748 
  • CKEditor/trunk/_source/core/dom/range.js

    r7242 r7249  
    19261926 
    19271927                                if ( node.type == CKEDITOR.NODE_ELEMENT 
    1928                                                 && !node.isReadOnly() 
    1929                                                 && node.isVisible() 
     1928                                                && node.isEditable( false ) 
    19301929                                                && !CKEDITOR.dtd.$nonEditable[ node.getName() ] ) 
    19311930                                { 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy