Opened 6 years ago

Closed 5 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 6 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 Changed 6 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 6 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 6 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 6 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 6 years ago by Garry Yao

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

Changed 6 years ago by Garry Yao

Attachment: 8921.patch added

comment:6 Changed 6 years ago by Garry Yao

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

comment:7 in reply to:  6 Changed 6 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 6 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 5 years ago by Garry Yao

Status: reviewassigned

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

comment:10 Changed 5 years ago by Garry Yao

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