Opened 3 years ago

Closed 3 years ago

#11850 closed Bug (fixed)

applyToAll not set on fakeobject filter

Reported by: dfl Owned by: Reinmar
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 3 years ago by Reinmar

  • Keywords fakeobjects htmlFilter removed
  • Milestone set to CKEditor 4.4.1
  • Status changed from new to confirmed
  • Version changed from 4.3.5 to 4.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 3 years ago by a.nowodzinski

  • Owner set to a.nowodzinski
  • Status changed from confirmed to assigned

comment:3 Changed 3 years ago by Reinmar

  • Owner changed from a.nowodzinski to Reinmar

comment:4 Changed 3 years ago by Reinmar

  • Status changed from assigned to review

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

comment:5 Changed 3 years ago by m.lewandowski

  • Status changed from review to review_passed

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

comment:6 Changed 3 years ago by Reinmar

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

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

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