Opened 3 years ago

Closed 3 years ago

#13879 closed Bug (fixed)

It is not possible to prevent text from dropping.

Reported by: Jakub Ś Owned by: Tomasz Jakut
Priority: Normal Milestone: CKEditor 4.5.5
Component: General Version: 4.5.0
Keywords: Support Cc:

Description

Steps to reproduce

  1. Use below code in CKEditor
    CKEDITOR.replace( 'editor1', {
                    on: {
                        instanceReady: function( evt ) {
                            evt.editor.document.on( 'drop',  function(ev){
                                console.log('precented');
                                ev.data.$.preventDefault();
                             });
                        }
                    }
                } );
    
  2. Select and D&D some text.

Expected result

Text should not be dropped.

Actual result

Text is dropped.

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

Happens in all browsers starting from CKEditor 4.5.0

Change History (6)

comment:1 Changed 3 years ago by Marek Lewandowski

Milestone: CKEditor 4.5.5

comment:2 Changed 3 years ago by Jakub Ś

Keywords: Support added
Milestone: CKEditor 4.5.5
Status: newconfirmed

comment:3 Changed 3 years ago by Marek Lewandowski

Milestone: CKEditor 4.5.5

comment:4 Changed 3 years ago by Tomasz Jakut

Owner: set to Tomasz Jakut
Status: confirmedassigned

comment:5 Changed 3 years ago by Tomasz Jakut

Status: assignedreview

There wasn't any check if the event hadn't been prevented in the drop handler. Pushed fix to branch:t/13879.

comment:6 Changed 3 years ago by Marek Lewandowski

Resolution: fixed
Status: reviewclosed

Well done! Merged to master with git:0f7494b.

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