#503: Unordered list paste: doesn't work correctly in IE
----------------------+-----------------------------------------------------
  Reporter:  tba21cn  |       Owner:  martinkou      
      Type:  Bug      |      Status:  assigned       
  Priority:  Normal   |   Milestone:  FCKeditor 2.6  
 Component:  General  |     Version:  FCKeditor 2.4.1
Resolution:           |    Keywords:  Review?        
----------------------+-----------------------------------------------------
Comment (by alfonsoml):

 Two questions about the code.
 In
 {{{
 if ( targetNode.lastChild && targetNode.lastChild.nodeName.IEquals( 'ul' )
 )
         newTarget = targetNode.lastChild ;
 else
         newTarget = FCKXHtml.XML.createElement( 'ul' ) ;
 }}}

 Shouldn't the check be done also with 'ol' like it's done at the start of
 the function?

 and then
 {{{
 FCKXHtml._AppendNode( newTarget, htmlNode ) ;
 }}}
 I've not tried to run it, but I'm not sure that the newTarget element is
 always right. In one case it takes the last node of an existing element,
 and in the other it does create a new element.

 I should check it myself, but I'm afraid that it might be tricky to get
 the exact conditions and maybe you already know that this is right.

-- 
Ticket URL: <http://dev.fckeditor.net/ticket/503#comment:13>
FCKeditor <http://www.fckeditor.net>
The text editor for Internet
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
FCKeditor-Trac mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac

Reply via email to