Ticket #8921 (closed Bug: fixed)

Opened 2 years ago

Last modified 22 months ago

Paste on a newly created ckeditor instance does not work on chrome

Reported by: Alex Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.6.4
Component: Core : Pasting Version: 3.6.3
Keywords: Webkit Cc:

Description

Steps to reproduce:

  • open a page where CK is instantiated
  • click in edit area
  • paste some text

What happens on Firefox: the pasted text appears in the editor What happens on Chrome: the caret disappears and no text is entered in the editor

Attachments

8921.patch (960 bytes) - added by garry.yao 23 months ago.

Change History

comment:1 Changed 2 years ago by Alex

After a bit of testing I discovered that the problem arise only when CK is istantiated with those options:

CKEDITOR.replace( 'area',{
 autoParagraph : false,
 enterMode : CKEDITOR.ENTER_BR
 });

comment:2 Changed 2 years ago by j.swiderski

  • Status changed from new to confirmed
  • Keywords Webkit added; chrome removed

Reproducible from CKEditor 3.6.3 rev. [7383]

Just insert one or both of the below:

config.autoParagraph = false;
config.enterMode = CKEDITOR.ENTER_BR;

comment:3 Changed 2 years ago by westcoastbc

I have created a test program at: http://www.adreflex.com/test/testedit.cfm

that illustrates the problem.

I have set: config.enterMode = CKEDITOR.ENTER_P;

and the problem remains

comment:4 Changed 2 years ago by j.swiderski

@westcoastbc I have written

one or both of the below

what means that only one of those options can cause this problem.

Yes you have set config.enterMode = CKEDITOR.ENTER_P; but you have also set CKEDITOR.config.autoParagraph = false; which causes the problem.

comment:5 Changed 23 months ago by garry.yao

Last edited 23 months ago by garry.yao (previous) (diff)

Changed 23 months ago by garry.yao

comment:6 follow-up: ↓ 7 Changed 23 months ago by garry.yao

  • Owner set to garry.yao
  • Status changed from confirmed to review
  • Milestone set to CKEditor 3.6.4

comment:7 in reply to: ↑ 6 Changed 23 months ago by westcoastbc

Replying to garry.yao: How do I implement the above patch? I have added the code in 8921.patch to _source\plugins\clipboard\plugin.js but the problem remains.

comment:8 Changed 23 months ago by j.swiderski

And what CKEditor script are you using on your page ckeditor.js or ckeditor_source.js? Changes in _source will be only visible when using ckeditor_source.js but this is good only for development. In release code ckeditor.js should be use.

Proper flow to apply the patch is to get SVN version of CKEditor, apply patch build new release. Please see comment from http://dev.ckeditor.com/ticket/8477#comment:3 and doc file from http://dev.ckeditor.com/ticket/8477#comment:4.

comment:9 Changed 22 months ago by garry.yao

  • Status changed from review to assigned

I've moved the review request to #9042 instead.

comment:10 Changed 22 months ago by garry.yao

  • Status changed from assigned to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy