Opened 9 years ago

Closed 7 years ago

#13763 closed Bug (fixed)

First alteration to text after it is pasted in (ctrl+v) does not trigger 'change' event

Reported by: pob Owned by:
Priority: Normal Milestone:
Component: General Version: 4.5.0
Keywords: Cc:

Description

Steps to reproduce

  1. Copy any text
  2. Paste into ckeditor
  3. Edit the text in anyway in one key press
    • enter a character
    • delete a character
    • ctrl+A then backspace

Expected result

Text in ckeditor should be updated, 'change' event triggered, and hidden text (visibility: hidden, display: none) should be updated

Actual result

Text in ckeditor is updated, 'change' event is not triggered, and hidden text is not updated

Other details (browser, OS, CKEditor version, installed plugins)

Tested in most recent versions of Chrome and Firefox; using 'change' event to trigger knockout observable change

Attachments (1)

_test.html (590 bytes) - added by Jakub Ś 9 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 9 years ago by pob

By "paste into ckeditor", I mean paste using the ctrl + V shortcut. My mistake.

comment:2 Changed 9 years ago by pob

Summary: Highlighting and deleting text that is pasted in does not trigger 'change' eventFirst alteration to text after it is pasted in (ctrl+v) does not trigger 'change' event

comment:3 Changed 9 years ago by pob

Further details:

We're using ckeditor in conjunction with knockout using some fun knocking bindings. We can create a textarea to store a knockout observable and then bind to it the data from ckeditor. Not sure if this is entirely relevant, though it should shed some light on what "hidden text (visibility: hidden, display: none) should be update" means.

Changed 9 years ago by Jakub Ś

Attachment: _test.html added

comment:4 Changed 9 years ago by Jakub Ś

Status: newconfirmed
Version: 4.5.34.5.0
  1. Please put attached file into samples folder and open it in a browser.
  2. Open console e.g. Firebug console.
  3. Click inside content area and press Ctrl+V to paste some text. Change event is fired.
  4. Press e.g. 'K' to type a letter.

Result: The first time you press key change event is not fired.

Problem can be reproduced in all browsers from CKEditor 4.5.0.

Version 0, edited 9 years ago by Jakub Ś (next)

comment:6 Changed 8 years ago by Jakub Ś

#14720 was marked as duplicate.

comment:7 Changed 8 years ago by Jakub Ś

#14635 was marked as duplicate.

comment:8 Changed 7 years ago by kkrzton

The PR has been made for this issue: https://github.com/ckeditor/ckeditor-dev/pull/308.

comment:9 Changed 7 years ago by Marek Lewandowski

Resolution: fixed
Status: confirmedclosed

Fixed in GH#554.

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