Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#5170 closed Bug (fixed)

Problems with pasting (executing insertHtml) and formatted text

Reported by: wwalc Owned by: brooks
Priority: Normal Milestone: CKEditor 3.3
Component: General Version: SVN (CKEditor) - OLD
Keywords: Confirmed Review+ Cc:

Description

When inserting content with insertHtml() inside of formatted text, editor breaks formatting and inserts content without formatting. Confirmed in FF 3.5. Works fine in IE8 and Safari 4.

Steps to reproduce

  • start with
    <p>
    	<span style="color: rgb(255, 0, 0);">Sample ^text</span></p>
    
  • Open "special character" dialog and insert some character somewhere in the middle of the text (before "text")
  • Result:
    <p>
    	<span style="color: rgb(255, 0, 0);">Sample </span>&euro;<span style="color: rgb(255, 0, 0);">text</span></p>
    

This can be reproduced in many different ways in CKEditor, everywhere where insertHtml() is used, like also when pasting content or when inserting content in a custom plugin.

Targetting to 3.3 as it might be a bit irritating.

Attachments (2)

5170.patch (2.9 KB) - added by brooks 7 years ago.
5170_2.patch (1.3 KB) - added by brooks 7 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 7 years ago by brooks

  • Version set to SVN (CKEditor)

comment:2 Changed 7 years ago by brooks

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

comfirmed in Firefox3

this is firefox's own bug,it was done by

'trunk/_source/plugins/wysiwygarea/plugin.js' #48 line

this.document.$.execCommand( 'inserthtml', false, data );

so,after a talk with Garry,we decided to use another way to insert special chars

Changed 7 years ago by brooks

Changed 7 years ago by brooks

comment:3 Changed 7 years ago by brooks

  • Keywords Review? added; Confirmed Firefox removed

comment:4 Changed 7 years ago by garry.yao

  • Keywords Review+ added; Review? removed

Cong to Brooks' first commit!

comment:5 Changed 7 years ago by brooks

  • Keywords Review+ removed
  • Resolution set to fixed
  • Status changed from assigned to closed

comment:6 Changed 7 years ago by brooks

  • Keywords Confirmed Review+ added

comment:7 Changed 7 years ago by brooks

fixed with http://dev.fckeditor.net/changeset/5273 ,where,this is still a temporary fix

comment:8 Changed 7 years ago by wwalc

This issue exists everywhere where insertHtml is used, so it shouldn't be fixed in the specialchar dialog, see #5536.

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