Ticket #4651 (closed Bug: fixed)

Opened 4 months ago

Last modified 4 months ago

[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) (diff)

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

4651.patch Download (1.4 KB) - added by garry.yao 4 months ago.
4651_2.patch Download (0.6 KB) - added by garry.yao 4 months ago.

Change History

Changed 4 months ago by fredck

  • keywords Confirmed added

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

Changed 4 months ago by garry.yao

Changed 4 months 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.

Changed 4 months ago by fredck

  • description modified (diff)

Changed 4 months 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 4 months ago by garry.yao

Changed 4 months ago by garry.yao

  • keywords Review? added; Review- removed

Changed 4 months ago by garry.yao

#4671 has been marked as a DUP.

Changed 4 months ago by fredck

  • keywords Review+ added; Review? removed

Changed 4 months ago by garry.yao

  • status changed from assigned to closed
  • resolution set to fixed

Fixed with [4496].

Note: See TracTickets for help on using tickets.