Opened 15 years ago

Closed 15 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 15 years ago.

Download all attachments as: .zip

Change History (4)

Changed 15 years ago by Garry Yao

Attachment: 4150.patch​ added

comment:1 Changed 15 years ago by Garry Yao

Keywords: Review? added
Owner: set to Garry Yao
Status: new → 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 15 years ago by Tobiasz Cudnik

Keywords: Review+ added; Review? removed

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

comment:3 Changed 15 years ago by Garry Yao

Resolution: → fixed
Status: assigned → closed

Fixed with [4030].

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy