Changeset 5103


Ignore:
Timestamp:
02/12/10 13:15:22 (5 years ago)
Author:
fredck
Message:

#4886 : Extra BR tags where being created in the output HTML.

Location:
CKEditor/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/trunk/CHANGES.html

    r5102 r5103  
    8989                <li><a href="http://dev.fckeditor.net/ticket/4893">#4893</a> : Editor should understand table cell inline styles.</li> 
    9090                <li><a href="http://dev.fckeditor.net/ticket/4611">#4611</a> : Selection around &lt;select&gt; in editor doesn't anymore cause error.</li> 
     91                <li><a href="http://dev.fckeditor.net/ticket/4886">#4886</a> : Extra BR tags where being created in the output HTML.</li> 
    9192                <li>Updated the following language files:<ul> 
    9293                        <li><a href="http://dev.fckeditor.net/ticket/5006">#5006</a> : Dutch;</li> 
  • CKEditor/trunk/_source/core/htmlparser/fragment.js

    r4858 r5103  
    6565                        fragment = new CKEDITOR.htmlParser.fragment(), 
    6666                        pendingInline = [], 
     67                        pendingBRs = [], 
    6768                        currentNode = fragment, 
    6869                    // Indicate we're inside a <pre> element, spaces should be touched differently. 
     
    7273                function checkPending( newTagName ) 
    7374                { 
     75                        var pendingBRsSent; 
     76 
    7477                        if ( pendingInline.length > 0 ) 
    7578                        { 
     
    8386                                        if ( ( !currentDtd || currentDtd[ pendingName ] ) && ( !newTagName || !pendingDtd || pendingDtd[ newTagName ] || !CKEDITOR.dtd[ newTagName ] ) ) 
    8487                                        { 
     88                                                if ( !pendingBRsSent ) 
     89                                                { 
     90                                                        sendPendingBRs(); 
     91                                                        pendingBRsSent = 1; 
     92                                                } 
     93 
    8594                                                // Get a clone for the pending element. 
    8695                                                pendingElement = pendingElement.clone(); 
     
    98107                                } 
    99108                        } 
     109                } 
     110 
     111                function sendPendingBRs() 
     112                { 
     113                        while ( pendingBRs.length ) 
     114                                currentNode.add( pendingBRs.shift() ); 
    100115                } 
    101116 
     
    182197                        } 
    183198 
     199                        if ( tagName == 'br' ) 
     200                        { 
     201                                pendingBRs.push( element ); 
     202                                return; 
     203                        } 
     204 
    184205                        var currentName = currentNode.name; 
    185206 
     
    309330                                currentNode = candidate; 
    310331 
    311                                 if( currentNode.name == 'pre' ) 
     332                                if ( currentNode.name == 'pre' ) 
    312333                                        inPre = false; 
     334 
     335                                if ( candidate._.isBlockLike ) 
     336                                        sendPendingBRs(); 
    313337 
    314338                                addElement( candidate, candidate.parent ); 
     
    322346                        } 
    323347 
    324                         if( tagName == 'body' ) 
     348                        if ( tagName == 'body' ) 
    325349                                fixForBody = false; 
    326350                }; 
     
    337361                        } 
    338362 
     363                        sendPendingBRs(); 
    339364                        checkPending(); 
    340365 
     
    366391                // Parse it. 
    367392                parser.parse( fragmentHtml ); 
     393 
     394                sendPendingBRs(); 
    368395 
    369396                // Close all pending nodes. 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy