Opened 8 years ago

Closed 8 years ago

#4150 closed Bug (fixed)

TCs in core/dom/range are broken

Reported by: garry.yao Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.0
Component: Core : Styles Version:
Keywords: Confirmed Review+ Cc:

Description

test_extractContents series TCs are broken in "CKEDITOR.dom.range" because of [4014].

Attachments (1)

4150.patch​ (6.2 KB) - added by garry.yao 8 years ago.

Download all attachments as: .zip

Change History (4)

Changed 8 years ago by garry.yao

comment:1 Changed 8 years ago by garry.yao

  • Keywords Review? added
  • Owner set to garry.yao
  • Status changed from new to assigned

Changes of [4014] isn't valid, what we really need to fix is the CKEDITOR.dom.range::enlarge( CKEDITOR.ENLARGE_LIST_ITEM_CONTENTS ) where it's result incorrect with the following case: 

<p><span>[Enlarge List Content]</span></p> 
// -> enlarged as:
<p><span>[Enlarge List Content]</span></p> 
// -> should be enlarged as:
<p>[<span>Enlarge List Content</span>]</p>

comment:2 Changed 8 years ago by tobiasz.cudnik

  • Keywords Review+ added; Review? removed

All browsers are passing CKEDITOR.dom.range test with this patch.

comment:3 Changed 8 years ago by garry.yao

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed with [4030].

Note: See TracTickets for help on using tickets.
© 2003 – 2016 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy