Opened 12 years ago

Closed 12 years ago

#8921 closed Bug (fixed)

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

Reported by: Alessandro Polverini 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 (1)

8921.patch (960 bytes) - added by Garry Yao 12 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 Changed 12 years ago by Alessandro Polverini

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 12 years ago by Jakub Ś

Keywords: Webkit added; chrome removed
Status: newconfirmed

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 12 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 12 years ago by Jakub Ś

@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 12 years ago by Garry Yao

Last edited 12 years ago by Garry Yao (previous) (diff)

Changed 12 years ago by Garry Yao

Attachment: 8921.patch added

comment:6 Changed 12 years ago by Garry Yao

Milestone: CKEditor 3.6.4
Owner: set to Garry Yao
Status: confirmedreview

comment:7 in reply to:  6 Changed 12 years 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 12 years ago by Jakub Ś

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 12 years ago by Garry Yao

Status: reviewassigned

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

comment:10 Changed 12 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed
Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy