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

Attachment: 4150.patch added

comment:1 Changed 8 years ago by Garry Yao

Keywords: Review? added
Owner: set to Garry Yao
Status: newassigned

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: fixed
Status: assignedclosed

Fixed with [4030].

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