Pressing backspace near (but not next to) widget selects it.

[Dusty Jewett]

Steps to reproduce

1. Open Placeholder demo
2. Select all text, delete it
3. Type 'Testing '
4. click placeholder button
5. hit right arrow, as placeholder is selected (boo)
6. hit Enter
7. type a letter
8. press left arrow key
9. press backspace

Expected result

Paragraphs are merged together, as they are when no widget is present

Actual result

Widget is selected, paragraphs don't change.

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

4.5.x, Chrome

Investigation

The issue happens inside of selection.js#getOnKeyDownListener, where it checks for getPreviousEditableNode. It finds the widget, but assumes that the backspace key would have deleted the widget. Instead, it should check if the text and widget are adjacent, maybe with next.getParent().contains(range.startContainer)?

[Dusty Jewett]

Submitted PR: ​https://github.com/ckeditor/ckeditor-dev/pull/211

[Piotrek Koszuliński]

It would make sense for the delete and backspace to remove a line break instead of moving the selection to the widget. However, that's a lot of work as the number of cases is pretty big.

[Jakub Ś]

In Android it is not possible to delete widget (using android keyboard) even after selecing it.

[Jakub Ś]

This issue has been reported on our support channel.

There are other TCs:

• Go to: ​http://ckeditor.com/demo#widgets
• Empty first paragraph, when there is no content in that paragraph, try deleting it by pressing delete. Same bug exists with backspace for paragraph after widget.

Actual Result: Empty paragraph won't be deleted. Instead widget is focused.
Expected Result: Paragraph will be deleted AND widget will be focused.

This also means that paragraph between two widgets cannot be removed at all without source editing.

[Tade0]

I have a working implementation, but so far the unit tests I created behave differently than the manual test.

Investigating.

[Tade0]

There's a blocking issue in IE: ​http://dev.ckeditor.com/ticket/16870#ticket

There's also an issue in Firefox, but it doesn't reproduce on major even though none of the code added in t/13703 is executed when reproducing it.

[Tade0]

There's a lingering issue in Firefox, namely in the manual test in the second editor(enterMode: CKEDITOR.ENTER_BR) if you place the caret before the "G" and hit backspace the elements in the previous paragraph switch places.

If you add a breakpoint say here: ​https://github.com/ckeditor/ckeditor-dev/blob/a3b2b66e4403375372875f7834d9ec9539203abd/core/event.js#L285 With the condition eventName == 'blur' && i == 1, the bug will occur after the current tick is over.

This means there's some asynchronous event after that handler and it's probably the one responsible for the bug.