#2909: V3: Move Range.CreateBookmark2 from V2
--------------------------------+-------------------------------------------
 Reporter:  fredck              |       Owner:  fredck      
     Type:  Task                |      Status:  assigned    
 Priority:  Normal              |   Milestone:  CKEditor 3.0
Component:  General             |     Version:              
 Keywords:  Confirmed Review?-  |  
--------------------------------+-------------------------------------------
Changes (by martinkou):

  * keywords:  Confirmed Review? => Confirmed Review?-


Comment:

 There's still a case where the bookmarks that came out of
 createBookmark2() is non-normalized.

 Let's say after running a certain command, a text node in the following
 paragraph is broken up to two pieces:
 {{{
 <p>Part 1 Part 2</p>
 }}}

 And the selection range looks like this, which means it selects "Part 2"
 only:
 {{{
 startContainer : <p>
 startOffset : 1
 }}}

 When we call createBookmark2() with this range:
  1. The normalization logic in createBookmark2() isn't executed, because
 startContainer is not a text node.
  2. The normalization logic in getAddress() is also not executed, for the
 same reason.

 But the index path in the bookmark is certainly not normalized.

-- 
Ticket URL: <http://dev.fckeditor.net/ticket/2909#comment:2>
FCKeditor <http://www.fckeditor.net/>
The text editor for Internet
------------------------------------------------------------------------------
Open Source Business Conference (OSBC), March 24-25, 2009, San Francisco, CA
-OSBC tackles the biggest issue in open source: Open Sourcing the Enterprise
-Strategies to boost innovation and cut costs with open source participation
-Receive a $600 discount off the registration fee with the source code: SFAD
http://p.sf.net/sfu/XcvMzF8H
_______________________________________________
FCKeditor-Trac mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac

Reply via email to