Opened 7 years ago

Closed 7 years ago

#4651 closed Bug (fixed)

[FF] Deleted and Inserted text styles being rendered

Reported by: JoeK Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.1
Component: Core : Styles Version: 3.0.1
Keywords: IBM Confirmed Review+ Cc: dchojna@…

Description (last modified by fredck)

Using the Ajax sample.

  1. Create an editor instance.
  2. Insert some text.
  3. Select the text and apply the "Deleted Text" style.

Observe that the text does not change.

When you remove the editor from the page the text remains the same. The text does get surrounded by del elements. It may be due to the fact that del and ins elements are not allowed to contain block elements when they are used like inline elements.

Attachments (2)

4651.patch (1.4 KB) - added by garry.yao 7 years ago.
4651_2.patch (641 bytes) - added by garry.yao 7 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 7 years ago by fredck

  • Keywords Confirmed added

<del> and <ins> should be always consider as inline elements in the editor when applying them.

Changed 7 years ago by garry.yao

comment:2 Changed 7 years ago by garry.yao

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

Ticket Test added at :
http://ckeditor.t/tt/4651/1.html.

comment:3 Changed 7 years ago by fredck

  • Description modified (diff)

comment:4 Changed 7 years ago by fredck

  • Keywords Review- added; Review? removed

I think that instead of doing hacks to workaround the issue, we could simply fix the DTD to make <ins> and <del> behave like any other inline only element, like <b>.

Changed 7 years ago by garry.yao

comment:5 Changed 7 years ago by garry.yao

  • Keywords Review? added; Review- removed

comment:6 Changed 7 years ago by garry.yao

#4671 has been marked as a DUP.

comment:7 Changed 7 years ago by fredck

  • Keywords Review+ added; Review? removed

comment:8 Changed 7 years ago by garry.yao

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

Fixed with [4496].

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