Changeset 5103


Ignore:
Timestamp:
02/12/2010 01:15:22 PM (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