Opened 4 years ago

Closed 4 years ago

#11850 closed Bug (fixed)

applyToAll not set on fakeobject filter

Reported by: (David *)Frenkiel Owned by: Piotrek Koszuliński
Priority: Normal Milestone: CKEditor 4.4.1
Component: Core : Parser Version: 4.3
Keywords: Cc:

Description

Create a fake object. The result is an image node. Set contentEditable to "false" on this node. Switch to source mode. Expected: The source contains the original object. Actual: The source contains the fake <img> markup.

Change History (6)

comment:1 Changed 4 years ago by Piotrek Koszuliński

Keywords: fakeobjects htmlFilter removed
Milestone: CKEditor 4.4.1
Status: newconfirmed
Version: 4.3.54.3

I don't know any reason why would the "downcast" filter be only applied to editable content, so let's check if we can change this.

comment:2 Changed 4 years ago by Olek Nowodziński

Owner: set to Olek Nowodziński
Status: confirmedassigned

comment:3 Changed 4 years ago by Piotrek Koszuliński

Owner: changed from Olek Nowodziński to Piotrek Koszuliński

comment:4 Changed 4 years ago by Piotrek Koszuliński

Status: assignedreview

I pushed branch:t/11850 on dev and tests.

comment:5 Changed 4 years ago by Marek Lewandowski

Status: reviewreview_passed

Fix solves the issue. I've force pushed branches rebased on master.

comment:6 Changed 4 years ago by Piotrek Koszuliński

Resolution: fixed
Status: review_passedclosed

Fixed on master with git:0f68337 on dev and 1267405 on tests.

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