Skip to content

It's possible to delete widget partially during selection #3851

New issue
New issue
Open
Open
It's possible to delete widget partially during selection#3851

Description

@jacekbogdanski

jacekbogdanski
openedon Feb 11, 2020

Type of report

Bug

Provide detailed reproduction steps (if any)

  1. Open tests/plugins/widget/integration/selection/manual/crosscontentselection manual test.
  2. Start selection from the beginning of the text and move the mouse over the widget still keeping the mouse button pressed.
  3. Press the Backspace/Delete key or just start typing.

Expected result

Nothing happens.

Actual result

Part of the widget gets deleted.

image

Other details

The issue is probably caused by timeout for:

ckeditor4/plugins/widget/plugin.js

Lines 2827 to 2833 in e6c0015

// The selectionCheck event is fired on keyup, so we must force refreshing
// widgets selection on key event. Also fire it only in WYSIWYG mode (#3352, #3704).
editor.on( 'contentDom', function() {
editor.editable().attachListener( editor, 'key', function() {
setTimeout( fireCheckSelection, 10 );
} );
} );

However, we need to make sure why the timeout has been introduced and if we can find out better solution for the original issue.

  • Browser: Safari
  • CKEditor version: 4.14.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Assignees

No one assigned

    Labels

    browser:firefoxThe issue can only be reproduced in the Firefox browser.The issue can only be reproduced in the Firefox browser.browser:safariThe issue can only be reproduced in the Safari browser.The issue can only be reproduced in the Safari browser.status:confirmedAn issue confirmed by the development team.An issue confirmed by the development team.type:bugA bug.A bug.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions