Ticket #4880 (closed Bug: fixed)

Opened 4 years ago

Last modified 3 years ago

CKEditor changes tag style inside html comment with cke_protected

Reported by: fabio.perrella Owned by: alfonsoml
Priority: Normal Milestone: CKEditor 3.5.3
Component: Core : Output Data Version: 3.0.2
Keywords: Cc: fabio.perrella, dean@…, vgarikipati@…

Description

Paste the following content in source mode, switch to WYSIWYG and back to source.

<style type="text/css">
<!--
.bloco1_titulo { font-family:Verdana, Arial, Helvetica, sans-serif; font-size:10px; color:#FFFFFF; font-weight:bold; background-color:#333366; padding-left:10px; }
-->
</style>

Result:

<p>
	<style type="text/css">
<!--{cke_protected}%3C!%2D%2D%0A.bloco1_titulo%20%7B%20font-family%3AVerdana%2C%20Arial%2C%20Helvetica%2C%20sans-serif%3B%20font-size%3A10px%3B%20color%3A%23FFFFFF%3B%20font-weight%3Abold%3B%20background-color%3A%23333366%3B%20padding-left%3A10px%3B%20%7D%0A%2D%2D%3E-->
</style></p>

Expected result: No change

Each time you click on source button, the result grows!

Change History

comment:1 Changed 4 years ago by fabio.perrella

If paste this source (without html comment tag), this error doesn't happen

<style type="text/css">
.bloco1_titulo { font-family:Verdana, Arial, Helvetica, sans-serif; font-size:10px; color:#FFFFFF; font-weight:bold; background-color:#333366; padding-left:10px; }
</style>

comment:2 Changed 4 years ago by fredck

  • Priority changed from High to Normal
  • Keywords Confirmed added

comment:3 Changed 4 years ago by alfonsoml

  • Status changed from new to assigned
  • Owner set to alfonsoml

comment:4 Changed 4 years ago by dean.harding

  • Cc dean@… added

Just a comment to say this affects us as well. We just upgraded from FCKeditor 2.x and a client noticed that their styles had stopped working (they had manually entered them in source view before). The workaround is easy enough, and I'll look at manually applying that patch to our code, but this is just to say "me too".

We love CKeditor otherwise, though... just had to say that :)

comment:5 Changed 4 years ago by fredck

  • Milestone CKEditor 3.x deleted

Milestone CKEditor 3.x deleted

comment:6 Changed 3 years ago by wwalc

  • Milestone set to CKEditor 3.5.2

comment:7 Changed 3 years ago by Saare

  • Keywords Confirmed removed
  • Cc vgarikipati@… added

#6922 is a dup.

comment:8 Changed 3 years ago by garry.yao

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

Fixed by #4475.

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